chore: remove data updates on bulk event updates

Author: smarcet

app/Http/Controllers/Apis/Protected/Summit/OAuth2SummitEventsApiController.php

@@ -1398,7 +1398,7 @@ public function updateEvents($summit_id)
                 );
             }
 
-            $this->service->updateEvents($summit, $data->all());
+            $this->service->updateEvents($summit, $data->all(), false);
 
             return $this->updated();
         });
@@ -1766,4 +1766,4 @@ public function getScheduledEventStreamingInfo($summit_id, $event_id)
 
         });
     }
-}
+}

app/Services/Model/ISummitService.php

@@ -52,9 +52,10 @@ public function addEvent(Summit $summit, array $data);
      * @param Summit $summit
      * @param int $event_id
      * @param array $data
+     * @param bool $trigger_data_update
      * @return SummitEvent
      */
-    public function updateEvent(Summit $summit, $event_id, array $data);
+    public function updateEvent(Summit $summit, $event_id, array $data, bool $trigger_data_update = true);
 
     /**
      * @param Summit $summit
@@ -147,7 +148,7 @@ public function deleteMyEventFeedback(Member $member, Summit $summit, int $event
      * @throws ValidationException
      */
     public function deleteEventFeedback(Summit $summit, int $event_id, int $feedback_id):void;
-    
+
     /**
      * @param Summit $summit
      * @param $external_order_id
@@ -221,11 +222,12 @@ public function updateAndPublishEvents(Summit $summit, array $data);
     /**
      * @param Summit $summit
      * @param array $data
+     * @param bool $trigger_data_update
      * @throws ValidationException
      * @throws EntityNotFoundException
      * @return bool
      */
-    public function updateEvents(Summit $summit, array $data);
+    public function updateEvents(Summit $summit, array $data, bool $trigger_data_update = true);
 
     /**
      * @param array $data
@@ -680,4 +682,4 @@ public function getEventForStreamingInfo(Summit $summit, Member $current_user, i
      * @throws ValidationException
      */
     public function validateBadge(Summit $summit, string $badge_qr_code): SummitAttendeeBadge;
-}
+}

app/Services/Model/Imp/SummitService.php

@@ -606,9 +606,9 @@ public function addEvent(Summit $summit, array $data)
      * @param array $data
      * @return SummitEvent
      */
-    public function updateEvent(Summit $summit, $event_id, array $data)
+    public function updateEvent(Summit $summit, $event_id, array $data, bool $trigger_data_update = true)
     {
-        return $this->saveOrUpdateEvent($summit, $data, $event_id);
+        return $this->saveOrUpdateEvent($summit, $data, $event_id, $trigger_data_update);
     }
 
     /**
@@ -645,21 +645,23 @@ private function canPerformEventTypeTransition(SummitEventType $old_event_type,
      * @param Summit $summit
      * @param array $data
      * @param null|int $event_id
+     * @param bool $trigger_data_update
      * @return SummitEvent
      * @throws Exception
      */
-    private function saveOrUpdateEvent(Summit $summit, array $data, $event_id = null)
+    private function saveOrUpdateEvent(Summit $summit, array $data, $event_id = null, bool $trigger_data_update = true)
     {
-        return $this->tx_service->transaction(function () use ($summit, $data, $event_id) {
+        return $this->tx_service->transaction(function () use ($summit, $data, $event_id, $trigger_data_update) {
 
             Log::debug
             (
                 sprintf
                 (
-                    "SummitService::saveOrUpdateEvent summit %s event_id %s data %s",
+                    "SummitService::saveOrUpdateEvent summit %s event_id %s data %s trigger_data_update %b",
                     $summit->getId(),
                     $event_id ?? "NEW",
-                    json_encode($data)
+                    json_encode($data),
+                    $trigger_data_update
                 )
             );
 
@@ -746,6 +748,9 @@ private function saveOrUpdateEvent(Summit $summit, array $data, $event_id = null
                 SummitEventFactory::populate($summit, $event, $data);
             }
 
+            if(!$trigger_data_update){
+                $event->skipDateUpdate();
+            }
             $created_by = null;
             if (isset($data['created_by_id'])) {
                 $created_by = $this->member_repository->getById(intval($data['created_by_id']));
@@ -841,6 +846,7 @@ private function saveOrUpdateEvent(Summit $summit, array $data, $event_id = null
 
             $this->event_repository->add($event);
             $event->updateLastEdited();
+
             return $event;
         });
     }
@@ -1473,18 +1479,20 @@ public function updateAndPublishEvents(Summit $summit, array $data)
     /**
      * @param Summit $summit
      * @param array $data
+     * @param bool $trigger_data_update
      * @return bool
      * @throws EntityNotFoundException
      * @throws ValidationException
      */
-    public function updateEvents(Summit $summit, array $data)
+    public function updateEvents(Summit $summit, array $data, bool $trigger_data_update = true)
     {
         return $this->tx_service->transaction(function () use (
             $summit,
-            $data
+            $data,
+            $trigger_data_update
         ) {
             foreach ($data['events'] as $event_data) {
-                $this->updateEvent($summit, intval($event_data['id']), $event_data);
+                $this->updateEvent($summit, intval($event_data['id']), $event_data, $trigger_data_update);
             }
 
             return true;
@@ -4172,4 +4180,4 @@ public function validateBadge(Summit $summit, string $badge_qr_code): SummitAtte
 
         return $badge;
     }
-}
+}