decidim-admin 0.28.5 → 0.29.0.rc1

data/app/controllers/decidim/admin/{organization_external_domain_whitelist_controller.rb → organization_external_domain_allowlist_controller.rb}

@@ -2,7 +2,7 @@
 
 module Decidim
   module Admin
-    class OrganizationExternalDomainWhitelistController < Decidim::Admin::ApplicationController
+    class OrganizationExternalDomainAllowlistController < Decidim::Admin::ApplicationController
       layout "decidim/admin/settings"
 
       add_breadcrumb_item_from_menu :admin_settings_menu
@@ -11,20 +11,20 @@ module Decidim
 
       def edit
         enforce_permission_to :update, :organization, organization: current_organization
-        @form = form(OrganizationExternalDomainWhitelistForm).from_model(current_organization)
+        @form = form(OrganizationExternalDomainAllowlistForm).from_model(current_organization)
       end
 
       def update
         enforce_permission_to :update, :organization, organization: current_organization
-        @form = form(OrganizationExternalDomainWhitelistForm).from_params(params)
+        @form = form(OrganizationExternalDomainAllowlistForm).from_params(params)
 
-        UpdateExternalDomainWhitelist.call(@form, current_organization, current_user) do
+        UpdateExternalDomainAllowlist.call(@form, current_organization, current_user) do
           on(:ok) do
-            flash[:notice] = t("domain_whitelist.update.success", scope: "decidim.admin")
-            redirect_to edit_organization_external_domain_whitelist_path
+            flash[:notice] = t("domain_allowlist.update.success", scope: "decidim.admin")
+            redirect_to edit_organization_external_domain_allowlist_path
           end
           on(:invalid) do
-            flash[:notice] = t("domain_whitelist.update.error", scope: "decidim.admin")
+            flash[:notice] = t("domain_allowlist.update.error", scope: "decidim.admin")
             render action: "edit"
           end
         end

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

@@ -26,7 +26,7 @@ module Decidim
         enforce_permission_to :create, :scope_type
         @form = form(ScopeTypeForm).from_params(params)
 
-        CreateScopeType.call(@form, current_user) do
+        CreateScopeType.call(@form) do
           on(:ok) do
             flash[:notice] = I18n.t("scope_types.create.success", scope: "decidim.admin")
             redirect_to scope_types_path
@@ -48,7 +48,7 @@ module Decidim
         enforce_permission_to(:update, :scope_type, scope_type:)
         @form = form(ScopeTypeForm).from_params(params)
 
-        UpdateScopeType.call(scope_type, @form, current_user) do
+        UpdateScopeType.call(@form, scope_type) do
           on(:ok) do
             flash[:notice] = I18n.t("scope_types.update.success", scope: "decidim.admin")
             redirect_to scope_types_path
@@ -64,13 +64,12 @@ module Decidim
       def destroy
         enforce_permission_to(:destroy, :scope_type, scope_type:)
 
-        Decidim.traceability.perform_action!("delete", scope_type, current_user) do
-          scope_type.destroy!
+        Decidim::Commands::DestroyResource.call(scope_type, current_user) do
+          on(:ok) do
+            flash[:notice] = I18n.t("scope_types.destroy.success", scope: "decidim.admin")
+            redirect_to scope_types_path
+          end
         end
-
-        flash[:notice] = I18n.t("scope_types.destroy.success", scope: "decidim.admin")
-
-        redirect_to scope_types_path
       end
 
       private

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

@@ -49,7 +49,7 @@ module Decidim
         enforce_permission_to(:update, :scope, scope:)
         @form = form(ScopeForm).from_params(params)
 
-        UpdateScope.call(scope, @form) do
+        UpdateScope.call(@form, scope) do
           on(:ok) do
             flash[:notice] = I18n.t("scopes.update.success", scope: "decidim.admin")
             redirect_to current_scopes_path

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

