decidim-admin 0.24.3 → 0.25.0.rc4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 325402c97956e2279a1f445f2cd88e268f3c02f3d960aefc148a77a695228043
-  data.tar.gz: 3e8728e72a5bc5eee562e55cb44021aa75377c8c0300139fb8c0345c02fbbf79
+  metadata.gz: 8ea05dffe648cb9ed9ec858c4c53df7fe0fa3c3eba2b50caaf262c9ed94019ba
+  data.tar.gz: 5acc3334f8aaf0dab7165b632a61c790ac4027ca6b28e97c83da8569c584af32
 SHA512:
-  metadata.gz: 8aa7a02f5aa51d9f15afc1bdd6a2aa450d885e92674328929ec02d2fac98e8a7b62e3f8daf32f13480790869e6e64c9d4fc019686b3758d027d8f0059699d22d
-  data.tar.gz: e7140826d72d65006e121608884e20aa7db8c837da7a105570d97535a151ed2493fcfef0bf203205641f791e5ca979b29b9c0c2a7b8f161da54d1fec2e502860
+  metadata.gz: 880981f7f0c5bd52a2514dcd6eaffba8da6e6aae8ce2a18c464b78c75204790b4c6f494eea517d9bf14395b5e595933dd3f4c96d0728eefd86e53a859caca3fe
+  data.tar.gz: fee5bed6ee9306bd4c488732db89ccbb56520e67ec9c199dce8590a2b1016fdb4d1fcb02f71cb220e0b08801ed83004d3490561d4f7fa568d81df1b448b32618

data/app/cells/decidim/admin/content_block/show.erb

@@ -4,10 +4,10 @@
     <div>
       <% if has_settings? %>
         <%= link_to edit_content_block_path, class: "mr-s text-muted" do %>
-          <%= icon "pencil", role: "img" %>
+          <%= icon "pencil", role: "img", "aria-hidden": true %>
         <% end %>
       <% end %>
-      <%= icon "menu", role: "img" %>
+      <%= icon "menu", role: "img", "aria-hidden": true %>
     </div>
   </div>
 </li>

data/app/commands/decidim/admin/block_user.rb

@@ -55,6 +55,12 @@ module Decidim
           extra: {
             reportable_type: form.user.class.name,
             current_justification: form.justification
+          },
+          resource: {
+            # Make sure the action log entry gets the original user name instead
+            # of "Blocked user". Otherwise the log entries would show funny
+            # messages such as "Mr. Admin blocked user Blocked user"-
+            title: form.user.name
           }
         ) do
           form.user.blocked = true

data/app/commands/decidim/admin/create_attachment.rb

@@ -46,7 +46,8 @@ module Decidim
           attached_to: @attached_to,
           weight: form.weight,
           attachment_collection: form.attachment_collection,
-          file: form.file # Define attached_to before this
+          file: form.file, # Define attached_to before this
+          content_type: form.file.content_type
         )
       end
 

data/app/commands/decidim/admin/hide_resource.rb

@@ -45,6 +45,7 @@ module Decidim
           }
         ) do
           @reportable.moderation.update!(hidden_at: Time.current)
+          @reportable.try(:touch)
         end
       end
 

data/app/commands/decidim/admin/process_participatory_space_private_user_import_csv.rb

@@ -32,7 +32,7 @@ module Decidim
       private
 
       def process_csv
-        CSV.foreach(@form.file.path) do |email, user_name|
+        CSV.foreach(@form.file.path, encoding: "BOM|UTF-8") do |email, user_name|
           ImportParticipatorySpacePrivateUserCsvJob.perform_later(email, user_name, @private_users_to, @current_user) if email.present? && user_name.present?
         end
       end

data/app/commands/decidim/admin/update_attachment.rb

@@ -5,6 +5,8 @@ module Decidim
     # A command with all the business logic to update an attachment from a
     # participatory process.
     class UpdateAttachment < Rectify::Command
