decidim-admin 0.29.1 → 0.30.0.rc1

data/app/forms/concerns/decidim/has_taxonomy_form_attributes.rb

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+module Decidim
+  module HasTaxonomyFormAttributes
+    extend ActiveSupport::Concern
+
+    included do
+      attribute :taxonomies, Array[Integer]
+
+      validate :taxonomies_belong_to_current_organization
+
+      # Returns the participatory space manifest for search the available filters (ie: participatory_processes, assemblies, etc)
+      # To implement where this concern is included.
+      def participatory_space_manifest
+        raise NotImplementedError
+      end
+
+      def taxonomizations
+        @taxonomizations ||= compact_taxonomies.map do |taxonomy_id|
+          Decidim::Taxonomization.new(taxonomy_id:)
+        end
+      end
+
+      def taxonomy_filters
+        @taxonomy_filters ||= if defined?(current_component) && current_component&.settings.respond_to?(:taxonomy_filters)
+                                all_taxonomy_filters.where(id: current_component.settings.taxonomy_filters)
+                              else
+                                all_taxonomy_filters.for_manifest(participatory_space_manifest)
+                              end
+      end
+
+      def all_taxonomy_filters
+        @all_taxonomy_filters ||= TaxonomyFilter.where(root_taxonomy: root_taxonomies)
+      end
+
+      def root_taxonomies
+        @root_taxonomies ||= current_organization.taxonomies.roots
+      end
+
+      private
+
+      def taxonomies_belong_to_current_organization
+        return if compact_taxonomies.empty?
+
+        Decidim::Taxonomy.where(id: compact_taxonomies).find_each do |taxonomy|
+          next if taxonomy.decidim_organization_id == current_organization.id
+
+          errors.add(:taxonomies, :invalid)
+        end
+      end
+
+      def compact_taxonomies
+        @compact_taxonomies ||= taxonomies.compact
+      end
+    end
+  end
+end

data/app/forms/decidim/admin/block_users_form.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Admin
+    # A form object used to block users or user groups on the admin dashboard.
+    class BlockUsersForm < Form
+      attribute :user_ids, Array[Integer]
+      attribute :justification, String
+      attribute :hide, Boolean, default: false
+
+      validates :justification, presence: true, length: { minimum: UserBlock::MINIMUM_JUSTIFICATION_LENGTH }
+
+      def users
+        @users ||= Decidim::UserBaseEntity.where(
+          id: user_ids,
+          organization: current_organization
+        )
+      end
+    end
+  end
+end

data/app/forms/decidim/admin/help_section_form.rb

@@ -9,7 +9,7 @@ module Decidim
       include TranslationsHelper
 
       attribute :id, String
-      translatable_attribute :content, String
+      translatable_attribute :content, Decidim::Attributes::RichText
 
       def name
         multi_translation("activerecord.models.#{manifest.model_class_name.underscore}.other")

data/app/forms/decidim/admin/impersonate_user_form.rb

@@ -14,6 +14,7 @@ module Decidim
       attribute :handler_name, String
 
       validates :user, presence: true
+      validates :name, presence: true, unless: :persisted_user?
       validates :reason, presence: true, unless: :managed_user?
 
       private
@@ -21,6 +22,10 @@ module Decidim
       def managed_user?
         user && user.managed?
       end
+
+      def persisted_user?
+        user&.persisted?
+      end
     end
   end
 end

data/app/forms/decidim/admin/import_example_form.rb

@@ -35,7 +35,7 @@ module Decidim
       end
 
       def reader
-        @reader ||= reader_klass ? reader_klass.new("/dev/null") : nil
+        @reader ||= reader_klass ? reader_klass.new(File::NULL) : nil
       end
 
       def reader_klass

data/app/forms/decidim/admin/newsletter_form.rb

@@ -12,7 +12,7 @@ module Decidim
       validates :subject, translatable_presence: true
 
       def map_model(content_block)
-        super(content_block)
+        super
         self.subject = newsletter_for(content_block).try(:subject)
       end
 

data/app/forms/decidim/admin/organization_appearance_form.rb

@@ -32,9 +32,9 @@ module Decidim
       attribute :tertiary_color, String
 
       translatable_attribute :cta_button_text, String
-      translatable_attribute :description, String
+      translatable_attribute :description, Decidim::Attributes::RichText
       translatable_attribute :highlighted_content_banner_title, String
-      translatable_attribute :highlighted_content_banner_short_description, String
+      translatable_attribute :highlighted_content_banner_short_description, Decidim::Attributes::RichText
       translatable_attribute :highlighted_content_banner_action_title, String
       translatable_attribute :highlighted_content_banner_action_subtitle, String
       translatable_attribute :omnipresent_banner_title, String