@@ -6,7 +6,7 @@ module Decidim
       def destroy
         enforce_permission_to(:destroy, :share_token, share_token:)
 
-        DestroyShareToken.call(share_token, current_user) do
+        Decidim::Commands::DestroyResource.call(share_token, current_user) do
           on(:ok) do
             flash[:notice] = I18n.t("share_tokens.destroy.success", scope: "decidim.admin")
           end

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

@@ -44,7 +44,7 @@ module Decidim
         enforce_permission_to :update, :static_page_topic, static_page_topic: topic
         @form = form(StaticPageTopicForm).from_params(params["static_page_topic"])
 
-        UpdateStaticPageTopic.call(topic, @form) do
+        UpdateStaticPageTopic.call(@form, topic) do
           on(:ok) do
             flash[:notice] = I18n.t("static_page_topics.update.success", scope: "decidim.admin")
             redirect_to static_page_topics_path
@@ -60,7 +60,7 @@ module Decidim
       def destroy
         enforce_permission_to :destroy, :static_page_topic, static_page_topic: topic
 
-        DestroyStaticPageTopic.call(topic, current_user) do
+        Decidim::Commands::DestroyResource.call(topic, current_user) do
           on(:ok) do
             flash[:notice] = I18n.t("static_page_topics.destroy.success", scope: "decidim.admin")
             redirect_to static_page_topics_path

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

@@ -77,7 +77,7 @@ module Decidim
         enforce_permission_to :update, :static_page, static_page: page
         @form = form(StaticPageForm).from_params(form_params)
 
-        UpdateStaticPage.call(page, @form) do
+        UpdateStaticPage.call(@form, page) do
           on(:ok) do
             flash[:notice] = I18n.t("static_pages.update.success", scope: "decidim.admin")
             redirect_to static_pages_path
@@ -93,7 +93,7 @@ module Decidim
       def destroy
         enforce_permission_to :destroy, :static_page, static_page: page
 
-        DestroyStaticPage.call(page, current_user) do
+        Decidim::Commands::DestroyResource.call(page, current_user) do
           on(:ok) do
             flash[:notice] = I18n.t("static_pages.destroy.success", scope: "decidim.admin")
             redirect_to static_pages_path

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

@@ -12,10 +12,12 @@ module Decidim
       translatable_attribute :description, String
       attribute :weight, Integer, default: 0
       attribute :attachment_collection_id, Integer
+      attribute :link, String
 
       mimic :attachment
 
-      validates :file, presence: true, unless: :persisted?
+      validates :file, presence: true, unless: :persisted_or_link?
+      validates :link, url: true
       validates :file, passthru: { to: Decidim::Attachment }
       validates :title, :description, translatable_presence: true
       validates :attachment_collection, presence: true, if: ->(form) { form.attachment_collection_id.present? }
@@ -25,6 +27,10 @@ module Decidim
 
       alias organization current_organization
 
+      def persisted_or_link?
+        persisted? || link.present?
+      end
+
       def attachment_collections
         @attachment_collections ||= attached_to.attachment_collections
       end

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

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

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

@@ -14,7 +14,6 @@ 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
@@ -22,10 +21,6 @@ module Decidim
       def managed_user?
         user && user.managed?
       end
-
-      def persisted_user?
-        user&.persisted?
-      end
     end
   end
 end

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

@@ -30,14 +30,11 @@ module Decidim
       attribute :primary_color, String
       attribute :secondary_color, String
       attribute :tertiary_color, String
-      attribute :success_color, String
-      attribute :warning_color, String
-      attribute :alert_color, String
 
       translatable_attribute :cta_button_text, String
-      translatable_attribute :description, Decidim::Attributes::RichText
+      translatable_attribute :description, String
       translatable_attribute :highlighted_content_banner_title, String
-      translatable_attribute :highlighted_content_banner_short_description, Decidim::Attributes::RichText
+      translatable_attribute :highlighted_content_banner_short_description, String
       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_external_domain_whitelist_form.rb → organization_external_domain_allowlist_form.rb}

