decidim-initiatives 0.26.2 → 0.27.0.rc2

data/app/forms/decidim/initiatives/admin/initiative_form.rb

@@ -19,7 +19,7 @@ module Decidim
         attribute :signature_start_date, Decidim::Attributes::LocalizedDate
         attribute :signature_end_date, Decidim::Attributes::LocalizedDate
         attribute :hashtag, String
-        attribute :offline_votes, Hash[Symbol => Integer]
+        attribute :offline_votes, Hash
         attribute :state, String
         attribute :attachment, AttachmentForm
 
@@ -47,7 +47,7 @@ module Decidim
         def signature_type_updatable?
           @signature_type_updatable ||= begin
             state ||= context.initiative.state
-            state == "validating" && context.current_user.admin? || state == "created"
+            (state == "validating" && context.current_user.admin?) || state == "created"
           end
         end
 
@@ -74,7 +74,7 @@ module Decidim
         # Private: set the in-person signatures to zero for every scope
         def zero_offine_votes_with_scopes_names(model)
           model.votable_initiative_type_scopes.each_with_object({}) do |initiative_scope_type, all_votes|
-            all_votes[initiative_scope_type.decidim_scopes_id&.to_s&.to_sym || "global"] = [0, initiative_scope_type.scope_name]
+            all_votes[initiative_scope_type.decidim_scopes_id || "global"] = [0, initiative_scope_type.scope_name]
           end
         end
 
@@ -83,7 +83,7 @@ module Decidim
           model.offline_votes.delete("total")
           model.offline_votes.each_with_object({}) do |(decidim_scope_id, votes), all_votes|
             scope_name = model.votable_initiative_type_scopes.find do |initiative_scope_type|
-              initiative_scope_type.global_scope? && decidim_scope_id == "global" ||
+              (initiative_scope_type.global_scope? && decidim_scope_id == "global") ||
                 initiative_scope_type.decidim_scopes_id == decidim_scope_id.to_i
             end.scope_name
 

data/app/forms/decidim/initiatives/admin/initiative_type_form.rb

@@ -13,11 +13,12 @@ module Decidim
 
         translatable_attribute :title, String
         translatable_attribute :description, String
-        attribute :banner_image, String
+        attribute :banner_image
         attribute :signature_type, String
         attribute :undo_online_signatures_enabled, Boolean
         attribute :attachments_enabled, Boolean
         attribute :custom_signature_end_date_enabled, Boolean
+        attribute :comments_enabled, Boolean
         attribute :area_enabled, Boolean
         attribute :child_scope_threshold_enabled, Boolean
         attribute :only_global_scope_enabled, Boolean

data/app/forms/decidim/initiatives/admin/initiatives_settings_form.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Initiatives
+    module Admin
+      # A form object used to create initiatives settings from the admin dashboard.
+      class InitiativesSettingsForm < Form
+        mimic :initiatives_settings
+
+        attribute :initiatives_order, String
+      end
+    end
+  end
+end

data/app/forms/decidim/initiatives/vote_form.rb

@@ -1,13 +1,10 @@
 # frozen_string_literal: true
 
-require "virtus/multiparams"
-
 module Decidim
   module Initiatives
     # A form object used to collect the data for a new initiative.
     class VoteForm < Form
       include TranslatableAttributes
-      include Virtus::Multiparams
 
       mimic :initiatives_vote
 

data/app/helpers/decidim/initiatives/create_initiative_helper.rb

@@ -27,7 +27,7 @@ module Decidim
           [
             I18n.t(
               "online",
-              scope: %w(activemodel attributes initiative signature_type_values)
+              scope: "activemodel.attributes.initiative.signature_type_values"
             ), "online"
           ]
         ]
@@ -38,7 +38,7 @@ module Decidim
           [
             I18n.t(
               "offline",
-              scope: %w(activemodel attributes initiative signature_type_values)
+              scope: "activemodel.attributes.initiative.signature_type_values"
             ), "offline"
           ]
         ]
@@ -49,7 +49,7 @@ module Decidim
           [
             I18n.t(
               type,
-              scope: %w(activemodel attributes initiative signature_type_values)
+              scope: "activemodel.attributes.initiative.signature_type_values"
             ), type
           ]
         end

