decidim-consultations 0.19.1 → 0.23.0

data/app/types/decidim/consultations/consultation_type.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Consultations
+    # This type represents a consultation.
+    ConsultationType = GraphQL::ObjectType.define do
+      interfaces [
+        -> { Decidim::Core::ParticipatorySpaceInterface }
+      ]
+
+      name "Consultation"
+      description "A consultation"
+
+      field :subtitle, Decidim::Core::TranslatedFieldType, "The subtitle of this consultation"
+      field :description, Decidim::Core::TranslatedFieldType, "The description of this consultation"
+      field :slug, !types.String, "Slug of this consultation"
+      field :createdAt, !Decidim::Core::DateTimeType, "The time this consultation was created", property: :created_at
+      field :updatedAt, !Decidim::Core::DateTimeType, "The time this consultation was updated", property: :updated_at
+      field :publishedAt, !Decidim::Core::DateTimeType, "The time this consultation was published", property: :published_at
+
+      field :introductoryVideoUrl, types.String, "The introductory video url for this consultation", property: :introductory_video_url
+      field :introductoryImage, types.String, "The introductory image for this consultation", property: :introductory_image
+      field :bannerImage, types.String, "The banner image for this consultation", property: :banner_image
+      field :highlightedScope, Decidim::Core::ScopeApiType, "This is the highlighted scope of this consultation", property: :highlighted_scope
+      field :startVotingDate, Decidim::Core::DateType, "Start date of the voting for this consultation", property: :start_voting_date
+      field :endVotingDate, Decidim::Core::DateType, "End date of the voting for this consultation", property: :end_voting_date
+      field :resultsPublishedAt, Decidim::Core::DateType, "Date when the results have been published", property: :results_published_at
+
+      field :questions, types[Decidim::Consultations::ConsultationQuestionType], ""
+    end
+  end
+end

data/app/views/decidim/consultations/admin/consultations/index.html.erb

@@ -1,4 +1,4 @@
-<div class="card" id="consultations">
+<div class="card with-overflow" id="consultations">
   <div class="card-divider">
     <h2 class="card-title">
       <%= t "decidim.admin.titles.consultations" %>
@@ -6,13 +6,14 @@
                   ["new", "consultation"], class: "button tiny button--title" if allowed_to? :create, :consultation %>
     </h2>
   </div>
+  <%= admin_filter_selector %>
   <div class="card-section">
     <div class="table-scroll">
       <table class="table-list">
         <thead>
           <tr>
             <th><%= t("models.consultation.fields.title", scope: "decidim.admin") %></th>
-            <th><%= t("models.consultation.fields.created_at", scope: "decidim.admin") %></th>
+            <th><%= sort_link(query, :created_at, t("models.consultation.fields.created_at", scope: "decidim.admin"), default_order: :desc) %></th>
             <th class="table-list__actions">
               <%= t("models.consultation.fields.published", scope: "decidim.admin") %>
             </th>

data/app/views/decidim/consultations/admin/response_groups/_form.html.erb

@@ -0,0 +1,11 @@
+<div class="card">
+  <div class="card-divider">
+    <h2 class="card-title"><%= t "response_groups.form.title", scope: "decidim.admin" %></h2>
+  </div>
+
+  <div class="card-section">
+    <div class="row column">
+      <%= form.translated :text_field, :title, autofocus: true %>
+    </div>
+  </div>
+</div>

data/app/views/decidim/consultations/admin/response_groups/edit.html.erb

@@ -0,0 +1,18 @@
+<%= decidim_form_for @form,
+                     url: response_group_path(current_question, current_response_group),
+                     html: { class: "form edit_response_group" } do |f| %>
+  <%= render partial: "form", object: f %>
+  <div class="button--double form-general-submit">
+    <%= f.submit t("response_groups.edit.update", scope: "decidim.admin"),
+                 class: "button",
+                 data: { disable_with: true } %>
+
+    <% if allowed_to? :destroy, :response_group, response_group: current_response_group %>
+      <%= link_to t("decidim.admin.actions.destroy"),
+                  response_group_path(current_question, current_response_group),
+                  method: :delete,
+                  class: "alert button",
+                  data: { confirm: t("decidim.admin.actions.confirm_destroy") } %>
+    <% end %>
+  </div>
+<% end %>

data/app/views/decidim/consultations/admin/response_groups/index.html.erb