@@ -2,20 +2,20 @@
 
 module Decidim
   module Admin
-    class OrganizationExternalDomainWhitelistForm < Form
+    class OrganizationExternalDomainAllowlistForm < 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|
+        self.external_domains = model.external_domain_allowlist.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
+          errors.add(:external_domains, I18n.t("decidim.admin.domain_allowlist.form.domain_too_short")) if domain.value.length <= 3
         end.map(&:value).uniq
       end
     end

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

@@ -10,7 +10,7 @@ module Decidim
 
       mimic :organization
 
-      attribute :name, String
+      translatable_attribute :name, String
       attribute :reference_prefix, String
       attribute :time_zone, String
       attribute :twitter_handler, String
@@ -31,13 +31,13 @@ module Decidim
       attribute :customize_welcome_notification, Boolean
 
       translatable_attribute :welcome_notification_subject, String
-      translatable_attribute :welcome_notification_body, Decidim::Attributes::RichText
+      translatable_attribute :welcome_notification_body, String
 
-      translatable_attribute :admin_terms_of_service_body, Decidim::Attributes::RichText
+      translatable_attribute :admin_terms_of_service_body, String
 
       validates :welcome_notification_subject, :welcome_notification_body, translatable_presence: true, if: proc { |form| form.customize_welcome_notification }
 
-      validates :name, presence: true
+      validates :name, translatable_presence: true
       validates :time_zone, presence: true
       validates :time_zone, time_zone: true
       validates :default_locale, :reference_prefix, presence: true

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

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

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

@@ -14,21 +14,6 @@ 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

@@ -16,19 +16,6 @@ module Decidim
       include Decidim::Admin::ResourceScopeHelper
       include Decidim::Admin::SearchFormHelper
 