data/app/forms/decidim/admin/organization_form.rb

@@ -31,9 +31,9 @@ module Decidim
       attribute :customize_welcome_notification, Boolean
 
       translatable_attribute :welcome_notification_subject, String
-      translatable_attribute :welcome_notification_body, String
+      translatable_attribute :welcome_notification_body, Decidim::Attributes::RichText
 
-      translatable_attribute :admin_terms_of_service_body, String
+      translatable_attribute :admin_terms_of_service_body, Decidim::Attributes::RichText
 
       validates :welcome_notification_subject, :welcome_notification_body, translatable_presence: true, if: proc { |form| form.customize_welcome_notification }
 

data/app/forms/decidim/admin/participatory_space_private_user_form.rb

@@ -6,10 +6,15 @@ module Decidim
     # admin dashboard.
     #
     class ParticipatorySpacePrivateUserForm < Form
+      include TranslatableAttributes
+
       mimic :participatory_space_private_user
 
       attribute :name, String
       attribute :email, String
+      attribute :published, Boolean
+
+      translatable_attribute :role, String
 
       validates :name, :email, presence: true
 

data/app/forms/decidim/admin/selective_newsletter_form.rb

@@ -7,33 +7,33 @@ module Decidim
       mimic :newsletter
 
       attribute :participatory_space_types, Array[SelectiveNewsletterParticipatorySpaceTypeForm]
-      attribute :scope_ids, Array
+      attribute :verification_types, Array[String]
       attribute :send_to_all_users, Boolean
+      attribute :send_to_verified_users, Boolean
       attribute :send_to_participants, Boolean
       attribute :send_to_followers, Boolean
+      attribute :send_to_private_members, Boolean
 
-      validates :send_to_all_users, presence: true, unless: ->(form) { form.send_to_participants.present? || form.send_to_followers.present? }
-      validates :send_to_followers, presence: true, if: ->(form) { form.send_to_all_users.blank? && form.send_to_participants.blank? }
-      validates :send_to_participants, presence: true, if: ->(form) { form.send_to_all_users.blank? && form.send_to_followers.blank? }
+      validates :send_to_all_users, presence: true, unless: :other_groups_selected_for_all_users?
+      validates :send_to_verified_users, presence: true, unless: :other_groups_selected_for_verified_users?
+      validates :send_to_followers, presence: true, if: :only_followers_selected?
+      validates :send_to_participants, presence: true, if: :only_participants_selected?
+      validates :send_to_private_members, presence: true, if: :only_private_members_selected?
 
       validate :at_least_one_participatory_space_selected
 
-      def map_model(_newsletter)
+      def map_model(newsletter)
         self.participatory_space_types = Decidim.participatory_space_manifests.map do |manifest|
           SelectiveNewsletterParticipatorySpaceTypeForm.from_model(manifest:)
         end
-      end
 
-      # Make sure the empty scope is not passed because then some logic could
-      # assume erroneously that some scope is selected.
-      def scope_ids
-        super.select(&:presence)
+        self.verification_types = newsletter.organization.available_authorizations
       end
 
       private
 
       def at_least_one_participatory_space_selected
-        return if send_to_all_users && current_user.admin?
+        return if (send_to_all_users || send_to_verified_users) && current_user.admin?
 
         errors.add(:base, :at_least_one_space) if spaces_selected.blank?
       end
@@ -44,6 +44,41 @@ module Decidim
           [type.manifest_name, spaces] if spaces.present?
         end.compact
       end
+
+      def other_groups_selected_for_all_users?
+        send_to_verified_users.present? ||
+          send_to_participants.present? ||
+          send_to_followers.present? ||
+          send_to_private_members.present?
+      end
+
+      def other_groups_selected_for_verified_users?
+        send_to_all_users.present? ||
+          send_to_participants.present? ||
+          send_to_followers.present? ||
+          send_to_private_members.present?
+      end
+
+      def only_followers_selected?
+        send_to_all_users.blank? &&
+          send_to_participants.blank? &&
+          send_to_private_members.blank? &&
+          send_to_verified_users.blank?
+      end
+
+      def only_participants_selected?
+        send_to_all_users.blank? &&
+          send_to_followers.blank? &&
+          send_to_private_members.blank? &&
+          send_to_verified_users.blank?
+      end
+
+      def only_private_members_selected?
+        send_to_all_users.blank? &&
+          send_to_followers.blank? &&
+          send_to_participants.blank? &&
+          send_to_verified_users.blank?
+      end
     end
   end
 end

