From 06d281a21683cbc2e0180e42dd8efd482234b97b Mon Sep 17 00:00:00 2001 From: Hillel Coren Date: Thu, 2 Feb 2017 14:02:50 +0200 Subject: [PATCH] Re-enable support for queues --- app/Http/Controllers/InvoiceController.php | 21 +++++++++++++------- app/Ninja/Repositories/InvoiceRepository.php | 8 ++++++-- 2 files changed, 20 insertions(+), 9 deletions(-) diff --git a/app/Http/Controllers/InvoiceController.php b/app/Http/Controllers/InvoiceController.php index f084f1b80364..bacc80d77399 100644 --- a/app/Http/Controllers/InvoiceController.php +++ b/app/Http/Controllers/InvoiceController.php @@ -411,10 +411,13 @@ class InvoiceController extends BaseController if ($invoice->is_recurring) { $response = $this->emailRecurringInvoice($invoice); } else { - app('App\Ninja\Mailers\ContactMailer')->sendInvoice($invoice, false, $pdfUpload); - $response = true; - //$this->dispatch(new SendInvoiceEmail($invoice, false, $pdfUpload)); - //return true; + // TODO remove this with Laravel 5.3 (https://github.com/invoiceninja/invoiceninja/issues/1303) + if (config('queue.default') === 'sync') { + $response = app('App\Ninja\Mailers\ContactMailer')->sendInvoice($invoice, false, $pdfUpload); + } else { + $this->dispatch(new SendInvoiceEmail($invoice, false, $pdfUpload)); + $response = true; + } } if ($response === true) { @@ -445,9 +448,13 @@ class InvoiceController extends BaseController if ($invoice->isPaid()) { return true; } else { - return app('App\Ninja\Mailers\ContactMailer')->sendInvoice($invoice); - //$this->dispatch(new SendInvoiceEmail($invoice)); - //return true; + // TODO remove this with Laravel 5.3 (https://github.com/invoiceninja/invoiceninja/issues/1303) + if (config('queue.default') === 'sync') { + return app('App\Ninja\Mailers\ContactMailer')->sendInvoice($invoice); + } else { + $this->dispatch(new SendInvoiceEmail($invoice)); + return true; + } } } diff --git a/app/Ninja/Repositories/InvoiceRepository.php b/app/Ninja/Repositories/InvoiceRepository.php index 918a15ddefc7..acee9c68ac8a 100644 --- a/app/Ninja/Repositories/InvoiceRepository.php +++ b/app/Ninja/Repositories/InvoiceRepository.php @@ -779,8 +779,12 @@ class InvoiceRepository extends BaseRepository */ public function emailInvoice(Invoice $invoice) { - app('App\Ninja\Mailers\ContactMailer')->sendInvoice($invoice); - //dispatch(new SendInvoiceEmail($invoice)); + // TODO remove this with Laravel 5.3 (https://github.com/invoiceninja/invoiceninja/issues/1303) + if (config('queue.default') === 'sync') { + app('App\Ninja\Mailers\ContactMailer')->sendInvoice($invoice); + } else { + dispatch(new SendInvoiceEmail($invoice)); + } } /**