decidim-forms 0.23.3 → 0.24.0.rc1

data/lib/decidim/forms/api.rb

@@ -3,5 +3,8 @@
 module Decidim
   module Forms
     autoload :QuestionnaireEntityInterface, "decidim/api/questionnaire_entity_interface"
+    autoload :AnswerOptionType, "decidim/api/answer_option_type"
+    autoload :QuestionType, "decidim/api/question_type"
+    autoload :QuestionnaireType, "decidim/api/questionnaire_type"
   end
 end

data/lib/decidim/forms/data_portability_user_answers_serializer.rb

@@ -30,7 +30,13 @@ module Decidim
       private
 
       def normalize_body(resource)
-        resource.body || resource.choices.pluck(:body)
+        attachments_for(resource) || resource.body || resource.choices.pluck(:body)
+      end
+
+      def attachments_for(resource)
+        return if resource.attachments.blank?
+
+        resource.attachments.map(&:url)
       end
     end
   end

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

@@ -17,10 +17,13 @@ FactoryBot.define do
 
     trait :with_questions do
       questions do
+        position = 0
         qs = %w(short_answer long_answer).collect do |text_question_type|
-          build(:questionnaire_question, question_type: text_question_type)
+          q = build(:questionnaire_question, question_type: text_question_type, position: position)
+          position += 1
+          q
         end
-        qs << build(:questionnaire_question, :with_answer_options, question_type: :single_option)
+        qs << build(:questionnaire_question, :with_answer_options, question_type: :single_option, position: position)
         qs
       end
     end
@@ -91,6 +94,13 @@ FactoryBot.define do
     question { create(:questionnaire_question, questionnaire: questionnaire) }
     user { create(:user, organization: questionnaire.questionnaire_for.organization) }
     session_token { Digest::MD5.hexdigest(user.id.to_s) }
+
+    trait :with_attachments do
+      after(:create) do |answer, _evaluator|
+        create :attachment, :with_image, attached_to: answer
+        create :attachment, :with_pdf, attached_to: answer
+      end
+    end
   end
 
   factory :answer_option, class: "Decidim::Forms::AnswerOption" do

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

@@ -194,6 +194,28 @@ shared_examples_for "has questionnaire" do
       end
     end
 
+    shared_examples_for "question has a character limit" do
+      context "when max_characters value is positive" do
+        let(:max_characters) { 30 }
+
+        it "shows a message indicating number of characters left" do
+          visit questionnaire_public_path
+
+          expect(page).to have_content("30 characters left")
+        end
+      end
+
+      context "when max_characters value is 0" do
+        let(:max_characters) { 0 }
+
+        it "doesn't show message indicating number of characters left" do
+          visit questionnaire_public_path
+
+          expect(page).not_to have_content("characters left")
+        end
+      end
+    end
+
     describe "leaving a blank question (without js)", driver: :rack_test do
       include_context "when a non multiple choice question is mandatory"
 
@@ -268,12 +290,13 @@ shared_examples_for "has questionnaire" do
 
     describe "free text options" do
       let(:answer_option_bodies) { Array.new(3) { Decidim::Faker::Localized.sentence } }