@@ -0,0 +1,56 @@
+<div class="card" id="questions">
+  <div class="card-divider">
+    <h2 class="card-title">
+      <%= t "decidim.admin.titles.response_groups" %>
+      <%= link_to t("actions.new_response_group", scope: "decidim.admin"),
+                  new_response_group_path(current_question),
+                  class: "button tiny button--title" if allowed_to? :create, :response_group, question: current_question %>
+      <%= link_to t("actions.back_to_responses", scope: "decidim.admin"),
+                  responses_path(current_question),
+                  class: "button tiny button--title" if allowed_to? :read, :response %>
+    </h2>
+  </div>
+  <div class="card-section">
+    <p>
+      <%= t "decidim.admin.response_groups.help" %>
+    </p>
+  </div>
+  <div class="card-section">
+    <div class="table-scroll">
+      <table class="table-list">
+        <thead>
+          <tr>
+            <th><%= t("models.response.fields.title", scope: "decidim.admin") %></th>
+            <th><%= t("models.response.fields.created_at", scope: "decidim.admin") %></th>
+            <th></th>
+          </tr>
+        </thead>
+        <tbody>
+          <% current_question.response_groups.each do |group| %>
+            <tr>
+              <td>
+                <% if allowed_to? :update, :response_group, response_group: group %>
+                  <%= link_to translated_attribute(group.title),
+                              edit_response_group_path(current_question, group) %>
+                <% else %>
+                  <%= translated_attribute(group.title) %>
+                <% end %>
+              </td>
+              <td>
+                <%= l group.created_at, format: :short %>
+              </td>
+              <td class="table-list__actions">
+                <% if allowed_to? :update, :response_group, response_group: group %>
+                  <%= icon_link_to "pencil",
+                                   edit_response_group_path(current_question, group),
+                                   t("actions.configure", scope: "decidim.admin"),
+                                   class: "action-icon--edit" %>
+                <% end %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+  </div>
+</div>

data/app/views/decidim/consultations/admin/response_groups/new.html.erb

@@ -0,0 +1,13 @@
+<h2 class="questions-title-summary">
+  <%= t "response_groups.new.title", scope: "decidim.admin" %>
+</h2>
+
+<%= decidim_form_for @form,
+                     url: response_groups_path(current_question),
+                     html: { class: "form new_response_group" } do |f| %>
+  <%= render partial: "form", object: f %>
+
+  <div class="button--double form-general-submit">
+    <%= f.submit t("response_groups.new.create", scope: "decidim.admin"), data: { disable_with: true } %>
+  </div>
+<% end %>

data/app/views/decidim/consultations/admin/responses/_form.html.erb

@@ -6,6 +6,9 @@
   <div class="card-section">
     <div class="row column">
       <%= form.translated :text_field, :title, autofocus: true %>
+      <% if allowed_to? :create, :response_group, question: current_question %>
+        <%= form.collection_select :decidim_consultations_response_group_id, question_response_groups, :id, :title %>
+      <% end %>
     </div>
   </div>
 </div>

data/app/views/decidim/consultations/admin/responses/index.html.erb

@@ -5,6 +5,9 @@
       <%= link_to t("actions.new_response", scope: "decidim.admin"),
                   new_response_path(current_question),
                   class: "button tiny button--title" if allowed_to? :create, :response %>
+      <%= link_to t("actions.response_groups", scope: "decidim.admin"),
+                  response_groups_path(current_question),
+                  class: "button tiny button--title" if allowed_to? :create, :response_group, question: current_question %>
     </h2>
   </div>
   <div class="card-section">
@@ -13,12 +16,15 @@
         <thead>
           <tr>
             <th><%= t("models.response.fields.title", scope: "decidim.admin") %></th>
+            <% if allowed_to? :create, :response_group, question: current_question %>
+              <th><%= t("models.response.fields.response_group", scope: "decidim.admin") %></th>
+            <% end %>
             <th><%= t("models.response.fields.created_at", scope: "decidim.admin") %></th>
             <th></th>
           </tr>
         </thead>
         <tbody>
-          <% current_question.responses.each do |response| %>
+          <% current_question.sorted_responses.each do |response| %>
             <tr>
               <td>
                 <% if allowed_to? :update, :response, response: response %>
@@ -28,6 +34,11 @@
                   <%= translated_attribute(response.title) %>
                 <% end %>
               </td>
+              <% if allowed_to? :create, :response_group, question: current_question %>
+              <td>
+                  <%= translated_attribute(response&.response_group&.title) %>
+              </td>
+              <% end %>
               <td>
                 <%= l response.created_at, format: :short %>
               </td>

data/app/views/decidim/consultations/consultations/_consultation_card.html.erb

@@ -1,12 +1,12 @@
 <div class="column">
