decidim-assemblies 0.30.3 → 0.31.0.rc2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: baaa774b194d4c6251382233dde63bf1ef227a5c9eb7648b7b2e93073a22c9c5
-  data.tar.gz: 9612faadc7b6a04eb16800dc0acc7184c113bc9283c66c8dad2250d78222ab0d
+  metadata.gz: 5812b1925cb6b6ea4340092bb4f6375ff8faf33fb6dd65fa875faebf6d23fa68
+  data.tar.gz: 320223272a252555df9bfdbd64761a0fb97f4a3b99cdddeacc1b634f3a448aa1
 SHA512:
-  metadata.gz: 60b76b4d9fa4bfbe7eaf73508059ecfea11fed7dcd7acced98a07c53dd1921cbefcb8ac3606de47feee0a2bdb0a9c6b9cc84c2b51826fa82ede5077d38508c23
-  data.tar.gz: a7409abfe0185953e5410bc6ae9ce180343ea9c37a4003fa08ffe80f0cfb90baee6045a38fce3c0bc7dd546825059101f6c2991eb10a9edc15636582c4fea7d4
+  metadata.gz: 20a195fba6c8b3d809267b2c538f960ecf2795d7a7aa8a2fe317cc6baf362235b640015f549da7246e28899ad504cb9b897313bd2c5df731160a6c9668e470bd
+  data.tar.gz: 2f6334b646b80a359b1089a0666573c4a0895a0967d00aba1abfaafe801ae3c788f5f2f022133619527e1dd4f7a8b8e4d0b860cae999b509c1368e90f0fd2efc

data/app/cells/decidim/assemblies/content_blocks/highlighted_assemblies_settings_form/show.erb

@@ -1,3 +1,5 @@
 <% form.fields_for :settings, form.object.settings do |settings_fields| %>
-  <%= settings_fields.number_field :max_results, label: %>
+  <div class="row column">
+    <%= settings_fields.number_field :max_results, label: %>
+  </div>
 <% end %>

data/app/commands/decidim/assemblies/admin/create_assembly.rb

@@ -8,7 +8,7 @@ module Decidim
       class CreateAssembly < Decidim::Commands::CreateResource
         fetch_file_attributes :hero_image, :banner_image
 
-        fetch_form_attributes :title, :subtitle, :weight, :slug, :hashtag, :description, :short_description,
+        fetch_form_attributes :title, :subtitle, :weight, :slug, :description, :short_description,
                               :promoted, :taxonomizations, :parent, :announcement, :organization,
                               :private_space, :developer_group, :local_area, :target, :participatory_scope,
                               :participatory_structure, :meta_scope, :purpose_of_action,

data/app/commands/decidim/assemblies/admin/{copy_assembly.rb → duplicate_assembly.rb}

@@ -3,9 +3,9 @@
 module Decidim
   module Assemblies
     module Admin
-      # A command with all the business logic when copying a new participatory
+      # A command with all the business logic when duplicating a new participatory
       # assembly in the system.
-      class CopyAssembly < Decidim::Command
+      class DuplicateAssembly < Decidim::Command
         # Public: Initializes the command.
         #
         # form - A form object with the params.
@@ -27,27 +27,26 @@ module Decidim
 
           Decidim.traceability.perform_action!("duplicate", @assembly, @user) do
             Assembly.transaction do
-              copy_assembly
-              copy_assembly_attachments
-              copy_assembly_components if @form.copy_components?
-              copy_landing_page_blocks if @form.copy_landing_page_blocks?
+              duplicate_assembly
+              duplicate_assembly_attachments
+              duplicate_assembly_components if @form.duplicate_components?
+              duplicate_landing_page_blocks if @form.duplicate_landing_page_blocks?
             end
           end
 
-          broadcast(:ok, @copied_assembly)
+          broadcast(:ok, @duplicated_assembly)
         end
 
         private
 
         attr_reader :form
 