-      # Public: Overwrites the `cell` helper method to automatically set some
-      # common context.
-      #
-      # name - the name of the cell to render
-      # model - the cell model
-      # options - a Hash with options
-      #
-      # Renders the cell contents.
-      def cell(name, model, options = {}, &)
-        options = { context: { view_context: self, current_user: } }.deep_merge(options)
-        super
-      end
-
       def participatory_space_active_link?(component)
         endpoints = component.manifest.admin_engine.try(:participatory_space_endpoints)
         endpoints && is_active_link?(decidim_admin_participatory_processes.components_path(current_participatory_space), %r{/\d+/manage/(#{endpoints.join("|")})\b})

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

@@ -32,7 +32,7 @@ module Decidim
         links.index_with { nil }
       end
 
-      # To be overriden. Useful for adding links that do not match with the filter.
+      # To be overridden. Useful for adding links that do not match with the filter.
       # Must return an Array.
       def extra_dropdown_submenu_options_items(_filter, _i18n_scope)
         []
@@ -98,9 +98,14 @@ module Decidim
       end
 
       def applied_filters_tags(i18n_ctx)
-        ransack_params.slice(*filters).map do |filter, value|
+        tags = ransack_params.slice(*filters).map do |filter, value|
           applied_filter_tag(filter, value, filterable_i18n_scope_from_ctx(i18n_ctx))
-        end.join.html_safe
+        end
+        return if tags.blank?
+
+        tags << remove_all_filters_tag if tags.count > 1
+
+        tags.join.html_safe
       end
 
       def applied_filter_tag(filter, value, i18n_scope)
@@ -111,6 +116,13 @@ module Decidim
         end
       end
 
+      def remove_all_filters_tag
+        link_to(url_for(blank_query_params), class: "label bg-transparent") do
+          concat t("decidim.admin.filters.remove_all")
+          concat icon("delete-bin-line", aria_label: t("decidim.admin.filters.remove_all"), role: "img")
+        end
+      end
+
       def remove_filter_icon_link(filter)
         icon_link_to(
           "delete-bin-line",
@@ -125,6 +137,10 @@ module Decidim
         i18n_scope += ".#{i18n_ctx}" if i18n_ctx
         i18n_scope
       end
+
+      def filtered_adjacent_paths(item, path_method)
+        adjacent_items(item).transform_values(&method(path_method))
+      end
     end
   end
 end

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

@@ -25,9 +25,11 @@ module Decidim
               when Decidim::Meetings::Meeting
                 content_tag :li do
                   link_to resource_locator(author).path, target: "_blank", rel: "noopener" do
-                    decidim_sanitize(translated_attribute(author.title))
+                    decidim_sanitize_translated(author.title)
                   end
                 end
+              when Decidim::Organization
+                content_tag :li, organization_name(author)
               else
                 content_tag(:li, author.name)
               end

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

@@ -65,7 +65,7 @@ module Decidim
 
       def sent_to_spaces(newsletter)
         html = "<p style='margin-bottom:0;'> "
-        newsletter.sended_to_partipatory_spaces.try(:each) do |type|
+        newsletter.sent_to_participatory_spaces.try(:each) do |type|
           next if type["ids"].blank?
 
           html += t("index.segmented_to", scope: "decidim.admin.newsletters", subject: t("activerecord.models.decidim/#{type["manifest_name"].singularize}.other"))
@@ -74,7 +74,7 @@ module Decidim
           else
             Decidim.find_participatory_space_manifest(type["manifest_name"].to_sym)
                    .participatory_spaces.call(current_organization).where(id: type["ids"]).each do |space|
-              html += "<strong>#{decidim_html_escape(translated_attribute(space.title))}</strong>"
+              html += "<strong>#{decidim_escape_translated(space.title)}</strong>"
             end
           end
           html += "<br/>"
@@ -88,7 +88,7 @@ module Decidim
           concat t("index.segmented_to", scope: "decidim.admin.newsletters", subject: nil)
           if newsletter.sent_scopes.any?
             newsletter.sent_scopes.each do |scope|
-              concat content_tag(:strong, decidim_html_escape(translated_attribute(scope.name)).to_s)
+              concat content_tag(:strong, decidim_escape_translated(scope.name).to_s)
             end
           else
             concat content_tag(:strong, t("index.no_scopes", scope: "decidim.admin.newsletters"))

data/app/helpers/decidim/admin/paginable/per_page_helper.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Admin
+    module Paginable
+      # This module includes helpers the :per_page cell's option
+      module PerPageHelper
+        def per_page_options
+          OpenStruct.new(
+            per_page:,
+            per_page_range: Decidim::Admin.per_page_range
+          )
+        end
+
+        # Renders the pagination dropdown menu in the admin panel.
+        def admin_filters_pagination
+          cell("decidim/admin/results_per_page", per_page_options)
+        end
+      end
+    end
+  end
+end

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

@@ -17,10 +17,10 @@ module Decidim
         private_user_form = ParticipatorySpacePrivateUserForm.from_params(params, privatable_to:)
                                                              .with_context(
                                                                current_user:,
-                                                               current_particiaptory_space: privatable_to
+                                                               current_participatory_space: privatable_to
                                                              )
 
-        Decidim::Admin::CreateParticipatorySpacePrivateUser.call(private_user_form, current_user, privatable_to, via_csv: true)
+        Decidim::Admin::CreateParticipatorySpacePrivateUser.call(private_user_form, privatable_to, via_csv: true)
       end
     end
   end

data/app/packs/entrypoints/decidim_admin.js

@@ -8,7 +8,7 @@ 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/external_domain_whitelist"
+import "src/decidim/admin/external_domain_allowlist"
 import "src/decidim/admin/draggable-list"
 import "src/decidim/admin/sortable"
 import "src/decidim/admin/moderations"
@@ -17,6 +17,9 @@ import "src/decidim/slug_form"
 import "src/decidim/admin/admin_autocomplete"
 import "src/decidim/admin/triadic_color_picker"
 import "src/decidim/admin/participatory_space_search"
+import "src/decidim/admin/css_preview"
+import "src/decidim/admin/sync_radio_buttons"
+import "src/decidim/admin/text_copy"
 
 // CSS
 import "entrypoints/decidim_admin.scss";

data/app/packs/src/decidim/admin/application.js

@@ -1,15 +1,15 @@
 /* eslint-disable no-invalid-this */
 
-import toggleNav from "src/decidim/admin/toggle_nav"
-import createSortList from "src/decidim/admin/sort_list.component"
-import FormFilterComponent from "src/decidim/form_filter"
-import Configuration from "src/decidim/configuration"
-import InputCharacterCounter from "src/decidim/input_character_counter"
-import managedUsersForm from "src/decidim/admin/managed_users"
+import toggleNav from "src/decidim/admin/toggle_nav";
+import createSortList from "src/decidim/admin/sort_list.component";
+import FormFilterComponent from "src/decidim/form_filter";
+import Configuration from "src/decidim/configuration";
+import InputCharacterCounter from "src/decidim/input_character_counter";
+import managedUsersForm from "src/decidim/admin/managed_users";
 
 window.Decidim = window.Decidim || {};
-window.Decidim.managedUsersForm = managedUsersForm
-window.Decidim.config = new Configuration()
+window.Decidim.managedUsersForm = managedUsersForm;
+window.Decidim.config = new Configuration();
 window.Decidim.InputCharacterCounter = InputCharacterCounter;
 
 // REDESIGN_PENDING: deprecated
@@ -31,23 +31,27 @@ $(() => {
   toggleNav();
 
   createSortList("#steps tbody", {
-    placeholder: $('<tr style="border-style: dashed; border-color: #000"><td colspan="4">&nbsp;</td></tr>')[0],
+    placeholder: $(
+      '<tr style="border-style: dashed; border-color: #000"><td colspan="4">&nbsp;</td></tr>'
+    )[0],
     onSortUpdate: ($children) => {
-      const sortUrl = $("#steps tbody").data("sort-url")
-      const order = $children.map((index, child) => $(child).data("id")).toArray();
+      const sortUrl = $("#steps tbody").data("sort-url");
+      const order = $children.
+        map((index, child) => $(child).data("id")).
+        toArray();
 
       $.ajax({
         method: "POST",
         url: sortUrl,
         contentType: "application/json",
-        data: JSON.stringify({ items_ids: order }) }, // eslint-disable-line camelcase
-      );
+        data: JSON.stringify({ items_ids: order }) // eslint-disable-line camelcase
+      });
     }
-  })
+  });
 
   $("form.new_filter").each(function () {
     const formFilter = new FormFilterComponent($(this));
 
     formFilter.mountComponent();
-  })
+  });
 });