data/app/forms/decidim/admin/share_token_form.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Admin
+    class ShareTokenForm < Decidim::Form
+      mimic :share_token
+
+      attribute :token, String
+      attribute :automatic_token, Boolean, default: true
+      attribute :expires_at, Decidim::Attributes::TimeWithZone
+      attribute :no_expiration, Boolean, default: true
+      attribute :registered_only, Boolean, default: false
+
+      validates :token, presence: true, if: ->(form) { form.automatic_token.blank? }
+      validate :token_uniqueness, if: ->(form) { form.automatic_token.blank? }
+
+      validates_format_of :token, with: /\A[a-zA-Z0-9_-]+\z/, allow_blank: true
+      validates :expires_at, presence: true, if: ->(form) { form.no_expiration.blank? }
+
+      def map_model(model)
+        self.no_expiration = model.expires_at.blank?
+      end
+
+      def token
+        super.strip.upcase.gsub(/\s+/, "-") if super.present?
+      end
+
+      def expires_at
+        return nil if no_expiration.present?
+
+        super
+      end
+
+      def token_for
+        context[:resource]
+      end
+
+      def organization
+        context[:current_organization]
+      end
+
+      def user
+        context[:current_user]
+      end
+
+      private
+
+      def token_uniqueness
+        return unless Decidim::ShareToken.where(organization:, token_for:, token:).where.not(id:).any?
+
+        errors.add(:token, :taken)
+      end
+    end
+  end
+end

data/app/forms/decidim/admin/static_page_form.rb

@@ -9,7 +9,7 @@ module Decidim
       attribute :slug, String
       validates :title, translatable_presence: true
       translatable_attribute :title, String
-      translatable_attribute :content, String
+      translatable_attribute :content, Decidim::Attributes::RichText
       attribute :changed_notably, Boolean
       attribute :allow_public_access, Boolean
       attribute :weight, Integer

data/app/forms/decidim/admin/taxonomy_filter_form.rb

@@ -0,0 +1,85 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Admin
+    # A form object to create or update areas.
+    class TaxonomyFilterForm < Form
+      include TranslatableAttributes
+      Item = Struct.new(:name, :value, :children)
+      Manifest = Struct.new(:id, :name)
+
+      attribute :root_taxonomy_id, Integer
+      translatable_attribute :name, String
+      translatable_attribute :internal_name, String
+      attribute :participatory_space_manifests, Array, default: []
+      attribute :taxonomy_items, Array
+
+      mimic :taxonomy_filter
+
+      validates :root_taxonomy_id, :taxonomy_items, presence: true
+      validate :valid_taxonomy_items
+
+      def map_model(model)
+        self.root_taxonomy_id = model.root_taxonomy_id
+        self.taxonomy_items = model.filter_items.map(&:taxonomy_item_id)
+        self.name = {} if model.attributes["name"]&.compact_blank.blank?
+        self.internal_name = {} if model.attributes["internal_name"]&.compact_blank.blank?
+      end
+
+      def taxonomy_items
+        super.compact_blank
+      end
+
+      def participatory_space_manifests
+        super.compact_blank
+      end
+
+      def filter_items
+        taxonomy_items.map do |item|
+          Decidim::TaxonomyFilterItem.new(taxonomy_item_id: item)
+        end
+      end
+
+      def items_collection
+        return [] unless root_taxonomy
+
+        @items_collection ||= map_items_collection(root_taxonomy)
+      end
+
+      def root_taxonomy
+        @root_taxonomy ||= current_organization.taxonomies.find_by(id: root_taxonomy_id)
+      end
+
+      def available_participatory_space_manifests
+        @participatory_space_manifests ||= Decidim.participatory_space_manifests.map do |manifest|
+          Manifest.new(
+            id: manifest.name.to_s,
+            name: I18n.t("decidim.admin.taxonomy_filters.space_filter_for.#{manifest.name}")
+          )
+        end
+      end
+
+      private
+
+      def map_items_collection(taxonomy)
+        taxonomy.children.map do |item|
+          Item.new(
+            name: translated_attribute(item.name),
+            value: item.id,
+            children: map_items_collection(item)
+          )
+        end
+      end
+
+      def valid_taxonomy_items
+        return if taxonomy_items.all? do |item|
+          next unless root_taxonomy
+
+          root_taxonomy.all_children.map(&:id).include?(item.to_i)
+        end
+
+        errors.add(:taxonomy_items, :invalid)
+      end
+    end
+  end
+end

data/app/forms/decidim/admin/taxonomy_form.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Admin
+    # A form object to be used when creating or updating a taxonomy.
+    class TaxonomyForm < Decidim::Form
+      include Decidim::TranslatableAttributes
+
+      mimic :taxonomy
+
+      translatable_attribute :name, String
+
+      validates :name, translatable_presence: true
+
+      alias organization current_organization
+
+      def parent_id = nil
+    end
+  end
+end

