Merge pull request #1773 from danskernesdigitalebibliotek/DDFHER-132_…

…event-api-changed

Update event API, editing series should reflect in changed date. DDFHER-132

web/modules/custom/dpl_event/src/EventWrapper.php

@@ -6,6 +6,7 @@
 use Drupal\drupal_typed\DrupalTyped;
 use Drupal\recurring_events\Entity\EventInstance;
 use Psr\Log\LoggerInterface;
+use Safe\DateTime;
 use Safe\DateTimeImmutable;
 
 /**
@@ -147,6 +148,36 @@ public function getState(): ?EventState {
     return NULL;
   }
 
+  /**
+   * Getting relevant updated date - either the series or instance.
+   *
+   * As we use inheritance, we want an updated series to also reflect update.
+   * We could implement this, by programmatically saving all instances when
+   * the series is saved, but this may have unforseen consequences, as it is
+   * working against the Drupal system.
+   * Instead, we'll look up the instance and series changed dates, and take
+   * which ever is newer.
+   */
+  public function getUpdatedDate(): ?DateTime {
+    $series = $this->event->getEventSeries();
+
+    $changed_instance = $this->event->getChangedTime();
+    $changed_series = $series->getChangedTime();
+
+    // Setting the timestamp to whichever is the larger.
+    $timestamp = ($changed_instance > $changed_series) ?
+      $changed_instance : $changed_series;
+
+    if (empty($timestamp)) {
+      return NULL;
+    }
+
+    $date = new DateTime();
+    $date->setTimestamp(intval($timestamp));
+
+    return $date;
+  }
+
   /**
    * Loading the field if it exists.
    *

web/modules/custom/dpl_event/src/Services/EventRestMapper.php

@@ -66,7 +66,7 @@ public function getResponse(EventInstance $event_instance): EventsGET200Response
       'ticketCapacity' => $this->getValue('event_ticket_capacity'),
       'ticketCategories' => $this->getTicketCategories(),
       'createdAt' => $this->getDateField('created'),
-      'updatedAt' => $this->getDateField('changed'),
+      'updatedAt' => $this->eventWrapper->getUpdatedDate(),
       'dateTime' => $this->getDate(),
       'externalData' => $this->getExternalData(),
       'series' => new EventsGET200ResponseInnerSeries([