Fix a migration error for sessions and ticket types

verbb · Nov 13, 2024 · 08bae14 · 08bae14
1 parent 11fff6c
commit 08bae14
Showing 1 changed file with 12 additions and 14 deletions.
diff --git a/src/migrations/m240921_000000_events3.php b/src/migrations/m240921_000000_events3.php
@@ -334,12 +334,11 @@ public function safeUp(): bool
                 ->endDate($event['endDate'])
                 ->one() ?? new Session();
 
-            $session->setAttributes([
-                'primaryOwnerId' => $event['id'],
-                'startDate' => $event['startDate'],
-                'endDate' => $event['endDate'],
-                'allDay' => (bool)$event['allDay'],
-            ], false);
+            // Use setters to ensure things work correctly
+            $session->setPrimaryOwnerId($event['id']);
+            $session->startDate = $event['startDate'];
+            $session->endDate = $event['endDate'];
+            $session->allDay = (bool)$event['allDay'];
 
             if (!Craft::$app->getElements()->saveElement($session)) {
                 throw new Exception(Json::encode($session->getErrors()));
@@ -372,14 +371,13 @@ public function safeUp(): bool
 
             $ticketType = new TicketType();
 
-            $ticketType->setAttributes([
-                'title' => $legacyTicketTypeElement->title ?? 'Ticket Type ' . rand(),
-                'primaryOwnerId' => $legacyTicket['eventId'],
-                'price' => $legacyTicket['price'],
-                'capacity' => $legacyTicket['quantity'],
-                'availableFrom' => $legacyTicket['availableFrom'],
-                'availableTo' => $legacyTicket['availableTo'],
-            ], false);
+            // Use setters to ensure things work correctly
+            $ticketType->title = $legacyTicketTypeElement->title ?? 'Ticket Type ' . rand();
+            $ticketType->setPrimaryOwnerId($legacyTicket['eventId']);
+            $ticketType->price = $legacyTicket['price'];
+            $ticketType->capacity = $legacyTicket['quantity'];
+            $ticketType->availableFrom = $legacyTicket['availableFrom'];
+            $ticketType->availableTo = $legacyTicket['availableTo'];
 
             $ticketType->setPrice($legacyTicket['price']);