data/app/packs/src/decidim/admin/css_preview.js

@@ -0,0 +1,39 @@
+// Preview CSS changes in real time through data attributes when the change event is dispatched.
+//
+// @example
+// ```erb
+// <%= form.radio_button :text_color, "Blue"
+//                       data: {
+//                         "css-preview" => true,
+//                         "css-preview-updates" => "[data-css-example]:color:blue;"
+//                       } %>
+//
+// <%= form.radio_button :text_color, "Red"
+//                       data: {
+//                         "css-preview" => true,
+//                         "css-preview-updates" => "[data-css-example]:color:red;"
+//                       } %>
+//
+// <div data-css-example>This is an example</div>
+// ```
+//
+// It supports multiple rules separated by semicolons:
+//   selector:property:value; selector:property:value;
+//
+// for example:
+//   strong[data-css-example]:color:#0000ff; strong[data-css-example]:backgroundColor:#eeeeee;
+//
+window.addEventListener("DOMContentLoaded", () => {
+  document.querySelectorAll("input[data-css-preview=true]").forEach((element) => {
+    element.addEventListener("change", (event) => {
+      const updateRules = event.target.dataset.cssPreviewUpdates.split(";");
+
+      updateRules.forEach((rule) => {
+        const [target, property, value] = rule.split(":");
+        if (target !== "") {
+          document.querySelector(target).style[property.trim()] = value.trim();
+        }
+      });
+    })
+  })
+})

