decidim-forms 0.25.0.rc4 → 0.26.0.rc1

data/config/locales/ro-RO.yml

@@ -4,13 +4,13 @@ ro:
       answer:
         body: Răspuns
         choices: Opțiuni
-        selected_choices: Alegeri selectate
+        selected_choices: Opțiuni selectate
       question:
         max_choices: Numărul maxim de opțiuni
         question_type: Tip
       questionnaire_question:
-        mandatory: Mandatory
-        max_characters: Limita de caractere (lăsați la 0 dacă nu este limită)
+        mandatory: Obligatoriu
+        max_characters: Limita de caractere (lăsați la 0 dacă nu este nicio limită)
     errors:
       models:
         answer:
@@ -20,23 +20,23 @@ ro:
             body:
               too_long: este prea lung
             choices:
-              missing: nu sunt complete
+              missing: nu sunt completate
               too_many: sunt prea multe
         questionnaire:
-          request_invalid: A apărut o eroare la procesarea cererii. Vă rugăm să încercați din nou
+          request_invalid: A apărut o eroare la procesarea cererii. Te rugăm să încerci din nou
   decidim:
     forms:
       admin:
         models:
           components:
             description: Descriere
-            tos: Termenii serviciului
+            tos: Condiţii de utilizare
         questionnaires:
           actions:
             back: Înapoi la întrebări
-            show: Arată răspunsurile
+            show: Afișează răspunsurile
           answer_option:
-            answer_option: Opțiune răspuns
+            answer_option: Opțiune de răspuns
             free_text: Text liber
             remove: Elimină
             statement: Declarație
@@ -44,76 +44,76 @@ ro:
             actions:
               back: Înapoi la răspunsuri
               export: Exportă
-              show: Arată răspunsurile
-            empty: Încă nu există răspunsuri
+              show: Afişează răspunsurile
+            empty: Niciun răspuns încă
             export:
               answer:
                 title: 'Răspuns #%{number}'
             export_response:
-              title: chestionar_utilizator_răspunsuri_%{token}
+              title: survey_user_answers_%{token}
             index:
               title: "%{total} total răspunsuri"
             show:
               title: 'Răspuns #%{number}'
           display_condition:
-            answer_option: Opțiune răspuns
+            answer_option: Opțiune de răspuns
             condition_question: Întrebare
             condition_type: Condiție
             condition_types:
-              answered: Răspuns
-              equal: Egal
+              answered: Răspunse
+              equal: Egal cu
               match: Include textul
-              not_answered: Nu s-a răspuns
-              not_equal: Nu este egal
+              not_answered: Fără răspuns
+              not_equal: Nu este egal cu
             condition_value: Text inclus
-            display_condition: Condiție de afișare
-            mandatory: Această condiție trebuie îndeplinită întotdeauna indiferent de statutul altor condiții
+            display_condition: Afișează condiția
+            mandatory: Această condiție trebuie îndeplinită întotdeauna indiferent de stadiul altor condiții
             remove: Elimină
-            save_warning: Amintiţi-vă să salvaţi formularul înainte de configurarea condiţiilor de afişare
-            select_answer_option: Selectați opțiunea de răspuns
+            save_warning: Ține minte să salvezi chestionarul înainte de configurarea condiţiilor de afişare
+            select_answer_option: Selectează opțiunea de răspuns
             select_condition_question: Selectează o întrebare
-            select_condition_type: Selectați un tip de condiție
+            select_condition_type: Selectează un tip de condiție
           edit:
             save: Salvează
-            title: Editare chestionar
+            title: Editează chestionarul
           form:
             add_question: Adaugă întrebare
             add_separator: Adaugă separator
-            already_answered_warning: Formularul a primit deja un răspuns de la unii utilizatori astfel încât nu îi puteți modifica întrebările.
+            already_answered_warning: Au fost înregistrare deja câteva răspunsuri de la utilizatori pentru acest chestionar. De aceea, nu mai poți modifica întrebările.
             collapse: Restrânge toate întrebările
             expand: Extinde toate întrebările
-            preview: Previzualizare
-            title: Editare formular pentru %{questionnaire_for}
-            unpublished_warning: Formularul nu este publicat. Puteți modifica întrebările, dar acest lucru va șterge răspunsurile curente.
+            preview: Previzualizează
+            title: Editează chestionarul pentru %{questionnaire_for}
+            unpublished_warning: Chestionarul nu este publicat. Poți modifica întrebările, dar această acțiune va duce la stergerea automată a răspunsurilor curente.
           matrix_row:
             matrix_row: Rând
             remove: Elimină
             statement: Declarație
           question:
-            add_answer_option: Adaugă opțiune răspuns
-            add_display_condition: Adăugați o condiție de afișare
-            add_display_condition_info: Salvați formularul pentru a configura condițiile de afișare
+            add_answer_option: Adaugă opțiune de răspuns
+            add_display_condition: Adaugă condiție de afișare
+            add_display_condition_info: Salvează formularul pentru a configura condițiile de afișare
             add_matrix_row: Adaugă rând
             any: Oricare
             collapse: Restrânge
             description: Descriere
-            down: Jos
+            down: Mergi în jos
             expand: Extinde
             question: Întrebare
             remove: Elimină
             statement: Declarație
-            up: Sus
+            up: Mergi în sus
           separator:
-            down: Jos
+            down: Mergi în jos
             remove: Elimină
             separator: Separator
-            up: Sus
+            up: Mergi în sus
           update:
-            invalid: A apărut o eroare la salvarea formularului.
-            success: Formular salvat cu succes.
+            invalid: A apărut o eroare la salvarea chestionarului.
+            success: Chestionarul a fost salvat cu succes.
       errors:
         answer:
-          body: Corpul nu poate fi gol
+          body: Conținutul nu poate fi gol
       files:
         extension_allowlist: 'Formate acceptate:'
       images:
@@ -121,58 +121,58 @@ ro:
         dimensions_info: 'Această imagine va fi:'
         processors:
           resize_and_pad: Redimensionat și padded la
-          resize_to_fit: Redimensionat pentru a se potrivi
+          resize_to_fit: Redimensionată pentru a se potrivi
       question_types:
         files: Fișiere
         long_answer: Răspuns lung
-        matrix_multiple: Matricea (opțiune multiplă)
-        matrix_single: Matricea (opțiunea unică)
-        multiple_option: Opțiune multiplă
+        matrix_multiple: Matrice (mai multe opțiuni)
+        matrix_single: Matrice (opțiune unică)
+        multiple_option: Mai multe opțiuni
         short_answer: Răspuns scurt
         single_option: Opțiune unică
         sorting: Sortare
       questionnaires:
         answer:
-          invalid: A apărut o problemă cu răspunsul la formular.
+          invalid: A apărut o problemă la completarea chestionarului.
           max_choices_alert: Există prea multe alegeri selectate
-          success: Formularul a răspuns cu succes.
+          success: Chestionarul a fost completat cu succes.
         question:
-          max_choices: 'Max alegeri: %{n}'
+          max_choices: 'Număr maxim de alegeri: %{n}'
         show:
           answer_questionnaire:
-            anonymous_user_message: <a href="%{sign_in_link}">Autentifică-te cu contul tău</a> sau <a href="%{sign_up_link}">înregistrează-te</a> pentru a răspunde la formular.
-            title: Răspundeți la formular
+            anonymous_user_message: <a href="%{sign_in_link}">Autentifică-te cu contul tău</a> sau <a href="%{sign_up_link}">înregistrează-te</a> pentru a răspunde la chestionar.
+            title: Completează chestionarul
           current_step: Pasul %{step}
           of_total_steps: din %{total_steps}
           questionnaire_answered:
-            body: Ați răspuns deja la acest formular.
-            title: Răspuns deja
+            body: Ai completat deja la acest chestionar.
+            title: Un răspuns a fost deja trimis
           questionnaire_closed:
-            body: Formularul este închis și nu se poate răspunde.
+            body: Chestionarul este închis și nu mai poate fi completat.
             title: Formular închis
           questionnaire_for_private_users:
-            body: Formularul este disponibil doar pentru utilizatorii privați
-            title: Formular închis
+            body: Chestionarul este disponibil doar pentru utilizatorii privați
+            title: Chestionar închis
           questionnaire_js_disabled:
-            body: Unele dintre funcțiile acestui formular vor fi dezactivate. Pentru a vă îmbunătăți experiența, vă rugăm să activați JavaScript în browser-ul dvs.
+            body: Unele dintre funcționalitățile acestui chestionar vor fi dezactivate. Pentru a îți îmbunătăți experiența, te rugăm să activezi JavaScript în browser-ul tău.
             title: JavaScript este dezactivat
           questionnaire_not_published:
-            body: Acest formular nu este publicat încă.
-          tos_agreement: Prin participare acceptați termenii și condițiile sale de utilizare
+            body: Acest chestionar nu este încă publicat.
+          tos_agreement: Prin participare accepți Termenii și condițiile sale de utilizare
       step_navigation:
         show:
-          are_you_sure: Această acțiune nu poate fi anulată și nu veți putea să vă editați răspunsurile. Sunteți sigur?
+          are_you_sure: Această acțiune nu poate fi anulată și nu vei putea să îți editezi răspunsurile. Sigur dorești asta?
           back: Înapoi
           continue: Continuă
-          submit: Trimiteți
+          submit: Trimite
       user_answers_serializer:
         body: Răspuns
         completion: Finalizare
-        created_at: Răspuns pornit
+        created_at: Răspuns completat la
         id: ID răspuns
-        ip_hash: Hash IP
+        ip_hash: IP Hash
         question: Întrebare
         registered: Înregistrat
         session_token: Identificator utilizator
         unregistered: Neînregistrat
-        user_status: Statusul utilizatorului
+        user_status: Stadiul utilizatorului

data/config/locales/val-ES.yml

@@ -0,0 +1,18 @@
+val:
+  decidim:
+    forms:
+      admin:
+        questionnaires:
+          form:
+            add_title_and_description: Afig títol i descripció
+          title_and_description:
+            collapse: Redueix
+            description: Descripció
+            down: Baixar
+            expand: Expandeix
+            remove: Elimina
+            title: Títol
+            title_and_description: Títol i descripció
+            up: Pujar
+      question_types:
+        title_and_description: Títol i descripció

data/lib/decidim/forms/engine.rb

@@ -11,6 +11,10 @@ module Decidim
       initializer "decidim_forms.add_cells_view_paths" do
         Cell::ViewModel.view_paths << File.expand_path("#{Decidim::Forms::Engine.root}/app/cells")
       end
+
+      initializer "decidim_forms.webpacker.assets_path" do
+        Decidim.register_assets_path File.expand_path("app/packs", root)
+      end
     end
   end
 end

data/lib/decidim/forms/test/factories.rb

@@ -123,6 +123,10 @@ FactoryBot.define do
     trait :separator do
       question_type { :separator }
     end
+
+    trait :title_and_description do
+      question_type { :title_and_description }
+    end
   end
 
   factory :answer, class: "Decidim::Forms::Answer" do
@@ -144,6 +148,14 @@ FactoryBot.define do
     question { create(:questionnaire_question) }
     body { generate_localized_title }
     free_text { false }
+
+    trait :free_text_enabled do
+      free_text { true }
+    end
+
+    trait :free_text_disabled do
+      free_text { false }
+    end
   end
 
   factory :answer_choice, class: "Decidim::Forms::AnswerChoice" do

data/lib/decidim/forms/test/shared_examples/manage_questionnaire_answers.rb

@@ -69,6 +69,24 @@ shared_examples_for "manage questionnaire answers" do
           expect(page).to have_content("User identifier")
         end
       end
+
+      context "when multiple answer choice" do
+        let(:first_type) { "multiple_option" }
+        let!(:answer1) { create :answer, questionnaire: questionnaire, question: first, body: nil }
+        let!(:answer_option) { create :answer_option, question: first }
+        let!(:answer_choice) { create :answer_choice, answer: answer1, answer_option: answer_option, body: translated(answer_option.body, locale: I18n.locale) }
+
+        before do
+          find_all("a.action-icon.action-icon--eye").first.click
+        end
+
+        it "shows the answers page with custom body" do
+          within "#answers" do
+            expect(page).to have_css("dt", text: translated(first.body))
+            expect(page).to have_css("li", text: translated(answer_option.body))
+          end
+        end
+      end
     end
 
     context "and managing individual answer page" do

data/lib/decidim/forms/test/shared_examples/manage_questionnaires/add_questions.rb

@@ -4,20 +4,21 @@ require "spec_helper"
 
 shared_examples_for "add questions" do
   it "adds a few questions and separators to the questionnaire" do
-    questions_body = ["This is the first question", "This is the second question"]
+    fields_body = ["This is the first question", "This is the second question", "This is the first title and description"]
 
     within "form.edit_questionnaire" do
       click_button "Add question"
       click_button "Add separator"