-  <article class="card card--process">
+  <div class="card card--process">
     <%= link_to consultation_path(consultation), class: "card__link" do %>
       <div class="card__image-top"
            style="background-image:url(<%= consultation.banner_image.url %>)"></div>
     <% end %>
     <div class="card__content">
       <%= link_to consultation_path(consultation), class: "card__link" do %>
-        <h4 class="card__title"><%= translated_attribute consultation.title %></h4>
+        <span class="card__title"><%= translated_attribute consultation.title %></span>
       <% end %>
       <p class="card__desc">
         <%= decidim_sanitize html_truncate(translated_attribute(consultation.subtitle), length: 630, separator: "...") %>
@@ -20,5 +20,5 @@
         <% end %>
       </div>
     </div>
-  </article>
+  </div>
 </div>

data/app/views/decidim/consultations/consultations/_consultation_details.html.erb

@@ -1,6 +1,6 @@
 <div class="row section" id="consultation-details">
   <div class="columns medium-6 large-5 large-push-1">
-    <p class="lead"><%= decidim_sanitize translated_attribute consultation.description %></p>
+    <p class="lead"><%= decidim_sanitize translated_attribute(consultation.description), strip_tags: true %></p>
   </div>
   <div class="columns medium-6 large-5 large-pull-1">
     <% if consultation.introductory_video_url.blank? %>
@@ -9,7 +9,7 @@
       </div>
     <% else %>
       <div class="embed-container">
-        <iframe src="<%= consultation.introductory_video_url %>" frameborder=0 allowfullscreen>
+        <iframe src="<%= consultation.introductory_video_url %>" allowfullscreen>
         </iframe>
       </div>
     <% end %>

data/app/views/decidim/consultations/consultations/_filters.html.erb

@@ -1,3 +1,5 @@
+<%= render partial: "decidim/shared/filter_form_help", locals: { skip_to_id: "consultations" } %>
+
 <%= filter_form_for filter do |form| %>
   <div class="filters__section">
     <div class="filters__search">
@@ -5,10 +7,12 @@
         <%= form.search_field :search_text,
                               label: false,
                               class: "input-group-field",
-                              placeholder: t("consultations.filters.search", scope: "decidim") %>
+                              placeholder: t("consultations.filters.search", scope: "decidim"),
+                              title: t("consultations.filters.search", scope: "decidim"),
+                              data: { disable_dynamic_change: true } %>
         <div class="input-group-button">
-          <button type="submit" class="button button--muted">
-            <%= icon "magnifying-glass", aria_label: t("consultations.filters.search", scope: "decidim") %>
+          <button type="submit" class="button" aria-controls="consultations">
+            <%= icon "magnifying-glass", aria_label: t("consultations.filters.search", scope: "decidim"), role: "img" %>
           </button>
         </div>
       </div>
@@ -19,7 +23,8 @@
                                     options_for_state_filter,
                                     :first,
                                     :last,
-                                    legend_title: t("consultations.filters.state", scope: "decidim") %>
+                                    { legend_title: t("consultations.filters.state", scope: "decidim") },
+                                    "aria-controls": "consultations" %>
 
   <%= hidden_field_tag :order, order, id: nil, class: "order_filter" %>
 <% end %>

data/app/views/decidim/consultations/consultations/_question.html.erb

@@ -1,4 +1,4 @@
-<article class="card consultations-card">
+<div class="card consultations-card">
   <div class="row collapse card--process__row">
     <div class="columns mediumlarge-8 large-9 card--process__column">
       <div class="card__content">
@@ -35,4 +35,4 @@
       </div>
     </div>
   </div>
-</article>
+</div>

data/app/views/decidim/consultations/consultations/index.html.erb