data/app/packs/src/decidim/admin/form.js

@@ -28,7 +28,7 @@ $(() => {
   // - amendment_promotion_enabled
 
   // (1) Hides target fields if amendments_enabled component setting is NOT checked.
-  // (2) Toggles visibilty of target fields when amendments_enabled component setting is clicked.
+  // (2) Toggles visibility of target fields when amendments_enabled component setting is clicked.
   const $amendmentsEnabled = $("input#component_settings_amendments_enabled");
 
   if ($amendmentsEnabled.length > 0) {
@@ -42,4 +42,21 @@ $(() => {
       $amendmentStepSettings.toggle();
     });
   }
+
+  // Toggles visibility of the attachments_allowed help text when the checkbox is clicked.
+  // Show the help text if the checkbox is checked.
+  const $attachmentsAllowedCheckbox = $("#component_settings_attachments_allowed");
+  const $attachmentsHelpText = $(".attachments_allowed_container .help-text");
+
+  if (!$attachmentsAllowedCheckbox.is(":checked")) {
+    $attachmentsHelpText.hide();
+  }
+
+  $attachmentsAllowedCheckbox.on("change", () => {
+    if ($attachmentsAllowedCheckbox.is(":checked")) {
+      $attachmentsHelpText.show();
+    } else {
+      $attachmentsHelpText.hide();
+    }
+  });
 });

data/app/packs/src/decidim/admin/newsletters.js

@@ -54,9 +54,9 @@ $(() => {
       $participatorySpacesForSelect.hide();
     }
 
-    $(".form .spaces-block-tag").each(function (_i, blockTag) {
+    $(".form .spaces-block-tag").each(function(_i, blockTag) {
       const selectTag = $(blockTag).find(".chosen-select")
-      selectTag.change(function () {
+      selectTag.change(function() {
         let optionSelected = selectTag.find("option:selected").val()
         if (optionSelected === "all") {
           selectTag.find("option").not(":first").prop("selected", true);
@@ -67,25 +67,16 @@ $(() => {
       });
     })
 
-    $form.on("change", function(event) {
-      let formData = new FormData(event.target.closest("form"));
-      let url = $form.data("recipients-count-newsletter-path");
+    $form.on("change", function() {
+      let $data = $form.serializeJSON().newsletter;
+      let $url = $form.data("recipients-count-newsletter-path");
       const $modal = $("#recipients_count_spinner");
       $modal.removeClass("hide");
-
-      const xhr = new XMLHttpRequest();
-      xhr.open("POST", url, true);
-      xhr.onload = function() {
-        if (xhr.status === 200) {
-          $("#recipients_count").text(xhr.responseText);
-        }
+      $.get($url, {data: $data}, function(recipientsCount) {
+        $("#recipients_count").text(recipientsCount);
+      }).always(function() {
         $modal.addClass("hide");
-      };
-      xhr.onerror = function() {
-        $modal.addClass("hide");
-      };
-      // Send the form data
-      xhr.send(formData);
+      });
     })
   }
 });

data/app/packs/src/decidim/admin/participatory_space_search.js

@@ -25,7 +25,7 @@ const apiRequest = (query) => {
 /**
  * Resolves the different root fields for listing different participatory spaces
  * available in the instance. Returns all LIST kind root fields that list
- * records implemeting the ParticipatorySpaceInterface.
+ * records implementing the ParticipatorySpaceInterface.
  *
  * @returns {Promise} A promise resolving the root fields for querying different
  *   participatory spaces.