-
+      let(:max_characters) { 0 }
       let!(:question) do
         create(
           :questionnaire_question,
           questionnaire: questionnaire,
           question_type: question_type,
+          max_characters: max_characters,
           position: 1,
           options: [
             { "body" => answer_option_bodies[0] },
@@ -346,6 +369,8 @@ shared_examples_for "has questionnaire" do
 
           expect(page).to have_field("questionnaire_responses_0_choices_2_custom_body", with: "Cacatua")
         end
+
+        it_behaves_like "question has a character limit"
       end
 
       context "when question is multiple_option type" do
@@ -392,27 +417,35 @@ shared_examples_for "has questionnaire" do
 
           expect(page).to have_field("questionnaire_responses_0_choices_2_custom_body", with: "Cacatua")
         end
+
+        it_behaves_like "question has a character limit"
       end
     end
 
     context "when question type is long answer" do
-      let!(:question) { create(:questionnaire_question, questionnaire: questionnaire, question_type: "long_answer") }
+      let(:max_characters) { 0 }
+      let!(:question) { create(:questionnaire_question, questionnaire: questionnaire, question_type: "long_answer", max_characters: max_characters) }
 
       it "renders the answer as a textarea" do
         visit questionnaire_public_path
 
         expect(page).to have_selector("textarea#questionnaire_responses_0")
       end
+
+      it_behaves_like "question has a character limit"
     end
 
     context "when question type is short answer" do
-      let!(:question) { create(:questionnaire_question, questionnaire: questionnaire, question_type: "short_answer") }
+      let(:max_characters) { 0 }
+      let!(:question) { create(:questionnaire_question, questionnaire: questionnaire, question_type: "short_answer", max_characters: max_characters) }
 
       it "renders the answer as a text field" do
         visit questionnaire_public_path
 
         expect(page).to have_selector("input[type=text]#questionnaire_responses_0")
       end
+
+      it_behaves_like "question has a character limit"
     end
 
     context "when question type is single option" do
@@ -665,7 +698,7 @@ shared_examples_for "has questionnaire" do
         end
 
         radio_buttons = page.all(".radio-button-collection input[type=radio]")
-        expect(radio_buttons.map { |b| b[:checked] }).to eq([nil, "true", nil, nil])
+        expect(radio_buttons.pluck(:checked)).to eq([nil, "true", nil, nil])
       end
 
       context "when the question is mandatory and the answer is not complete" do
@@ -830,7 +863,7 @@ shared_examples_for "has questionnaire" do
           end
 
           checkboxes = page.all(".check-box-collection input[type=checkbox]")
-          expect(checkboxes.map { |c| c[:checked] }).to eq(["true", "true", "true", nil, nil, "true"])
+          expect(checkboxes.pluck(:checked)).to eq(["true", "true", "true", nil, nil, "true"])
         end
       end
     end

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

@@ -2,10 +2,10 @@
 
 require "spec_helper"
 
-require "decidim/forms/test/shared_examples/manage_questionnaires/add_questions.rb"
-require "decidim/forms/test/shared_examples/manage_questionnaires/update_questions.rb"
-require "decidim/forms/test/shared_examples/manage_questionnaires/add_display_conditions.rb"
-require "decidim/forms/test/shared_examples/manage_questionnaires/update_display_conditions.rb"
+require "decidim/forms/test/shared_examples/manage_questionnaires/add_questions"
+require "decidim/forms/test/shared_examples/manage_questionnaires/update_questions"
+require "decidim/forms/test/shared_examples/manage_questionnaires/add_display_conditions"
+require "decidim/forms/test/shared_examples/manage_questionnaires/update_display_conditions"
 
 shared_examples_for "manage questionnaires" do
   let(:body) do
@@ -63,6 +63,7 @@ shared_examples_for "manage questionnaires" do
       expect(page).to have_selector("input[value='This is the first question'][disabled]")
       expect(page).to have_selector("select[id$=question_type][disabled]")
       expect(page).to have_selector("select[id$=max_choices][disabled]")
+      expect(page).to have_selector("input[id$=max_characters][disabled]")
       expect(page).to have_selector(".ql-editor[contenteditable=false]")
     end
   end

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

@@ -39,18 +39,21 @@ shared_examples_for "add display conditions" do
     context "when questionnaire has more than one question" do
       let!(:question_short_answer) do
         create(:questionnaire_question,
+               position: 0,
                questionnaire: questionnaire,
                body: Decidim::Faker::Localized.sentence,
                question_type: "short_answer")
       end
       let!(:question_long_answer) do
         create(:questionnaire_question,
+               position: 1,
                questionnaire: questionnaire,
                body: Decidim::Faker::Localized.sentence,
                question_type: "long_answer")
       end
       let!(:question_single_option) do
         create(:questionnaire_question,
+               position: 2,
                questionnaire: questionnaire,
                body: Decidim::Faker::Localized.sentence,
                question_type: "single_option",
@@ -58,6 +61,7 @@ shared_examples_for "add display conditions" do
       end
       let!(:question_multiple_option) do
         create(:questionnaire_question,
+               position: 3,
                questionnaire: questionnaire,
                body: Decidim::Faker::Localized.sentence,
                question_type: "multiple_option",
@@ -101,7 +105,7 @@ shared_examples_for "add display conditions" do
 
             within "select[id$=decidim_condition_question_id]" do
               elements = page.all("option[data-type]")
-              expect(elements.map { |element| element[:"data-type"] }).to eq(questions.map(&:question_type))
+              expect(elements.map { |element| element[:"data-type"] }).to match_array(questions.map(&:question_type))
               expect(page.find("option[value='#{questions.last.id}']")).to be_disabled
             end
           end
@@ -117,7 +121,7 @@ shared_examples_for "add display conditions" do
               option_elements = page.all("select[id$=condition_type] option")
               option_elements = option_elements.to_a.reject { |option| option[:style].match? "display: none" }
 
-              expect(option_elements.map(&:text)).to eq(options)
+              expect(option_elements.map(&:text)).to match_array(options)
             end
           end
         end
@@ -132,7 +136,7 @@ shared_examples_for "add display conditions" do
               option_elements = page.all("select[id$=condition_type] option")
               option_elements = option_elements.to_a.reject { |option| option[:style].match? "display: none" }
 
-              expect(option_elements.map(&:text)).to eq(options)
+              expect(option_elements.map(&:text)).to match_array(options)
             end
           end
         end

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

@@ -41,7 +41,7 @@ shared_examples_for "add questions" do
       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: "<b>Superkalifragilistic description</b>"
+        fill_in_editor find_nested_form_field_locator("description_en", visible: false), with: "<strong>Superkalifragilistic description</strong>"
       end
 
       click_button "Save"

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

@@ -15,6 +15,7 @@ shared_examples_for "update questions" do
       within "form.edit_questionnaire" do
         within ".questionnaire-question" do
           fill_in "questionnaire_questions_#{question.id}_body_en", with: "Modified question"
+          fill_in "questionnaire_questions_#{question.id}_max_characters", with: 30
           check "Mandatory"
           select "Long answer", from: "Type"
         end
@@ -29,6 +30,7 @@ shared_examples_for "update questions" do
       expect(page).to have_selector("input[value='Modified question']")
       expect(page).to have_no_selector("input[value='This is the first question']")
       expect(page).to have_selector("input#questionnaire_questions_#{question.id}_mandatory[checked]")
+      expect(page).to have_selector("input#questionnaire_questions_#{question.id}_max_characters[value='30']")
       expect(page).to have_selector("select#questionnaire_questions_#{question.id}_question_type option[value='long_answer'][selected]")
     end
 
@@ -39,6 +41,7 @@ shared_examples_for "update questions" do
         within ".questionnaire-question" do
           expect(page).to have_content("Statement*")
           fill_in "questionnaire_questions_#{question.id}_body_en", with: ""
+          fill_in "questionnaire_questions_#{question.id}_max_characters", with: -3
           check "Mandatory"
           select "Matrix (Multiple option)", from: "Type"
           select "2", from: "Maximum number of choices"
@@ -51,10 +54,12 @@ shared_examples_for "update questions" do
 
       expect(page).to have_admin_callout("There was a problem saving")
       expect(page).to have_content("can't be blank", count: 5) # emtpy question, 2 empty default answer options, 2 empty default matrix rows
+      expect(page).to have_content("must be greater than or equal to 0", count: 1)
 
       expect(page).to have_selector("input[value='']")
       expect(page).to have_no_selector("input[value='This is the first question']")
       expect(page).to have_selector("input#questionnaire_questions_#{question.id}_mandatory[checked]")
+      expect(page).to have_selector("input#questionnaire_questions_#{question.id}_max_characters[value='-3']")
       expect(page).to have_select("Maximum number of choices", selected: "2")
       expect(page).to have_selector("select#questionnaire_questions_#{question.id}_question_type option[value='matrix_multiple'][selected]")
     end

data/lib/decidim/forms/user_answers_serializer.rb

@@ -28,7 +28,15 @@ module Decidim
       private
 
       def normalize_body(answer)
-        answer.body || normalize_choices(answer, answer.choices)
+        answer.body ||
+          normalize_attachments(answer) ||
+          normalize_choices(answer, answer.choices)
+      end
+
+      def normalize_attachments(answer)
+        return if answer.attachments.blank?
+
+        answer.attachments.map(&:url)
       end
 
       def normalize_choices(answer, choices)

data/lib/decidim/forms/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: decidim-forms
 version: !ruby/object:Gem::Version
-  version: 0.23.3
+  version: 0.24.0.rc1
 platform: ruby
 authors:
 - Josep Jaume Rey Peroy
@@ -11,7 +11,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-23 00:00:00.000000000 Z
+date: 2021-03-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: decidim-core
@@ -19,14 +19,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.23.3
+        version: 0.24.0.rc1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.23.3
+        version: 0.24.0.rc1
 - !ruby/object:Gem::Dependency
   name: wicked_pdf
   requirement: !ruby/object:Gem::Requirement
@@ -61,28 +61,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.23.3
+        version: 0.24.0.rc1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.23.3
+        version: 0.24.0.rc1
 - !ruby/object:Gem::Dependency
   name: decidim-dev
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.23.3
+        version: 0.24.0.rc1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.23.3
+        version: 0.24.0.rc1
 description: A forms gem for decidim.
 email:
 - josepjaume@gmail.com
@@ -156,9 +156,6 @@ files:
 - app/queries/decidim/forms/questionnaire_participant.rb
 - app/queries/decidim/forms/questionnaire_participants.rb
 - app/queries/decidim/forms/questionnaire_user_answers.rb
-- app/types/decidim/forms/answer_option_type.rb
-- app/types/decidim/forms/question_type.rb
-- app/types/decidim/forms/questionnaire_type.rb
 - app/views/decidim/forms/admin/questionnaires/_answer_option.html.erb
 - app/views/decidim/forms/admin/questionnaires/_answer_option_template.html.erb
 - app/views/decidim/forms/admin/questionnaires/_display_condition.html.erb
@@ -174,6 +171,7 @@ files:
 - app/views/decidim/forms/admin/questionnaires/answers/show.html.erb
 - app/views/decidim/forms/admin/questionnaires/edit.html.erb
 - app/views/decidim/forms/questionnaires/_answer.html.erb
+- app/views/decidim/forms/questionnaires/answers/_files.html.erb
 - app/views/decidim/forms/questionnaires/answers/_long_answer.html.erb
 - app/views/decidim/forms/questionnaires/answers/_matrix_multiple.html.erb
 - app/views/decidim/forms/questionnaires/answers/_matrix_single.html.erb
@@ -264,7 +262,11 @@ files:
 - db/migrate/20200225123810_create_decidim_forms_question_matrix_rows.rb
 - db/migrate/20200304152939_add_matrix_row_id_to_decidim_forms_answer_choices.rb
 - db/migrate/20201110152921_add_salt_to_decidim_forms_questionnaires.rb
+- db/migrate/20210208094442_add_max_characters_to_decidim_forms_questions.rb
+- lib/decidim/api/answer_option_type.rb
+- lib/decidim/api/question_type.rb
 - lib/decidim/api/questionnaire_entity_interface.rb
+- lib/decidim/api/questionnaire_type.rb
 - lib/decidim/exporters/form_pdf.rb
 - lib/decidim/exporters/form_pdf_controller_helper.rb
 - lib/decidim/forms.rb
@@ -296,14 +298,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.6'
+      version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.2
 signing_key:
 specification_version: 4
 summary: Decidim forms

data/app/types/decidim/forms/answer_option_type.rb

@@ -1,14 +0,0 @@
-# frozen_string_literal: true
-
-module Decidim
-  module Forms
-    AnswerOptionType = GraphQL::ObjectType.define do
-      name "AnswerOption"
-      description "An answer option for a multi-choice question in a questionnaire"
-
-      field :id, !types.ID, "ID of this answer option"
-      field :body, !Decidim::Core::TranslatedFieldType, "The text answer response option."
-      field :freeText, !types.Boolean, "Whether if this answer accepts any free text from the user.", property: :free_text
-    end
-  end
-end