update_assembly.rb in decidim-assemblies-0.29.0.rc1

- old
+ new
@@ -1,126 +1,49 @@
 # frozen_string_literal: true
 
 module Decidim
   module Assemblies
     module Admin
-      # A command with all the business logic when creating a new participatory
-      # assembly in the system.
-      class UpdateAssembly < Decidim::Command
-        include ::Decidim::AttachmentAttributesMethods
+      # A command with all the business logic when updating a new assembly
+      # in the system.
+      class UpdateAssembly < Decidim::Commands::UpdateResource
+        fetch_file_attributes :hero_image, :banner_image
 
-        # Public: Initializes the command.
-        #
-        # assembly - the Assembly to update
-        # form - A form object with the params.
-        def initialize(assembly, form)
-          @assembly = assembly
-          @form = form
-          @parent = Assembly.find_by(id: @assembly.parent)
-        end
+        fetch_form_attributes :title, :subtitle, :slug, :hashtag, :promoted, :description, :short_description,
+                              :scopes_enabled, :scope, :area, :parent, :private_space, :developer_group, :local_area,
+                              :target, :participatory_scope, :participatory_structure, :meta_scope,
+                              :purpose_of_action, :composition, :assembly_type, :creation_date, :created_by,
+                              :created_by_other, :duration, :included_at, :closing_date, :closing_date_reason,
+                              :internal_organisation, :is_transparent, :special_features, :twitter_handler, :announcement,
+                              :facebook_handler, :instagram_handler, :youtube_handler, :github_handler, :weight
 
-        # Executes the command. Broadcasts these events:
-        #
-        # - :ok when everything is valid.
-        # - :invalid if the form was not valid and we could not proceed.
-        #
-        # Returns nothing.
-        def call
-          return broadcast(:invalid) if form.invalid?
-
-          update_assembly
-          link_participatory_processes(@assembly)
-
-          if @assembly.valid?
-            broadcast(:ok, @assembly)
-          else
-            form.errors.add(:hero_image, @assembly.errors[:hero_image]) if @assembly.errors.include? :hero_image
-            form.errors.add(:banner_image, @assembly.errors[:banner_image]) if @assembly.errors.include? :banner_image
-            broadcast(:invalid)
-          end
-        end
-
         private
 
-        attr_reader :form, :assembly
-
-        def update_assembly
-          @assembly.assign_attributes(attributes)
-          save_assembly if @assembly.valid?
+        def run_after_hooks
+          link_participatory_processes
           update_children_count
         end
 
-        def save_assembly
-          transaction do
-            @assembly.save!
-            Decidim.traceability.perform_action!(:update, @assembly, form.current_user) do
-              @assembly
-            end
-          end
+        def parent
+          @parent ||= Assembly.find_by(id: resource.parent)
         end
 
-        def attributes
-          {
-            title: form.title,
-            subtitle: form.subtitle,
-            slug: form.slug,
-            hashtag: form.hashtag,
-            promoted: form.promoted,
-            description: form.description,
-            short_description: form.short_description,
-            scopes_enabled: form.scopes_enabled,
-            scope: form.scope,
-            area: form.area,
-            parent: form.parent,
-            private_space: form.private_space,
-            developer_group: form.developer_group,
-            local_area: form.local_area,
-            target: form.target,
-            participatory_scope: form.participatory_scope,
-            participatory_structure: form.participatory_structure,
-            meta_scope: form.meta_scope,
-            show_statistics: form.show_statistics,
-            purpose_of_action: form.purpose_of_action,
-            composition: form.composition,
-            assembly_type: form.assembly_type,
-            creation_date: form.creation_date,
-            created_by: form.created_by,
-            created_by_other: form.created_by_other,
-            duration: form.duration,
-            included_at: form.included_at,
-            closing_date: form.closing_date,
-            closing_date_reason: form.closing_date_reason,
-            internal_organisation: form.internal_organisation,
-            is_transparent: form.is_transparent,
-            special_features: form.special_features,
-            twitter_handler: form.twitter_handler,
-            facebook_handler: form.facebook_handler,
-            instagram_handler: form.instagram_handler,
-            youtube_handler: form.youtube_handler,
-            github_handler: form.github_handler,
-            weight: form.weight,
-            announcement: form.announcement
-          }.merge(
-            attachment_attributes(:hero_image, :banner_image)
-          )
+        def participatory_processes
+          @participatory_processes ||= resource.participatory_space_sibling_scope(:participatory_processes).where(id: form.participatory_processes_ids)
         end
 
-        def participatory_processes(assembly)
-          @participatory_processes ||= assembly.participatory_space_sibling_scope(:participatory_processes).where(id: @form.participatory_processes_ids)
+        def link_participatory_processes
+          resource.link_participatory_space_resources(participatory_processes, "included_participatory_processes")
         end
 
-        def link_participatory_processes(assembly)
-          assembly.link_participatory_space_resources(participatory_processes(assembly), "included_participatory_processes")
-        end
-
         # Resets the children counter cache to its correct value using an SQL count query.
         # Fixes Rails decrementing twice error when updating the parent to nil.
         #
         # Returns nothing.
         def update_children_count
-          return unless @parent
+          return unless parent
 
-          Assembly.reset_counters(@parent.id, :children_count)
+          Assembly.reset_counters(parent.id, :children_count)
         end
       end
     end
   end
 end