data/app/helpers/decidim/initiatives/initiative_helper.rb

@@ -43,7 +43,9 @@ module Decidim
       def popularity_tag(initiative)
         content_tag(:div, class: "extra__popularity popularity #{popularity_class(initiative)}".strip) do
           5.times do
-            concat(content_tag(:span, class: "popularity__item") {})
+            concat(content_tag(:span, class: "popularity__item") do
+              # empty block
+            end)
           end
 
           concat(content_tag(:span, class: "popularity__desc") do
@@ -89,11 +91,11 @@ module Decidim
         tag = "button"
         html_options ||= {}
 
-        if !current_user
-          html_options["data-open"] = "loginModal"
-        else
+        if current_user
           html_options["data-open"] = "authorizationModal"
           html_options["data-open-url"] = authorization_sign_modal_initiative_path(initiative)
+        else
+          html_options["data-open"] = "loginModal"
         end
 
         html_options["onclick"] = "event.preventDefault();"

data/app/mailers/decidim/initiatives/initiatives_mailer.rb

@@ -7,8 +7,8 @@ module Decidim
       include Decidim::TranslatableAttributes
       include Decidim::SanitizeHelper
 
-      add_template_helper Decidim::TranslatableAttributes
-      add_template_helper Decidim::SanitizeHelper
+      helper Decidim::TranslatableAttributes
+      helper Decidim::SanitizeHelper
 
       # Notifies initiative creation
       def notify_creation(initiative)

data/app/models/decidim/initiative.rb

@@ -22,6 +22,8 @@ module Decidim
     include Decidim::Initiatives::HasArea
     include Decidim::TranslatableResource
     include Decidim::HasResourcePermission
+    include Decidim::HasArea
+    include Decidim::FilterableResource
 
     translatable_fields :title, :description, :answer
 
@@ -64,9 +66,8 @@ module Decidim
 
     validates :title, :description, :state, :signature_type, presence: true
     validates :hashtag,
-              uniqueness: true,
-              allow_blank: true,
-              case_sensitive: false
+              uniqueness: { allow_blank: true, case_sensitive: false }
+
     validate :signature_type_allowed
 
     scope :open, lambda {
@@ -80,6 +81,8 @@ module Decidim
     scope :published, -> { where.not(published_at: nil) }
     scope :with_state, ->(state) { where(state: state) if state.present? }
 
+    scope_search_multi :with_any_state, [:accepted, :rejected, :answered, :open, :closed]
+
     scope :currently_signable, lambda {
       where("signature_start_date <= ?", Date.current)
         .where("signature_end_date >= ?", Date.current)
@@ -106,6 +109,42 @@ module Decidim
     scope :future_spaces, -> { none }
     scope :past_spaces, -> { closed }
 
+    scope :with_any_type, lambda { |*original_type_ids|
+      type_ids = original_type_ids.flatten
+      return self if type_ids.include?("all")
+
+      types = InitiativesTypeScope.where(decidim_initiatives_types_id: type_ids).pluck(:id)
+      where(scoped_type: types)
+    }
+
+    # Redefine the with_any_scope method as the initiative scope is defined by
+    # the initiative type scope.
+    scope :with_any_scope, lambda { |*original_scope_ids|
+      scope_ids = original_scope_ids.flatten
+      return self if scope_ids.include?("all")
+
+      clean_scope_ids = scope_ids
+
+      conditions = []
+      conditions << "decidim_initiatives_type_scopes.decidim_scopes_id IS NULL" if clean_scope_ids.delete("global")
+      conditions.concat(["? = ANY(decidim_scopes.part_of)"] * clean_scope_ids.count) if clean_scope_ids.any?
+
+      joins(:scoped_type).references(:decidim_scopes).where(conditions.join(" OR "), *clean_scope_ids.map(&:to_i))
+    }
+
+    scope :authored_by, lambda { |author|
+      co_authoring_initiative_ids = Decidim::InitiativesCommitteeMember.where(
+        decidim_users_id: author
+      ).pluck(:decidim_initiatives_id)
+
+      where(
+        decidim_author_id: author,
+        decidim_author_type: Decidim::UserBaseEntity.name
+      ).or(
+        where(id: co_authoring_initiative_ids)
+      )
+    }
+
     before_update :set_offline_votes_total
     after_commit :notify_state_change
     after_create :notify_creation
@@ -124,6 +163,10 @@ module Decidim
       Decidim::Initiatives::AdminLog::InitiativePresenter
     end
 
+    def self.ransackable_scopes(_auth_object = nil)
+      [:with_any_state, :with_any_type, :with_any_scope, :with_any_area]
+    end
+
     delegate :document_number_authorization_handler, :promoting_committee_enabled?, to: :type
     delegate :type, :scope, :scope_name, to: :scoped_type, allow_nil: true
 
@@ -135,6 +178,11 @@ module Decidim
       type.attached_uploader(:banner_image)
     end
 
+    # Public: Whether the object's comments are visible or not.
+    def commentable?
+      type.comments_enabled?
+    end
+
     # Public: Check if an initiative has been created by an individual person.
     # If it's false, then it has been created by an authorized organization.
     #
@@ -215,7 +263,7 @@ module Decidim
         published_at: Time.current,
         state: "published",
         signature_start_date: Date.current,
-        signature_end_date: signature_end_date || Date.current + Decidim::Initiatives.default_signature_time_period_length
+        signature_end_date: signature_end_date || (Date.current + Decidim::Initiatives.default_signature_time_period_length)
       )
     end
 
@@ -420,6 +468,10 @@ module Decidim
       ActionAuthorizer.new(user, "comment", self, nil).authorize.ok?
     end
 
+    def self.ransack(params = {}, options = {})
+      Initiatives::InitiativeSearch.new(self, params, options)
+    end
+
     private
 
     # Private: This is just an alias because the naming on InitiativeTypeScope
@@ -453,11 +505,10 @@ module Decidim
     end
 
     # Allow ransacker to search for a key in a hstore column (`title`.`en`)
-    [:title, :description].each do |column|
-      ransacker column do |parent|
-        Arel::Nodes::InfixOperation.new("->>", parent.table[column], Arel::Nodes.build_quoted(I18n.locale.to_s))
-      end
-    end
+    [:title, :description].each { |column| ransacker_i18n(column) }
+
+    # Alias search_text as a grouped OR query with all the text searchable fields.
+    ransack_alias :search_text, :id_string_or_title_or_description_or_author_name_or_author_nickname
 
     # Allow ransacker to search on an Enum Field
     ransacker :state, formatter: proc { |int| states[int] }

data/app/models/decidim/initiatives_settings.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module Decidim
+  # Initiatives setting.
+  class InitiativesSettings < ApplicationRecord
+    include Decidim::Traceable
+    include Decidim::Loggable
+
+    belongs_to :organization,
+               foreign_key: "decidim_organization_id",
+               class_name: "Decidim::Organization"
+
+    def self.log_presenter_class_for(_log)
+      Decidim::Initiatives::AdminLog::InitiativesSettingsPresenter
+    end
+  end
+end

data/app/models/decidim/initiatives_type.rb

@@ -6,6 +6,7 @@ module Decidim
     include Decidim::HasResourcePermission
     include Decidim::TranslatableResource
     include Decidim::HasUploadValidations
+    include Decidim::Traceable
 
     translatable_fields :title, :description, :extra_fields_legal_information
 
@@ -48,5 +49,9 @@ module Decidim
     def mounted_params
       { host: organization.host }
     end
+
+    def self.log_presenter_class_for(_log)
+      Decidim::Initiatives::AdminLog::InitiativesTypePresenter
+    end
   end
 end

data/app/permissions/decidim/initiatives/admin/permissions.rb

@@ -19,6 +19,7 @@ module Decidim
             initiative_committee_action?
             initiative_user_action?
             attachment_action?
+            initiatives_settings_action?
 
             return permission_action
           end
@@ -36,6 +37,7 @@ module Decidim
           initiative_committee_action?
           initiative_admin_user_action?
           initiative_export_action?
+          initiatives_settings_action?
           moderator_action?
           allow! if permission_action.subject == :attachment
 
@@ -134,12 +136,10 @@ module Decidim
           case permission_action.action
           when :read
             toggle_allow(Decidim::Initiatives.print_enabled)
-          when :publish
+          when :publish, :discard
             toggle_allow(initiative.validating?)
           when :unpublish
             toggle_allow(initiative.published?)
-          when :discard
-            toggle_allow(initiative.validating?)
           when :export_pdf_signatures
             toggle_allow(initiative.published? || initiative.accepted? || initiative.rejected?)
           when :export_votes
@@ -165,6 +165,13 @@ module Decidim
           allow! if permission_action.subject == :initiatives && permission_action.action == :export
         end
 
+        def initiatives_settings_action?
+          return unless permission_action.action == :update &&
+                        permission_action.subject == :initiatives_settings
+
+          toggle_allow(user.admin?)
+        end
+
         def moderator_action?
           return unless permission_action.subject == :moderation
 

data/app/permissions/decidim/initiatives/permissions.rb

@@ -107,7 +107,7 @@ module Decidim
       end
 
       def access_request_without_user?
-        !initiative.published? && initiative.promoting_committee_enabled? || Decidim::Initiatives.do_not_require_authorization
+        (!initiative.published? && initiative.promoting_committee_enabled?) || Decidim::Initiatives.do_not_require_authorization
       end
 
       def access_request_membership?

data/app/presenters/decidim/initiatives/admin_log/initiatives_settings_presenter.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Initiatives
+    module AdminLog
+      # This class holds the logic to present a `Decidim::InitiativesSettings`
+      # for the `AdminLog` log.
+      #
+      # Usage should be automatic and you shouldn't need to call this class
+      # directly, but here's an example:
+      #
+      #    action_log = Decidim::ActionLog.last
+      #    view_helpers # => this comes from the views
+      #    InitiativesSettingsPresenter.new(action_log, view_helpers).present
+      class InitiativesSettingsPresenter < Decidim::Log::BasePresenter
+        private
+
+        def action_string
+          case action
+          when "update"
+            "decidim.initiatives.admin_log.initiatives_settings.#{action}"
+          end
+        end
+      end
+    end
+  end
+end

data/app/presenters/decidim/initiatives/admin_log/initiatives_type_presenter.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Initiatives
+    module AdminLog
+      # This class holds the logic to present a `Decidim::InitiativesType`
+      # for the `AdminLog` log.
+      #
+      # Usage should be automatic and you shouldn't need to call this class
+      # directly, but here's an example:
+      #
+      #    action_log = Decidim::ActionLog.last
+      #    view_helpers # => this comes from the views
+      #    InitiativesTypePresenter.new(action_log, view_helpers).present
+      class InitiativesTypePresenter < Decidim::Log::BasePresenter
+        private
+
+        def action_string
+          case action
+          when "create", "update", "delete"
+            "decidim.initiatives.admin_log.initiatives_type.#{action}"
+          else
+            super
+          end
+        end
+
+        def diff_fields_mapping
+          {
+            description: :i18n,
+            title: :i18n,
+            extra_fields_legal_information: :i18n,
+            minimum_committee_members: :integer,
+            document_number_authorization_handler: :i18n,
+            undo_online_signatures_enabled: :boolean,
+            promoting_committee_enabled: :boolean
+          }
+        end
+
+        def diff_actions
+          super + %w(update)
+        end
+      end
+    end
+  end
+end

data/app/presenters/decidim/initiatives/initiative_stats_presenter.rb

@@ -3,8 +3,10 @@
 module Decidim
   module Initiatives
     # A presenter to render statistics in the homepage.
-    class InitiativeStatsPresenter < Rectify::Presenter
-      attribute :initiative, Decidim::Initiative
+    class InitiativeStatsPresenter < SimpleDelegator
+      def initiative
+        __getobj__.fetch(:initiative)
+      end
 
       def comments_count
         Rails.cache.fetch(

data/app/queries/decidim/initiatives/admin/admin_users.rb

@@ -4,7 +4,7 @@ module Decidim
   module Initiatives
     module Admin
       # A class used to find the admins for an initiative or an organization initiatives.
-      class AdminUsers < Rectify::Query
+      class AdminUsers < Decidim::Query
         # Syntactic sugar to initialize the class and return the queried objects.
         #
         # initiative - Decidim::Initiative

data/app/queries/decidim/initiatives/admin/manageable_initiatives.rb

@@ -6,7 +6,7 @@ module Decidim
       # Class that retrieves manageable initiatives for the given user.
       # Regular users will get only their initiatives. Administrators will
       # retrieve all initiatives.
-      class ManageableInitiatives < Rectify::Query
+      class ManageableInitiatives < Decidim::Query
         # Syntactic sugar to initialize the class and return the queried objects
         #
         # user - Decidim::User

data/app/queries/decidim/initiatives/freetext_initiative_types.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Initiatives
     # This query searches scopes by name.
-    class FreetextInitiativeTypes < Rectify::Query
+    class FreetextInitiativeTypes < Decidim::Query
       # Syntactic sugar to initialize the class and return the queried objects.
       #
       # organization - an Organization context for the initiative type search

data/app/queries/decidim/initiatives/initiative_types.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Initiatives
     # Class uses to retrieve the available initiative types.
-    class InitiativeTypes < Rectify::Query
+    class InitiativeTypes < Decidim::Query
       # Syntactic sugar to initialize the class and return the queried objects.
       #
       # organization - Decidim::Organization

data/app/queries/decidim/initiatives/initiatives_created.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Initiatives
     # Class uses to retrieve the initiatives created by the given user.
-    class InitiativesCreated < Rectify::Query
+    class InitiativesCreated < Decidim::Query
       attr_reader :user
 
       # Syntactic sugar to initialize the class and return the queried objects.

data/app/queries/decidim/initiatives/initiatives_promoted.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Initiatives
     # Class uses to retrieve initiatives promoted by the given  user
-    class InitiativesPromoted < Rectify::Query
+    class InitiativesPromoted < Decidim::Query
       attr_reader :user
 
       # Syntactic sugar to initialize the class and return the queried objects.

data/app/queries/decidim/initiatives/organization_prioritized_initiatives.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Initiatives
     # This query retrieves the organization prioritized initiatives that will appear in the homepage
-    class OrganizationPrioritizedInitiatives < Rectify::Query
+    class OrganizationPrioritizedInitiatives < Decidim::Query
       attr_reader :organization, :order
 
       def initialize(organization, order)

data/app/queries/decidim/initiatives/outdated_validating_initiatives.rb

@@ -4,7 +4,7 @@ module Decidim
   module Initiatives
     # Class uses to retrieve initiatives that have been a long time in
     # validating state
-    class OutdatedValidatingInitiatives < Rectify::Query
+    class OutdatedValidatingInitiatives < Decidim::Query
       # Syntactic sugar to initialize the class and return the queried objects.
       #
       # period_length - Maximum time in validating state

data/app/queries/decidim/initiatives/similar_initiatives.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Initiatives
     # Class uses to retrieve similar initiatives types.
-    class SimilarInitiatives < Rectify::Query
+    class SimilarInitiatives < Decidim::Query
       include Decidim::TranslationsHelper
       include CurrentLocale
 

data/app/queries/decidim/initiatives/support_period_finished_initiatives.rb

@@ -4,7 +4,7 @@ module Decidim
   module Initiatives
     # Class uses to retrieve initiatives that have been a long time in validating
     # state
-    class SupportPeriodFinishedInitiatives < Rectify::Query
+    class SupportPeriodFinishedInitiatives < Decidim::Query
       # Retrieves the initiatives ready to be evaluated to decide if they've been
       # accepted or not.
       def query

data/app/queries/decidim/initiatives/user_authorizations.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Initiatives
     # Class used the retrieve the authorizations for a user.
-    class UserAuthorizations < Rectify::Query
+    class UserAuthorizations < Decidim::Query
       attr_reader :user
 
       # Syntactic sugar to initialize the class and return the queried objects.