@@ -11,9 +11,9 @@ edit_link(
   <div class="row columns">
     <div class="title-action">
       <div class="row collapse">
-        <h2 id="consultations-count" class="title-action__title section-heading">
+        <h1 id="consultations-count" class="title-action__title section-heading">
           <%= render partial: "count", locals: { count: consultations.count } %>
-        </h2>
+        </h1>
       </div>
     </div>
   </div>
@@ -32,4 +32,4 @@ edit_link(
 <% end %>
 
 <%= javascript_include_tag "decidim/filters" %>
-<%= javascript_include_tag "decidim/orders" %>
+<%= javascript_include_tag "decidim/results_listing" %>

data/app/views/decidim/consultations/question_multiple_votes/_form.html.erb

@@ -1,14 +1,18 @@
-<div class="row column">
-  <div class="large-offset-4 large-4 medium-offset-3 medium-6">
-    <div class="card">
-      <div class="card__content multiple_votes_form">
-        <% question.responses.each do |response| %>
-          <div>
-            <%= check_box_tag "responses[]", response.id, false, id: "vote_id_#{response.id}" %>
-            <%= form.label :id, translated_attribute(response.title), for: "vote_id_#{response.id}" %>
-          </div>
-        <% end %>
+<div class="card">
+  <% if group %>
+    <div class="card__top">
+      <div class="card__content">
+        <h5 class="card__title"><%= translated_attribute group.title %></h5>
       </div>
     </div>
+  <% end %>
+
+  <div class="card__content multiple_votes_form">
+    <% responses.each do |response| %>
+      <div>
+        <%= check_box_tag "responses[]", response.id, false, id: "vote_id_#{response.id}" %>
+        <%= form.label :id, translated_attribute(response.title), for: "vote_id_#{response.id}" %>
+      </div>
+    <% end %>
   </div>
 </div>

data/app/views/decidim/consultations/question_multiple_votes/show.html.erb

@@ -3,9 +3,15 @@
     method: :post,
     html: { class: "form" }) do |f| %>
 
-  <%= render partial: "form", locals: { form: f, question: current_participatory_space } %>
+  <div class="section row<%= " question-masonry-group" if current_participatory_space.grouped? %>">
+    <% current_participatory_space.grouped_responses.each do |group, responses| %>
+      <div class="column large-4 medium-6<%= " large-offset-4 medium-offset-3" unless current_participatory_space.grouped? %>">
+        <%= render partial: "form", locals: { form: f, group: group, responses: responses } %>
+      </div>
+    <% end %>
+  </div>
 
-  <div class="section question-vote-cabin">
+  <div class="question-vote-cabin">
 
     <div class="row column">
       <div class="large-offset-4 large-4 medium-offset-3 medium-6">

data/app/views/decidim/consultations/questions/_vote_button.html.erb

@@ -8,88 +8,91 @@
   </div>
 </div>
 
-<div class="row column question-vote-cabin">
-  <div class="columns mediumlarge-4 show-for-mediumlarge previous-question">
-    <%= display_next_previous_button(:previous, "float-left") %>
-  </div>
-  <div class="columns mediumlarge-4">
-    <% if question.consultation.upcoming? %>
-      <div class="card__button button expanded disabled">
-        <div class="vote-button-caption"><%= t("questions.vote_button.vote", scope: "decidim") %></div>
-        <div>
-          <%= t "questions.vote_button.starting_from",
-                scope: "decidim",
-                date: l(question.start_voting_date) %>
-        </div>
-      </div>
-    <% elsif question.consultation.finished? && signed_in? && question.voted_by?(current_user) %>
-      <%= button_to decidim_consultations.question_question_votes_path(question),
-                  method: :delete,
-                  remote: true,
-                  data: { disable: true },
-                  class: "card__button button expanded success disabled",
-                  id: "unvote_button" do %>
-        <div class="vote-button-caption">
-          <%= t("questions.vote_button.already_voted", scope: "decidim") %>
+<%# Check if controller has questions_helper %>
+<% if defined? display_next_previous_button %>
+  <div class="row column question-vote-cabin">
+    <div class="columns mediumlarge-4 show-for-mediumlarge previous-question">
+      <%= display_next_previous_button(:previous, "float-left") %>
+    </div>
+    <div class="columns mediumlarge-4">
+      <% if question.consultation.upcoming? %>
+        <div class="card__button button expanded disabled">
+          <div class="vote-button-caption"><%= t("questions.vote_button.vote", scope: "decidim") %></div>
+          <div>
+            <%= t "questions.vote_button.starting_from",
+                  scope: "decidim",
+                  date: l(question.start_voting_date) %>
+          </div>
         </div>
-      <% end %>
-    <% elsif signed_in? && question.consultation.active? %>
-      <% if allowed_to? :unvote, :question, question: question %>
+      <% elsif question.consultation.finished? && signed_in? && question.voted_by?(current_user) %>
         <%= button_to decidim_consultations.question_question_votes_path(question),
-                      method: :delete,
-                      remote: true,
-                      data: { disable: true },
-                      class: "card__button button expanded success",
-                      id: "unvote_button" do %>
-          <div class="vote-button-caption"
-               data-original="<%= t("questions.vote_button.already_voted", scope: "decidim") %>"
-               data-replace="<%= t("questions.vote_button.already_voted_hover", scope: "decidim") %>">
+                    method: :delete,
+                    remote: true,
+                    data: { disable: true },
+                    class: "card__button button expanded success disabled",
+                    id: "unvote_button" do %>
+          <div class="vote-button-caption">
             <%= t("questions.vote_button.already_voted", scope: "decidim") %>
           </div>
         <% end %>
-      <% elsif allowed_to? :vote, :question, question: question %>
-        <% if question.multiple? %>
-          <%= link_to decidim_consultations.question_question_multiple_votes_path(question),
-                      class: "card__button button expanded",
-                      id: "multivote_button" do %>
-            <div class="vote-button-caption"><%= t "questions.vote_button.vote", scope: "decidim" %></div>
+      <% elsif signed_in? && question.consultation.active? %>
+        <% if allowed_to? :unvote, :question, question: question %>
+          <%= button_to decidim_consultations.question_question_votes_path(question),
+                        method: :delete,
+                        remote: true,
+                        data: { disable: true },
+                        class: "card__button button expanded success",
+                        id: "unvote_button" do %>
+            <div class="vote-button-caption"
+                 data-original="<%= t("questions.vote_button.already_voted", scope: "decidim") %>"
+                 data-replace="<%= t("questions.vote_button.already_voted_hover", scope: "decidim") %>">
+              <%= t("questions.vote_button.already_voted", scope: "decidim") %>
+            </div>
+          <% end %>
+        <% elsif allowed_to? :vote, :question, question: question %>
+          <% if question.multiple? %>
+            <%= link_to decidim_consultations.question_question_multiple_votes_path(question),
+                        class: "card__button button expanded",
+                        id: "multivote_button" do %>
+              <div class="vote-button-caption"><%= t "questions.vote_button.vote", scope: "decidim" %></div>
+            <% end %>
+          <% else %>
+            <%= link_to "#", class: "card__button button expanded", id: "vote_button" do %>
+              <div class="vote-button-caption"><%= t "questions.vote_button.vote", scope: "decidim" %></div>
+            <% end %>
           <% end %>
         <% else %>
-          <%= link_to "#", class: "card__button button expanded", id: "vote_button" do %>
-            <div class="vote-button-caption"><%= t "questions.vote_button.vote", scope: "decidim" %></div>
+          <%= authorized_vote_modal_button(question, remote: true, class: "button expanded light button--sc") do %>
+            <%= t("questions.vote_button.verification_required", scope: "decidim") %>
           <% end %>
         <% end %>
-      <% else %>
-        <%= authorized_vote_modal_button(question, remote: true, class: "button expanded light button--sc") do %>
-          <%= t("questions.vote_button.verification_required", scope: "decidim") %>
+      <% elsif question.consultation.active? %>
+        <%= logged_button_to decidim_consultations.question_question_votes_path(question),
+                             class: "card__button button expanded",
+                             data: { disable: true },
+                             id: "vote_button" do %>
+          <div class="vote-button-caption">
+            <%= t("questions.vote_button.vote", scope: "decidim") %>
+          </div>
         <% end %>
       <% end %>
-    <% elsif question.consultation.active? %>
-      <%= logged_button_to decidim_consultations.question_question_votes_path(question),
-                           class: "card__button button expanded",
-                           data: { disable: true },
-                           id: "vote_button" do %>
-        <div class="vote-button-caption">
-          <%= t("questions.vote_button.vote", scope: "decidim") %>
-        </div>
-      <% end %>
-    <% end %>
-  </div>
-  <div class="columns mediumlarge-4 show-for-mediumlarge next-question">
-    <%= display_next_previous_button(:next, "float-right") %>
+    </div>
+    <div class="columns mediumlarge-4 show-for-mediumlarge next-question">
+      <%= display_next_previous_button(:next, "float-right") %>
+    </div>
   </div>
-</div>
 
-<div class="row column section hide-for-mediumlarge">
-  <div class="small-buttons">
-    <div class="columns small-6 previous-question">
-      <%= display_next_previous_button(:previous, "expanded") %>
-    </div>
-    <div class="columns small-6 next-question">
-      <%= display_next_previous_button(:next, "expanded") %>
+  <div class="row column section hide-for-mediumlarge">
+    <div class="small-buttons">
+      <div class="columns small-6 previous-question">
+        <%= display_next_previous_button(:previous, "expanded") %>
+      </div>
+      <div class="columns small-6 next-question">
+        <%= display_next_previous_button(:next, "expanded") %>
+      </div>
     </div>
   </div>
-</div>
+<% end %>
 
 <%= render partial: "decidim/consultations/questions/vote_modal", locals: { question: question } %>
 <%= render partial: "decidim/consultations/questions/vote_modal_confirm", locals: { question: question } %>