-        def copy_assembly
-          @copied_assembly = Assembly.create!(
+        def duplicate_assembly
+          @duplicated_assembly = Assembly.create!(
             organization: @assembly.organization,
             title: form.title,
             subtitle: @assembly.subtitle,
             slug: form.slug,
-            hashtag: @assembly.hashtag,
             description: @assembly.description,
             short_description: @assembly.short_description,
             promoted: @assembly.promoted,
@@ -64,47 +63,47 @@ module Decidim
           )
         end
 
-        def copy_assembly_attachments
+        def duplicate_assembly_attachments
           [:hero_image, :banner_image].each do |attribute|
             next unless @assembly.attached_uploader(attribute).attached?
 
-            @copied_assembly.send(attribute).attach(@assembly.send(attribute).blob)
+            @duplicated_assembly.send(attribute).attach(@assembly.send(attribute).blob)
           end
         end
 
-        def copy_assembly_components
+        def duplicate_assembly_components
           @assembly.components.each do |component|
             new_component = Component.create!(
               manifest_name: component.manifest_name,
               name: component.name,
-              participatory_space: @copied_assembly,
+              participatory_space: @duplicated_assembly,
               settings: component.settings,
               step_settings: component.step_settings,
               weight: component.weight
             )
-            component.manifest.run_hooks(:copy, new_component:, old_component: component)
+            component.manifest.run_hooks(:duplicate, new_component:, old_component: component)
           end
         end
 
-        def copy_landing_page_blocks
+        def duplicate_landing_page_blocks
           blocks = Decidim::ContentBlock.where(scoped_resource_id: @assembly.id, scope_name: "assembly_homepage", organization: @assembly.organization)
           return if blocks.blank?
 
           blocks.each do |block|
             new_block = Decidim::ContentBlock.create!(
-              organization: @copied_assembly.organization,
+              organization: @duplicated_assembly.organization,
               scope_name: "assembly_homepage",
-              scoped_resource_id: @copied_assembly.id,
+              scoped_resource_id: @duplicated_assembly.id,
               manifest_name: block.manifest_name,
               settings: block.settings,
               weight: block.weight,
-              published_at: block.published_at.present? ? @copied_assembly.created_at : nil # determine if block is active/inactive
+              published_at: block.published_at.present? ? @duplicated_assembly.created_at : nil # determine if block is active/inactive
             )
-            copy_block_attachments(block, new_block) if block.attachments.present?
+            duplicate_block_attachments(block, new_block) if block.attachments.present?
           end
         end
 
-        def copy_block_attachments(block, new_block)
+        def duplicate_block_attachments(block, new_block)
           block.attachments.map(&:name).each do |name|
             original_image = block.images_container.send(name).blob
             next if original_image.blank?

data/app/commands/decidim/assemblies/admin/update_assembly.rb

@@ -8,7 +8,7 @@ module Decidim
       class UpdateAssembly < Decidim::Commands::UpdateResource
         fetch_file_attributes :hero_image, :banner_image
 
-        fetch_form_attributes :title, :subtitle, :slug, :hashtag, :promoted, :description, :short_description,
+        fetch_form_attributes :title, :subtitle, :slug, :promoted, :description, :short_description,
                               :taxonomizations, :parent, :private_space, :developer_group, :local_area,
                               :target, :participatory_scope, :participatory_structure, :meta_scope,
                               :purpose_of_action, :composition, :creation_date, :created_by,

data/app/controllers/decidim/assemblies/admin/assemblies_controller.rb

@@ -37,7 +37,7 @@ module Decidim
 
             on(:invalid) do
               flash.now[:alert] = I18n.t("assemblies.create.error", scope: "decidim.admin")
-              render :new
+              render :new, status: :unprocessable_entity
             end
           end
         end
@@ -63,12 +63,12 @@ module Decidim
 
             on(:invalid) do
               flash.now[:alert] = I18n.t("assemblies.update.error", scope: "decidim.admin")
-              render :edit, layout: "decidim/admin/assembly"
+              render :edit, layout: "decidim/admin/assembly", status: :unprocessable_entity
             end
           end
         end
 
-        def copy
+        def duplicate
           enforce_permission_to :create, :assembly, assembly: collection.find_by(id: params[:parent_id])
         end
 

data/app/controllers/decidim/assemblies/admin/assembly_duplicates_controller.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Assemblies
+    module Admin
+      # Controller that allows managing assemblies.
+      #
+      class AssemblyDuplicatesController < Decidim::Assemblies::Admin::ApplicationController
+        include Concerns::AssemblyAdmin
+
+        def new
+          enforce_permission_to :create, :assembly
+          @form = form(AssemblyDuplicateForm).from_model(current_assembly)
+        end
+
+        def create
+          enforce_permission_to :create, :assembly
+          @form = form(AssemblyDuplicateForm).from_params(params)
+
+          DuplicateAssembly.call(@form, current_assembly, current_user) do
+            on(:ok) do
+              flash[:notice] = I18n.t("assemblies_duplicates.create.success", scope: "decidim.admin")
+              redirect_to assemblies_path
+            end
+
+            on(:invalid) do
+              flash.now[:alert] = I18n.t("assemblies_duplicates.create.error", scope: "decidim.admin")
+              render :new, status: :unprocessable_entity
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/app/controllers/decidim/assemblies/admin/assembly_imports_controller.rb

@@ -23,7 +23,7 @@ module Decidim
 
             on(:invalid) do
               flash.now[:alert] = I18n.t("assembly_imports.create.error", scope: "decidim.admin")
-              render :new
+              render :new, status: :unprocessable_entity
             end
           end
         end

data/app/controllers/decidim/assemblies/admin/assembly_landing_page_controller.rb

@@ -31,6 +31,10 @@ module Decidim
         def resource_content_block_cell
           "decidim/assemblies/content_block"
         end
+
+        def content_blocks_title
+          t("decidim.admin.menu.assemblies_submenu.landing_page")
+        end
       end
     end
   end

data/app/forms/decidim/assemblies/admin/{assembly_copy_form.rb → assembly_duplicate_form.rb}

@@ -3,10 +3,10 @@
 module Decidim
   module Assemblies
     module Admin
-      # A form object used to copy a assemblies from the admin
+      # A form object used to duplicate a assemblies from the admin
       # dashboard.
       #
-      class AssemblyCopyForm < Form
+      class AssemblyDuplicateForm < Form
         include TranslatableAttributes
 
         translatable_attribute :title, String
@@ -14,8 +14,8 @@ module Decidim
         mimic :assembly
 
         attribute :slug, String
-        attribute :copy_components, Boolean
-        attribute :copy_landing_page_blocks, Boolean
+        attribute :duplicate_components, Boolean
+        attribute :duplicate_landing_page_blocks, Boolean
 
         validates :slug, presence: true, format: { with: Decidim::Assembly.slug_format }
         validates :title, translatable_presence: true

data/app/forms/decidim/assemblies/admin/assembly_form.rb

@@ -36,7 +36,6 @@ module Decidim
         attribute :created_by, String
         attribute :facebook_handler, String
         attribute :github_handler, String
-        attribute :hashtag, String
         attribute :instagram_handler, String
         attribute :slug, String
         attribute :twitter_handler, String

data/app/helpers/decidim/assemblies/filter_assemblies_helper.rb

@@ -13,7 +13,7 @@ module Decidim
           items.append(method: "with_any_taxonomies[#{taxonomy_filter.root_taxonomy_id}]",
                        collection: filter_taxonomy_values_for(taxonomy_filter),
                        label: decidim_sanitize_translated(taxonomy_filter.name),
-                       id: "taxonomy")
+                       id: "taxonomy-#{taxonomy_filter.root_taxonomy_id}")
         end
 
         items.reject { |item| item[:collection].blank? }

