Moved forgetting session and where events are fired

1. After we have save the order to the database forget the session.
2. Firing events to create and send tickets should only be done after the we have saved the order to the database
This commit is contained in:
Jeremy Quinton 2018-08-17 16:15:31 +02:00
parent ec69e0711b
commit 68c82e0b34
1 changed files with 8 additions and 13 deletions

View File

@ -671,18 +671,6 @@ class EventCheckoutController extends Controller
}
}
/*
* Kill the session
*/
session()->forget('ticket_order_' . $event->id);
/*
* Queue up some tasks - Emails to be sent, PDFs etc.
*/
Log::info('Firing the event');
event(new OrderCompletedEvent($order));
} catch (Exception $e) {
Log::error($e);
@ -694,8 +682,15 @@ class EventCheckoutController extends Controller
]);
}
//save the order to the database
DB::commit();
//forget the order in the session
session()->forget('ticket_order_' . $event->id);
// Queue up some tasks - Emails to be sent, PDFs etc.
Log::info('Firing the event');
event(new OrderCompletedEvent($order));
if ($return_json) {
return response()->json([