data/app/forms/decidim/admin/taxonomy_item_form.rb

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Admin
+    # A form object to be used when creating or updating a taxonomy.
+    class TaxonomyItemForm < Decidim::Form
+      include Decidim::TranslatableAttributes
+
+      mimic :taxonomy
+
+      # we do not use "name" here to avoid collisions when using foundation tabs for multilingual fields tabs
+      # as this is used in a modal and the name identifier is used for the root taxonomy
+      translatable_attribute :item_name, String
+      attribute :parent_id, Integer
+
+      validates :item_name, translatable_presence: true
+      validate :validate_parent_id_within_same_root_taxonomy
+
+      alias name item_name
+
+      def map_model(model)
+        self.item_name = model.name
+      end
+
+      def self.from_params(params, additional_params = {})
+        additional_params[:taxonomy] = {}
+        if params[:taxonomy]
+          params[:taxonomy].each do |key, value|
+            additional_params[:taxonomy][key[8..]] = value if key.start_with?("item_name_")
+          end
+        end
+        super
+      end
+
+      def validate_parent_id_within_same_root_taxonomy
+        if parent
+          current_root_taxonomy = if parent.root?
+                                    parent
+                                  else
+                                    parent.root_taxonomy
+                                  end
+
+          errors.add(:parent_id, :invalid) unless parent.root_taxonomy.id == current_root_taxonomy.id
+        else
+          errors.add(:parent_id, :invalid)
+        end
+      end
+
+      def parent
+        @parent ||= Decidim::Taxonomy.find_by(id: parent_id)
+      end
+    end
+  end
+end

data/app/forms/decidim/admin/transfer_user_form.rb

@@ -14,6 +14,21 @@ module Decidim
 
       validates :current_user, presence: true
       validates :conflict, presence: true
+      validates :email, presence: true
+      validate :unique_email
+
+      private
+
+      def unique_email
+        return if conflict.blank?
+        return true if Decidim::UserBaseEntity.where(
+          organization: context.current_organization,
+          email:
+        ).where.not(id: [conflict.current_user.id, conflict.managed_user_id]).empty?
+
+        errors.add :email, :taken
+        false
+      end
     end
   end
 end

data/app/helpers/decidim/admin/application_helper.rb

@@ -13,7 +13,6 @@ module Decidim
       include Decidim::MapHelper
       include Decidim::Admin::LogRenderHelper
       include Decidim::Admin::UserRolesHelper
-      include Decidim::Admin::ResourceScopeHelper
       include Decidim::Admin::SearchFormHelper
 
       # Public: Overwrites the `cell` helper method to automatically set some

data/app/helpers/decidim/admin/bulk_actions_helper.rb

@@ -3,37 +3,6 @@
 module Decidim
   module Admin
     module BulkActionsHelper
-      # Public: Generates a select field with the categories. Only leaf categories can be set as selected.
-      #
-      # categories - A collection of categories.
-      #
-      # Returns a String.
-      def bulk_categories_select(collection)
-        categories = bulk_categories_for_select collection
-        prompt = t("decidim.proposals.admin.proposals.index.change_category")
-        select(:category, :id, options_for_select(categories, selected: []), prompt:)
-      end
-
-      def bulk_categories_for_select(scope)
-        sorted_main_categories = scope.first_class.includes(:subcategories).sort_by do |category|
-          translated_attribute(category.name, category.participatory_space.organization)
-        end
-
-        sorted_main_categories.flat_map do |category|
-          parent = [[translated_attribute(category.name, category.participatory_space.organization), category.id]]
-
-          sorted_subcategories = category.subcategories.sort_by do |subcategory|
-            translated_attribute(subcategory.name, subcategory.participatory_space.organization)
-          end
-
-          sorted_subcategories.each do |subcategory|
-            parent << ["- #{translated_attribute(subcategory.name, subcategory.participatory_space.organization)}", subcategory.id]
-          end
-
-          parent
-        end
-      end
-
       # Public: Generates a select field with the components.
       #
       # siblings - A collection of components.

data/app/helpers/decidim/admin/moderations/reports_helper.rb

@@ -14,7 +14,7 @@ module Decidim
         def reportable_author_name(reportable)
           reportable_authors = reportable.try(:authors) || [reportable.try(:normalized_author)]
           content_tag :ul, class: "reportable-authors" do
-            reportable_authors.select(&:present?).map do |author|
+            reportable_authors.compact_blank.map do |author|
               case author
               when User
                 content_tag :li do

data/app/helpers/decidim/admin/moderations_helper.rb

@@ -13,7 +13,7 @@ module Decidim
 
             attribute_value
           end
-          reportable_content.filter(&:present?).join(". ").truncate(options.fetch(:limit, 100))
+          reportable_content.compact_blank.join(". ").truncate(options.fetch(:limit, 100))
         end
       end