+      include ::Decidim::AttachmentAttributesMethods
+
       attr_reader :attachment
 
       # Public: Initializes the command.
@@ -44,7 +46,9 @@ module Decidim
           description: form.description,
           weight: form.weight,
           attachment_collection: form.attachment_collection
-        }.reject do |attribute, value|
+        }.merge(
+          attachment_attributes(:file)
+        ).reject do |attribute, value|
           value.blank? && attribute != :attachment_collection
         end
       end

data/app/commands/decidim/admin/update_content_block.rb

@@ -70,7 +70,7 @@ module Decidim
           if form.images[image_name]
             content_block.images_container.send("#{image_name}=", form.images[image_name])
           elsif form.images["remove_#{image_name}".to_sym] == "1"
-            content_block.images_container.send("remove_#{image_name}=", true)
+            content_block.images_container.send("#{image_name}=", nil)
           end
         end
       end

data/app/commands/decidim/admin/update_external_domain_whitelist.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Admin
+    class UpdateExternalDomainWhitelist < Rectify::Command
+      attr_reader :form, :organization
+
+      def initialize(form, organization)
+        @form = form
+        @organization = organization
+      end
+
+      def call
+        return broadcast(:invalid) if form.invalid?
+
+        save_domains!
+
+        broadcast(:ok)
+      end
+
+      private
+
+      def save_domains!
+        organization.external_domain_whitelist = form.external_domains.filter_map do |external_domain_form|
+          external_domain_form.value unless external_domain_form.deleted
+        end.flatten
+
+        organization.save!
+      end
+    end
+  end
+end

data/app/commands/decidim/admin/update_organization.rb

@@ -56,7 +56,8 @@ module Decidim
           comments_max_length: form.comments_max_length,
           enable_machine_translations: form.enable_machine_translations,
           admin_terms_of_use_body: form.admin_terms_of_use_body,
-          rich_text_editor_in_public_views: form.rich_text_editor_in_public_views
+          rich_text_editor_in_public_views: form.rich_text_editor_in_public_views,
+          enable_participatory_space_filters: form.enable_participatory_space_filters
         }.merge(welcome_notification_attributes)
           .merge(machine_translation_attributes || {})
       end

data/app/commands/decidim/admin/update_organization_appearance.rb

@@ -5,14 +5,13 @@ module Decidim
     # A command with all the business logic for updating the current
     # organization appearance.
     class UpdateOrganizationAppearance < Rectify::Command
+      include ::Decidim::AttachmentAttributesMethods
+
       # Public: Initializes the command.
       #
       # organization - The Organization that will be updated.
       # form - A form object with the params.
       def initialize(organization, form)
-        image_fields.each do |field|
-          form.send("#{field}=".to_sym, organization.send(field)) if form.send(field).blank?
-        end
         @organization = organization
         @form = form
       end
@@ -40,7 +39,7 @@ module Decidim
       private
 
       def image_fields
-        [:highlighted_content_banner_image, :logo, :favicon, :official_img_header, :official_img_footer]
+        [:logo, :highlighted_content_banner_image, :favicon, :official_img_header, :official_img_footer]
       end
 
       attr_reader :form, :organization
@@ -55,6 +54,7 @@ module Decidim
 
       def attributes
         appearance_attributes
+          .merge(attachment_attributes(*image_fields))
           .merge(highlighted_content_banner_attributes)
           .merge(omnipresent_banner_attributes)
           .merge(colors_attributes)
@@ -69,14 +69,6 @@ module Decidim
           cta_button_path: form.cta_button_path,
           cta_button_text: form.cta_button_text,
           description: form.description,
-          logo: form.logo,
-          remove_logo: form.remove_logo,
-          favicon: form.favicon,
-          remove_favicon: form.remove_favicon,
-          official_img_header: form.official_img_header,
-          remove_official_img_header: form.remove_official_img_header,
-          official_img_footer: form.official_img_footer,
-          remove_official_img_footer: form.remove_official_img_footer,
           official_url: form.official_url
         }
       end
