update_meeting.rb in decidim-meetings-0.29.0.rc1

- old
+ new
@@ -3,108 +3,70 @@
 module Decidim
   module Meetings
     module Admin
       # This command is executed when the user changes a Meeting from the admin
       # panel.
-      class UpdateMeeting < Decidim::Command
-        # Initializes a UpdateMeeting Command.
-        #
-        # form - The form from which to get the data.
-        # meeting - The current instance of the page to be updated.
-        def initialize(form, meeting)
-          @form = form
-          @meeting = meeting
-        end
+      class UpdateMeeting < Decidim::Commands::UpdateResource
+        fetch_form_attributes :scope, :category, :end_time, :start_time, :online_meeting_url, :registration_type,
+                              :registration_url, :registrations_enabled, :address, :latitude, :longitude, :location,
+                              :location_hints,
+                              :private_meeting, :transparent, :iframe_embed_type, :comments_enabled,
+                              :comments_start_time, :comments_end_time, :iframe_access_level
 
-        # Updates the meeting if valid.
-        #
-        # Broadcasts :ok if successful, :invalid otherwise.
-        def call
-          return broadcast(:invalid) if form.invalid?
+        protected
 
-          transaction do
-            update_meeting!
-            send_notification if should_notify_followers?
-            schedule_upcoming_meeting_notification if meeting.published? && start_time_changed?
-            update_services!
-          end
-
-          broadcast(:ok, meeting)
+        def run_after_hooks
+          send_notification if should_notify_followers?
+          schedule_upcoming_meeting_notification if resource.published? && start_time_changed?
+          update_services!
         end
 
-        private
-
-        attr_reader :form, :meeting
-
-        def update_meeting!
+        def attributes
           parsed_title = Decidim::ContentProcessor.parse_with_processor(:hashtag, form.title, current_organization: form.current_organization).rewrite
           parsed_description = Decidim::ContentProcessor.parse(form.description, current_organization: form.current_organization).rewrite
-
-          Decidim.traceability.update!(
-            meeting,
-            form.current_user,
-            scope: form.scope,
-            category: form.category,
-            title: parsed_title,
-            description: parsed_description,
-            end_time: form.end_time,
-            start_time: form.start_time,
-            online_meeting_url: form.online_meeting_url,
-            registration_type: form.registration_type,
-            registration_url: form.registration_url,
-            registrations_enabled: form.registrations_enabled,
-            type_of_meeting: form.clean_type_of_meeting,
-            address: form.address,
-            latitude: form.latitude,
-            longitude: form.longitude,
-            location: form.location,
-            location_hints: form.location_hints,
-            private_meeting: form.private_meeting,
-            transparent: form.transparent,
-            iframe_embed_type: form.iframe_embed_type,
-            comments_enabled: form.comments_enabled,
-            comments_start_time: form.comments_start_time,
-            comments_end_time: form.comments_end_time,
-            iframe_access_level: form.iframe_access_level
-          )
+          super.merge({
+                        title: parsed_title,
+                        description: parsed_description,
+                        type_of_meeting: form.clean_type_of_meeting
+                      })
         end
 
         def update_services!
-          meeting.services = form.services_to_persist.map do |service|
-            Decidim::Meetings::Service.new("title" => service.title, "description" => service.description)
+          resource.services = form.services_to_persist.map do |service|
+            Decidim::Meetings::Service.new(title: service.title, description: service.description)
           end
-          meeting.save!
+          resource.save!
         end
 
         def send_notification
           Decidim::EventsManager.publish(
             event: "decidim.events.meetings.meeting_updated",
             event_class: Decidim::Meetings::UpdateMeetingEvent,
-            resource: meeting,
-            followers: meeting.followers
+            resource:,
+            followers: resource.followers
           )
         end
 
         def should_notify_followers?
-          meeting.published? && important_attributes.any? { |attr| meeting.previous_changes[attr].present? }
+          resource.published? && important_attributes.any? { |attr| resource.previous_changes[attr].present? }
         end
 
         def important_attributes
           %w(start_time end_time address)
         end
 
         def start_time_changed?
-          meeting.previous_changes["start_time"].present?
+          resource.previous_changes["start_time"].present?
         end
 
         def schedule_upcoming_meeting_notification
-          return if meeting.start_time < Time.zone.now
+          return if resource.start_time < Time.zone.now
 
-          checksum = Decidim::Meetings::UpcomingMeetingNotificationJob.generate_checksum(meeting)
+          checksum = Decidim::Meetings::UpcomingMeetingNotificationJob.generate_checksum(resource)
 
           Decidim::Meetings::UpcomingMeetingNotificationJob
-            .set(wait_until: meeting.start_time - Decidim::Meetings.upcoming_meeting_notification)
-            .perform_later(meeting.id, checksum)
+            .set(wait_until: resource.start_time - Decidim::Meetings.upcoming_meeting_notification)
+            .perform_later(resource.id, checksum)
         end
       end
     end
   end
 end