data/app/models/decidim/assembly.rb

@@ -123,10 +123,6 @@ module Decidim
       published? && (!private_space? || (private_space? && is_transparent?))
     end
 
-    def hashtag
-      attributes["hashtag"].to_s.delete("#")
-    end
-
     def to_param
       slug
     end

data/app/packs/entrypoints/decidim_assemblies_admin.js

@@ -1,2 +1 @@
 import "src/decidim/assemblies/admin/assemblies"
-import "src/decidim/slug_form"

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

@@ -1,63 +1,4 @@
-$(() => {
-  const $assemblyScopeEnabled = $("#assembly_scopes_enabled");
-  const $assemblyScopeId = $("#assembly_scope_id");
+import { definitionsFromContext } from "src/decidim/refactor/support/stimulus"
 
-  if ($(".edit_assembly, .new_assembly").length > 0) {
-    $assemblyScopeEnabled.on("change", (event) => {
-      const checked = event.target.checked;
-      $assemblyScopeId.attr("disabled", !checked);
-    })
-    $assemblyScopeId.attr("disabled", !$assemblyScopeEnabled.prop("checked"));
-  }
-
-  const $form = $(".assembly_form_admin");
-
-  if ($form.length > 0) {
-
-    const $privateSpace = $form.find("#private_space");
-    const $isTransparent = $form.find("#is_transparent");
-    const $specialFeatures = $form.find("#special_features");
-
-
-    const toggleDisabledHiddenFields = () => {
-      const enabledPrivateSpace = $privateSpace.find("input[type='checkbox']").prop("checked");
-      $isTransparent.find("input[type='checkbox']").attr("disabled", "disabled");
-      $specialFeatures.hide();
-
-      if (enabledPrivateSpace) {
-        $isTransparent.find("input[type='checkbox']").attr("disabled", !enabledPrivateSpace);
-        $specialFeatures.show();
-      }
-    };
-
-    $privateSpace.on("change", toggleDisabledHiddenFields);
-    toggleDisabledHiddenFields();
-
-    const $assemblyType = $form.find("#assembly_assembly_type");
-    const $assemblyTypeOther = $form.find("#assembly_type_other");
-
-    const $assemblyCreatedBy = $form.find("#assembly_created_by");
-    const $assemblyCreatedByOther = $form.find("#created_by_other");
-
-    const toggleDependsOnSelect = ($target, $showDiv) => {
-      const value = $target.val();
-      $showDiv.hide();
-      if (value === "others") {
-        $showDiv.show();
-      }
-    };
-
-    $assemblyType.on("change", (ev) => {
-      const $target = $(ev.target);
-      toggleDependsOnSelect($target, $assemblyTypeOther);
-    });
-
-    $assemblyCreatedBy.on("change", (ev) => {
-      const $target = $(ev.target);
-      toggleDependsOnSelect($target, $assemblyCreatedByOther);
-    });
-
-    toggleDependsOnSelect($assemblyType, $assemblyTypeOther);
-    toggleDependsOnSelect($assemblyCreatedBy, $assemblyCreatedByOther);
-  }
-});
+const context = require.context("../controllers", true, /controller\.js$/)
+window.Stimulus.load(definitionsFromContext(context))