@@ -85,8 +77,6 @@ module Decidim
         {
           highlighted_content_banner_enabled: form.highlighted_content_banner_enabled,
           highlighted_content_banner_action_url: form.highlighted_content_banner_action_url,
-          highlighted_content_banner_image: form.highlighted_content_banner_image,
-          remove_highlighted_content_banner_image: form.remove_highlighted_content_banner_image,
           highlighted_content_banner_title: form.highlighted_content_banner_title,
           highlighted_content_banner_short_description: form.highlighted_content_banner_short_description,
           highlighted_content_banner_action_title: form.highlighted_content_banner_action_title,

data/app/controllers/concerns/decidim/admin/filterable.rb

@@ -105,9 +105,7 @@ module Decidim
         # with true and false as values. If these values fit your filtering needs,
         # you won't have to declare a `filters_with_values` method in your concern.
         def filters_with_values
-          filters.each_with_object({}) do |filter, hash|
-            hash[filter] = [true, false]
-          end
+          filters.index_with { [true, false] }
         end
 
         # Plural model name. Used in search_field placeholder.

data/app/controllers/concerns/decidim/participatory_space_private_users/admin/filterable.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+require "active_support/concern"
+
+module Decidim
+  module ParticipatorySpacePrivateUsers
+    module Admin
+      module Filterable
+        extend ActiveSupport::Concern
+
+        included do
+          include Decidim::Admin::Filterable
+
+          private
+
+          def base_query
+            collection
+          end
+
+          def filters
+            [
+              :user_invitation_sent_at_not_null,
+              :user_invitation_accepted_at_not_null
+            ]
+          end
+
+          def search_field_predicate
+            :user_name_or_user_email_cont
+          end
+        end
+      end
+    end
+  end
+end

data/app/controllers/decidim/admin/concerns/has_attachments.rb

@@ -116,10 +116,7 @@ module Decidim
           private
 
           def attachment_params
-            {
-              id: params[:id],
-              file: @attachment.file
-            }.merge(params[:attachment].to_unsafe_h)
+            { id: params[:id] }.merge(params[:attachment].to_unsafe_h)
           end
         end
       end

data/app/controllers/decidim/admin/concerns/has_private_users.rb

@@ -13,8 +13,9 @@ module Decidim
         extend ActiveSupport::Concern
 
         included do
+          include Decidim::ParticipatorySpacePrivateUsers::Admin::Filterable
           helper PaginateHelper
-          helper_method :privatable_to, :collection
+          helper_method :privatable_to, :participatory_space_private_users
 
           def index
             enforce_permission_to :read, :space_private_user
@@ -99,8 +100,10 @@ module Decidim
             @collection ||= privatable_to
                             .participatory_space_private_users
                             .includes(:user).where.not("decidim_users.id" => nil)
-                            .page(params[:page])
-                            .per(20)
+          end
+
+          def participatory_space_private_users
+            filtered_collection
           end
         end
       end

data/app/controllers/decidim/admin/imports_controller.rb

@@ -5,6 +5,7 @@ module Decidim
     # This controller allows admins to import resources from a file.
     class ImportsController < Decidim::Admin::ApplicationController
       include Decidim::ComponentPathHelper
+      helper UserGroupHelper
 
       def new
         enforce_permission_to :import, :component_data, component: current_component

data/app/controllers/decidim/admin/moderated_users_controller.rb

@@ -32,12 +32,16 @@ module Decidim
       private
 
       def reportable
-        @reportable ||= UserModeration.find(params[:id]).user
+        @reportable ||= base_query_finder.find(params[:id]).user
+      end
+
+      def base_query_finder
+        UserModeration.joins(:user).where(decidim_users: { decidim_organization_id: current_organization.id })
       end
 
       def collection
         target_scope = params[:blocked] && params[:blocked] == "true" ? :blocked : :unblocked
-        UserModeration.send(target_scope)
+        base_query_finder.send(target_scope)
       end
     end
   end

data/app/controllers/decidim/admin/organization_controller.rb

@@ -30,10 +30,20 @@ module Decidim
       end
 
       def users
+        search(current_organization.users)
+      end
+
+      def user_entities
+        search(current_organization.user_entities)
+      end
+
+      private
+
+      def search(relation)
         respond_to do |format|
           format.json do
             if (term = params[:term].to_s).present?
-              query = current_organization.users.order(name: :asc)
+              query = relation.order(name: :asc)
               query = if term.start_with?("@")
                         query.where("nickname ILIKE ?", "#{term.delete("@")}%")
                       else

data/app/controllers/decidim/admin/organization_external_domain_whitelist_controller.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Admin
+    class OrganizationExternalDomainWhitelistController < Decidim::Admin::ApplicationController
+      layout "decidim/admin/settings"
+
+      helper_method :blank_external_domain
+
+      def edit
+        enforce_permission_to :update, :organization, organization: current_organization
+        @form = form(OrganizationExternalDomainWhitelistForm).from_model(current_organization)
+      end
+
+      def update
+        enforce_permission_to :update, :organization, organization: current_organization
+        @form = form(OrganizationExternalDomainWhitelistForm).from_params(params)
+
+        UpdateExternalDomainWhitelist.call(@form, current_organization) do
+          on(:ok) do
+            flash[:notice] = t("domain_whitelist.update.success", scope: "decidim.admin")
+            redirect_to edit_organization_external_domain_whitelist_path
+          end
+          on(:invalid) do
+            flash[:notice] = t("domain_whitelist.update.error", scope: "decidim.admin")
+            render action: "edit"
+          end
+        end
+      end
+
+      private
+
+      def blank_external_domain
+        @blank_external_domain ||= Admin::ExternalDomainForm.new
+      end
+    end
+  end
+end

data/app/controllers/decidim/admin/users_controller.rb

@@ -5,9 +5,11 @@ module Decidim
     # Controller that allows managing all admins at the admin panel.
     #
     class UsersController < Decidim::Admin::ApplicationController
+      include Decidim::Admin::Officializations::Filterable
+
       def index
         enforce_permission_to :read, :admin_user
-        @users = collection.page(params[:page]).per(15)
+        @users = filtered_collection
       end
 
       def new
@@ -80,6 +82,10 @@ module Decidim
       def collection
         @collection ||= current_organization.admins.or(current_organization.users_with_any_role)
       end
+
+      def filters
+        [:invitation_accepted_at_present, :last_sign_in_at_present]
+      end
     end
   end
 end

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

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Admin
+    class ExternalDomainForm < Form
+      attribute :value, String
+      attribute :position, Integer
+      attribute :deleted, Boolean, default: false
+
+      validates :value, presence: true, unless: :deleted
+
+      def to_param
+        return id if id.present?
+
+        "external-domain-id"
+      end
+    end
+  end
+end

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

@@ -4,6 +4,7 @@ module Decidim
   module Admin
     class ImportForm < Form
       ACCEPTED_MIME_TYPES = Decidim::Admin::Import::Readers::ACCEPTED_MIME_TYPES
+      include Decidim::HasUploadValidations
 
       attribute :creator, String, default: ->(form, _attribute) { form.creators.first[:creator].to_s }
       attribute :file

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

@@ -12,20 +12,20 @@ module Decidim
       mimic :organization
 
       attribute :logo
-      attribute :remove_logo
+      attribute :remove_logo, Boolean, default: false
       attribute :favicon
-      attribute :remove_favicon
+      attribute :remove_favicon, Boolean, default: false
       attribute :official_img_header
-      attribute :remove_official_img_header
+      attribute :remove_official_img_header, Boolean, default: false
       attribute :official_img_footer
-      attribute :remove_official_img_footer
+      attribute :remove_official_img_footer, Boolean, default: false
       attribute :official_url
       attribute :header_snippets, String
       attribute :cta_button_path, String
       attribute :highlighted_content_banner_enabled, Boolean, default: false
       attribute :highlighted_content_banner_action_url, String
       attribute :highlighted_content_banner_image
-      attribute :remove_highlighted_content_banner_image
+      attribute :remove_highlighted_content_banner_image, Boolean, default: false
       attribute :enable_omnipresent_banner, Boolean, default: false
       attribute :omnipresent_banner_url, String
 
@@ -52,7 +52,7 @@ module Decidim
                 :logo,
                 passthru: { to: Decidim::Organization }
 
-      validates :highlighted_content_banner_action_url, presence: true, if: :highlighted_content_banner_enabled?
+      validates :highlighted_content_banner_action_url, url: true, presence: true, if: :highlighted_content_banner_enabled?
       validates :highlighted_content_banner_image,
                 presence: true,
                 passthru: { to: Decidim::Organization },
@@ -70,7 +70,7 @@ module Decidim
                 translatable_presence: true,
                 if: :highlighted_content_banner_enabled?
 
-      validates :omnipresent_banner_url, presence: true, if: :enable_omnipresent_banner?
+      validates :omnipresent_banner_url, url: true, presence: true, if: :enable_omnipresent_banner?
       validates :omnipresent_banner_title, translatable_presence: true, if: :enable_omnipresent_banner?
       validates :omnipresent_banner_short_description, translatable_presence: true, if: :enable_omnipresent_banner?
 

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

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Admin
+    class OrganizationExternalDomainWhitelistForm < Form
+      attribute :external_domains, Array[Decidim::Admin::ExternalDomainForm]
+
+      validate :external_domains_validator
+
+      def map_model(model)
+        self.external_domains = model.external_domain_whitelist.map do |external_domain|
+          ExternalDomainForm.new(value: external_domain)
+        end
+      end
+
+      def external_domains_validator
+        @external_domains_validator ||= external_domains.reject(&:deleted).each do |domain|
+          errors.add(:external_domains, I18n.t("decidim.admin.domain_whitelist.form.domain_too_short")) if domain.value.length <= 3
+        end.map(&:value).uniq
+      end
+    end
+  end
+end

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

@@ -25,6 +25,7 @@ module Decidim
       attribute :rich_text_editor_in_public_views, Boolean
       attribute :enable_machine_translations, Boolean
       attribute :machine_translation_display_priority, String
+      attribute :enable_participatory_space_filters, Boolean
 
       attribute :send_welcome_notification, Boolean
       attribute :customize_welcome_notification, Boolean

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

@@ -7,6 +7,8 @@ module Decidim
     # A form object used to upload CSV to batch participatory space private users.
     #
     class ParticipatorySpacePrivateUserCsvImportForm < Form
+      include Decidim::HasUploadValidations
+
       attribute :file
       attribute :user_name, String
       attribute :email, String

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

@@ -5,6 +5,8 @@ module Decidim
     # A form object used to upload CSV to batch verify user groups.
     #
     class UserGroupCsvVerificationForm < Form
+      include Decidim::HasUploadValidations
+
       attribute :file
 
       validates :file, presence: true

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

@@ -29,7 +29,7 @@ module Decidim
         links = []
         links += extra_dropdown_submenu_options_items(filter, i18n_scope)
         links += values.map { |value| filter_link_value(filter, value, i18n_scope) }
-        links.each_with_object({}) { |link, hash| hash[link] = nil }
+        links.index_with { nil }
       end
 
       # To be overriden. Useful for adding links that do not match with the filter.

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

@@ -4,23 +4,23 @@ module Decidim
   module Admin
     # This module includes helpers to manage menus in admin layout
     module MenuHelper
-      include Decidim::Admin::SidebarMenuHelper
-
       # Public: Returns the main menu presenter object
       def main_menu
         @main_menu ||= ::Decidim::MenuPresenter.new(
           :admin_menu,
           self,
-          active_class: "is-active"
+          active_class: "is-active",
+          label: t("layouts.decidim.header.main_menu")
         )
       end
 
-      def global_moderation_menu
-        @global_moderation_menu ||= sidebar_menu(:admin_global_moderation_menu)
+      def sidebar_menu(target_menu)
+        ::Decidim::Admin::SecondaryMenuPresenter.new(target_menu, self, active_class: "is-active")
       end
 
-      def organization_settings_menu
-        @organization_settings_menu ||= sidebar_menu(:admin_settings_menu)
+      def simple_menu(target_menu:, options: {})
+        options = { active_class: "is-active" }.merge(options)
+        ::Decidim::Admin::SimpleMenuPresenter.new(target_menu, self, options)
       end
     end
   end

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

@@ -12,7 +12,8 @@ module Decidim
       #
       # Returns an integer array with [width, height]
       def image_dimensions(model, image_name)
-        versions = model.send(image_name).dimensions_info
+        uploader = model.attached_uploader(image_name) || model.send(image_name)
+        versions = uploader.dimensions_info
         [:small, :medium, :default].map { |v| versions.dig(v, :dimensions) }.compact.first
       end
 

data/app/jobs/decidim/admin/expire_impersonation_job.rb

@@ -9,8 +9,7 @@ module Decidim
         impersonation_log = Decidim::ImpersonationLog.where(admin: current_user, user: user).active.first
         return unless impersonation_log
 
-        impersonation_log.expired_at = Time.current
-        impersonation_log.save!
+        impersonation_log.expire!
       end
     end
   end

data/app/jobs/decidim/admin/import_participatory_space_private_user_csv_job.rb

@@ -5,7 +5,7 @@ module Decidim
     # Custom ApplicationJob scoped to the admin panel.
     #
     class ImportParticipatorySpacePrivateUserCsvJob < ApplicationJob
-      queue_as :default
+      queue_as :exports
 
       def perform(email, user_name, privatable_to, current_user)
         return if email.blank? || user_name.blank?

data/app/packs/entrypoints/decidim_admin.js

@@ -0,0 +1,43 @@
+/* eslint no-unused-vars: 0 */
+/* eslint id-length: ["error", { "exceptions": ["$"] }] */
+
+import "core-js/stable";
+import "regenerator-runtime/runtime";
+import $ from "jquery"
+import Quill from "quill"
+import Rails from "@rails/ujs"
+
+import "foundation-sites"
+import "src/decidim/vendor/foundation-datepicker"
+import "src/decidim/foundation_datepicker_locales"
+import "jquery.autocomplete"
+import "jquery-serializejson"
+
+import "src/decidim/admin/tab_focus"
+import "src/decidim/admin/choose_language"
+import "src/decidim/admin/application"
+import "src/decidim/admin/resources_permissions"
+import "src/decidim/admin/welcome_notification"
+import "src/decidim/admin/newsletters"
+import "src/decidim/admin/form"
+import "src/decidim/admin/import_guidance"
+import "src/decidim/admin/external_domain_whitelist"
+import "src/decidim/confirm"
+import "src/decidim/admin/draggable-list"
+import "src/decidim/admin/sortable"
+import "src/decidim/gallery"
+import "src/decidim/admin/moderations"
+import "src/decidim/input_tags"
+import "src/decidim/input_hashtags"
+import "src/decidim/input_mentions"
+import "src/decidim/vizzs"
+import "src/decidim/ajax_modals"
+import "src/decidim/admin/officializations"
+import "src/decidim/session_timeouter"
+import "src/decidim/slug_form"
+
+// CSS
+import "entrypoints/decidim_admin.scss";
+
+// This needs to be loaded after confirm dialog to bind properly
+Rails.start()

data/app/packs/entrypoints/decidim_admin.scss

@@ -0,0 +1 @@
+@import "stylesheets/decidim/admin/application.scss"