+      click_button "Add title and description"
       click_button "Add question"
 
-      expect(page).to have_selector(".questionnaire-question", count: 3)
+      expect(page).to have_selector(".questionnaire-question", count: 4)
 
       expand_all_questions
 
-      page.all(".questionnaire-question .collapsible").each_with_index do |question, idx|
-        within question do
-          fill_in find_nested_form_field_locator("body_en"), with: questions_body[idx]
+      page.all(".questionnaire-question .collapsible").each_with_index do |field, idx|
+        within field do
+          fill_in find_nested_form_field_locator("body_en"), with: fields_body[idx]
         end
       end
 
@@ -30,6 +31,7 @@ shared_examples_for "add questions" do
 
     expect(page).to have_selector("input[value='This is the first question']")
     expect(page).to have_selector("input[value='This is the second question']")
+    expect(page).to have_selector("input[value='This is the first title and description']")
     expect(page).to have_content("Separator #2")
   end
 
@@ -62,6 +64,35 @@ shared_examples_for "add questions" do
     expect(page).to have_selector("strong", text: "Superkalifragilistic description")
   end
 
+  it "adds a title-and-description" do
+    within "form.edit_questionnaire" do
+      click_button "Add title and description"
+      expand_all_questions
+
+      within ".questionnaire-question" do
+        fill_in find_nested_form_field_locator("body_en"), with: "Body"
+
+        fill_in_editor find_nested_form_field_locator("description_en", visible: false), with: "<strong>Superkalifragilistic description</strong>"
+      end
+
+      click_button "Save"
+    end
+
+    expect(page).to have_admin_callout("successfully")
+
+    component.update!(
+      step_settings: {
+        component.participatory_space.active_step.id => {
+          allow_answers: true
+        }
+      }
+    )
+
+    visit questionnaire_public_path
+
+    expect(page).to have_selector("strong", text: "Superkalifragilistic description")
+  end
+
   it "adds a question with answer options" do
     question_body = ["This is the first question", "This is the second question"]
     answer_options_body = [

data/lib/decidim/forms/test/shared_examples/manage_questionnaires/update_questions.rb

@@ -118,6 +118,104 @@ shared_examples_for "update questions" do
     end
   end
 
+  context "when a questionnaire has a title and description" do
+    let!(:question) { create(:questionnaire_question, :title_and_description, questionnaire: questionnaire, body: title_and_description_body) }
+
+    before do
+      visit questionnaire_edit_path
+      expand_all_questions
+    end
+
+    it "modifies the question when the information is valid" do
+      within "form.edit_questionnaire" do
+        within ".questionnaire-question" do
+          fill_in "questionnaire_questions_#{question.id}_body_en", with: "Modified title and description"
+        end
+
+        click_button "Save"
+      end
+
+      expect(page).to have_admin_callout("successfully")
+
+      visit_questionnaire_edit_path_and_expand_all
+
+      expect(page).to have_selector("input[value='Modified title and description']")
+      expect(page).to have_no_selector("input[value='This is the first title and description']")
+    end
+
+    it "re-renders the form when the information is invalid and displays errors" do
+      expand_all_questions
+
+      within "form.edit_questionnaire" do
+        within ".questionnaire-question" do
+          fill_in "questionnaire_questions_#{question.id}_body_en", with: ""
+        end
+
+        click_button "Save"
+      end
+
+      expand_all_questions
+
+      expect(page).to have_admin_callout("There was a problem saving")
+      expect(page).to have_content("can't be blank", count: 1)
+      expect(page).to have_selector("input[value='']")
+      expect(page).to have_no_selector("input[value='This is the first title and description']")
+    end
+
+    it "preserves deleted status across submission failures" do
+      within "form.edit_questionnaire" do
+        within ".questionnaire-question" do
+          click_button "Remove"
+        end
+      end
+
+      click_button "Add question"
+
+      click_button "Save"
+
+      expect(page).to have_selector(".questionnaire-question", count: 1)
+
+      within ".questionnaire-question" do
+        expect(page).to have_selector(".card-title", text: "#1")
+        expect(page).to have_no_button("Up")
+      end
+    end
+
+    it "removes the question" do
+      within "form.edit_questionnaire" do
+        within ".questionnaire-question" do
+          click_button "Remove"
+        end
+
+        click_button "Save"
+      end
+
+      expect(page).to have_admin_callout("successfully")
+
+      visit questionnaire_edit_path
+
+      within "form.edit_questionnaire" do
+        expect(page).to have_selector(".questionnaire-question", count: 0)
+      end
+    end
+
+    it "cannot be moved up" do
+      within "form.edit_questionnaire" do
+        within ".questionnaire-question" do
+          expect(page).to have_no_button("Up")
+        end
+      end
+    end
+
+    it "cannot be moved down" do
+      within "form.edit_questionnaire" do
+        within ".questionnaire-question" do
+          expect(page).to have_no_button("Down")
+        end
+      end
+    end
+  end
+
   context "when a questionnaire has an existing question with answer options" do
     let!(:question) do
       create(

data/lib/decidim/forms/test/shared_examples/manage_questionnaires.rb

@@ -16,6 +16,14 @@ shared_examples_for "manage questionnaires" do
     }
   end
 
+  let(:title_and_description_body) do
+    {
+      en: "Este es el primer separador de texto",
+      ca: "Aquest és el primer separador de text",
+      es: "Esta es la primera pregunta"
+    }
+  end
+
   it "updates the questionnaire" do
     visit questionnaire_edit_path
 

data/lib/decidim/forms/user_answers_serializer.rb

@@ -14,15 +14,14 @@ module Decidim
 
       # Public: Exports a hash with the serialized data for the user answers.
       def serialize
-        @answers.each_with_index.inject({}) do |serialized, (answer, idx)|
-          serialized.update(
-            answer_translated_attribute_name(:id) => answer.session_token,
-            answer_translated_attribute_name(:created_at) => answer.created_at.to_s(:db),
-            answer_translated_attribute_name(:ip_hash) => answer.ip_hash,
-            answer_translated_attribute_name(:user_status) => answer_translated_attribute_name(answer.decidim_user_id.present? ? "registered" : "unregistered"),
-            "#{idx + 1}. #{translated_attribute(answer.question.body)}" => normalize_body(answer)
-          )
+        answers_hash = hash_for(@answers.first)
+        answers_hash.merge!(questions_hash)
+
+        @answers.each do |answer|
+          answers_hash[translated_question_key(answer.question.position, answer.question.body)] = normalize_body(answer)
         end
+
+        answers_hash
       end
 
       private
@@ -30,6 +29,33 @@ module Decidim
       attr_reader :answers
       alias resource answers
 
+      def hash_for(answer)
+        {
+          answer_translated_attribute_name(:id) => answer&.session_token,
+          answer_translated_attribute_name(:created_at) => answer&.created_at&.to_s(:db),
+          answer_translated_attribute_name(:ip_hash) => answer&.ip_hash,
+          answer_translated_attribute_name(:user_status) => answer_translated_attribute_name(answer&.decidim_user_id.present? ? "registered" : "unregistered")
+        }
+      end
+
+      def questions_hash
+        return {} if questionnaire&.questions.blank?
+
+        questionnaire.questions.each.inject({}) do |serialized, question|
+          serialized.update(
+            translated_question_key(question.position, question.body) => ""
+          )
+        end
+      end
+
+      def questionnaire
+        @answers.first&.questionnaire
+      end
+
+      def translated_question_key(idx, body)
+        "#{idx + 1}. #{translated_attribute(body)}"
+      end
+
       def normalize_body(answer)
         answer.body ||
           normalize_attachments(answer) ||
@@ -47,7 +73,7 @@ module Decidim
           normalize_matrix_choices(answer, choices)
         else
           choices.map do |choice|
-            choice.try(:custom_body) || choice.try(:body)
+            format_free_text_for choice
           end
         end
       end
@@ -68,6 +94,12 @@ module Decidim
       def answer_translated_attribute_name(attribute)
         I18n.t(attribute.to_sym, scope: "decidim.forms.user_answers_serializer")
       end
+
+      def format_free_text_for(choice)
+        return choice.try(:body) if choice.try(:custom_body).blank?
+
+        "#{choice.try(:body)} (#{choice.try(:custom_body)})"
+      end
     end
   end
 end

data/lib/decidim/forms/version.rb

@@ -4,7 +4,7 @@ module Decidim
   # This holds the decidim-forms version.
   module Forms
     def self.version
-      "0.25.0.rc4"
+      "0.26.0.rc1"
     end
   end
 end