Delay events slightly to prevent race conditions

app/Listeners/Invoice/CreateInvoiceActivity.php

@@ -21,6 +21,8 @@ class CreateInvoiceActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+
     /**
      * Create the event listener.
      *

app/Listeners/Invoice/InvoiceArchivedActivity.php

@@ -21,6 +21,8 @@ class InvoiceArchivedActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Invoice/InvoiceCancelledActivity.php

@@ -21,6 +21,8 @@ class InvoiceCancelledActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Invoice/InvoiceCreatedNotification.php

@@ -24,6 +24,8 @@ class InvoiceCreatedNotification implements ShouldQueue
 {
     use UserNotifies;
 
+    public $delay = 5;
+    
     public function __construct()
     {
     }

app/Listeners/Invoice/InvoiceDeletedActivity.php

@@ -21,6 +21,8 @@ class InvoiceDeletedActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Invoice/InvoiceEmailActivity.php

@@ -21,6 +21,8 @@ class InvoiceEmailActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+
     /**
      * Create the event listener.
      *

app/Listeners/Invoice/InvoiceEmailFailedActivity.php

@@ -21,6 +21,8 @@ class InvoiceEmailFailedActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Invoice/InvoiceEmailedNotification.php

@@ -24,6 +24,8 @@ class InvoiceEmailedNotification implements ShouldQueue
 {
     use UserNotifies;
 
+    public $delay = 5;
+    
     public function __construct()
     {
     }

app/Listeners/Invoice/InvoiceFailedEmailNotification.php

@@ -24,6 +24,8 @@ class InvoiceFailedEmailNotification
 {
     use UserNotifies;
 
+    public $delay = 5;
+    
     public function __construct()
     {
     }

app/Listeners/Invoice/InvoicePaidActivity.php

@@ -21,6 +21,8 @@ class InvoicePaidActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Invoice/InvoiceReminderEmailActivity.php

@@ -20,6 +20,8 @@ class InvoiceReminderEmailActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Invoice/InvoiceRestoredActivity.php

@@ -21,6 +21,8 @@ class InvoiceRestoredActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Invoice/InvoiceReversedActivity.php

@@ -21,6 +21,8 @@ class InvoiceReversedActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Invoice/InvoiceViewedActivity.php

@@ -21,6 +21,8 @@ class InvoiceViewedActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Invoice/UpdateInvoiceActivity.php

@@ -21,6 +21,8 @@ class UpdateInvoiceActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Invoice/UpdateInvoiceInvitations.php

@@ -16,6 +16,9 @@ use Illuminate\Contracts\Queue\ShouldQueue;
 
 class UpdateInvoiceInvitations implements ShouldQueue
 {
+    
+    public $delay = 5;
+
     /**
      * Create the event listener.
      *

app/Listeners/Payment/PaymentEmailFailureActivity.php

@@ -19,6 +19,8 @@ class PaymentEmailFailureActivity implements ShouldQueue
 {
     use UserNotifies;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Payment/PaymentEmailedActivity.php

@@ -17,6 +17,8 @@ use Illuminate\Contracts\Queue\ShouldQueue;
 
 class PaymentEmailedActivity implements ShouldQueue
 {
+    public $delay = 5;
+    
     use UserNotifies;
 
     /**

app/Listeners/Payment/PaymentNotification.php

@@ -26,6 +26,9 @@ class PaymentNotification implements ShouldQueue
 {
     use UserNotifies;
 
+    public $delay = 5;
+
+    
     /**
      * Create the event listener.
      *

app/Listeners/Payment/PaymentRestoredActivity.php

@@ -21,6 +21,8 @@ class PaymentRestoredActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Quote/CreateQuoteInvitation.php

@@ -21,6 +21,8 @@ class CreateQuoteInvitation implements ShouldQueue
 {
     use MakesHash;
     
+    public $delay = 5;
+    
     /**
      * Handle the event.
      *

app/Listeners/Quote/QuoteApprovedActivity.php

@@ -21,6 +21,8 @@ class QuoteApprovedActivity implements ShouldQueue
 {
     protected $activity_repo;
 
+    public $delay = 5;
+    
     /**
      * Create the event listener.
      *

app/Listeners/Quote/QuoteApprovedNotification.php

@@ -25,6 +25,8 @@ class QuoteApprovedNotification implements ShouldQueue
 {
     use UserNotifies;
 
+    public $delay = 5;
+    
     public function __construct()
     {
     }

app/Listeners/Quote/QuoteApprovedWebhook.php

@@ -18,7 +18,8 @@ use Illuminate\Contracts\Queue\ShouldQueue;
 
 class QuoteApprovedWebhook implements ShouldQueue
 {
-
+    public $delay = 5;
+    
     public function __construct()
     {
     }

app/Listeners/Quote/QuoteArchivedActivity.php

@@ -19,6 +19,8 @@ use stdClass;
 
 class QuoteArchivedActivity implements ShouldQueue
 {
+    public $delay = 5;
+        
     protected $activity_repo;
 
     /**

app/Listeners/Quote/QuoteCreatedNotification.php

@@ -22,6 +22,8 @@ use Illuminate\Contracts\Queue\ShouldQueue;
 
 class QuoteCreatedNotification implements ShouldQueue
 {
+    public $delay = 5;
+        
     use UserNotifies;
 
     public function __construct()

app/Listeners/Quote/QuoteDeletedActivity.php

@@ -19,6 +19,8 @@ use stdClass;
 
 class QuoteDeletedActivity implements ShouldQueue
 {
+    public $delay = 5;
+        
     protected $activity_repo;
 
     /**

app/Listeners/Quote/QuoteEmailActivity.php

@@ -19,6 +19,8 @@ use stdClass;
 
 class QuoteEmailActivity implements ShouldQueue
 {
+    public $delay = 5;
+        
     protected $activity_repo;
 
     /**

app/Listeners/Quote/QuoteEmailedNotification.php

@@ -22,6 +22,9 @@ use Illuminate\Contracts\Queue\ShouldQueue;
 
 class QuoteEmailedNotification implements ShouldQueue
 {
+
+    public $delay = 5;
+        
     use UserNotifies;
 
     public function __construct()

app/Listeners/Quote/QuoteRestoredActivity.php

@@ -19,6 +19,8 @@ use stdClass;
 
 class QuoteRestoredActivity implements ShouldQueue
 {
+    public $delay = 5;
+        
     protected $activity_repo;
 
     /**

app/Listeners/Quote/QuoteViewedActivity.php

@@ -19,6 +19,8 @@ use stdClass;
 
 class QuoteViewedActivity implements ShouldQueue
 {
+    public $delay = 5;
+        
     protected $activity_repo;
 
     /**

app/Services/Payment/UpdateInvoicePayment.php

@@ -91,9 +91,16 @@ class UpdateInvoicePayment
                         ->workFlow()
                         ->save();
 
-            event(new InvoiceWasUpdated($invoice, $invoice->company, Ninja::eventVars(auth()->user() ? auth()->user()->id : null)));
         });
         
+        /* Remove the event updater from within the loop to prevent race conditions */
+
+        $invoices->each(function ($invoice) {
+        
+            event(new InvoiceWasUpdated($invoice, $invoice->company, Ninja::eventVars(auth()->user() ? auth()->user()->id : null)));
+
+        });
+
         $this->payment->saveQuietly();
 
         return $this->payment;

app/Utils/Traits/MakesInvoiceValues.php

@@ -330,17 +330,17 @@ trait MakesInvoiceValues
             // but that's no longer necessary.
 
             if (isset($item->tax_rate1)) {
-                $data[$key][$table_type.'.tax_rate1'] = round($item->tax_rate1, 2).'%';
+                $data[$key][$table_type.'.tax_rate1'] = floatval($item->tax_rate1).'%';
                 $data[$key][$table_type.'.tax1'] = &$data[$key][$table_type.'.tax_rate1'];
             }
 
             if (isset($item->tax_rate2)) {
-                $data[$key][$table_type.'.tax_rate2'] = round($item->tax_rate2, 2).'%';
+                $data[$key][$table_type.'.tax_rate2'] = floatval($item->tax_rate2).'%';
                 $data[$key][$table_type.'.tax2'] = &$data[$key][$table_type.'.tax_rate2'];
             }
 
             if (isset($item->tax_rate3)) {
-                $data[$key][$table_type.'.tax_rate3'] = round($item->tax_rate3, 2).'%';
+                $data[$key][$table_type.'.tax_rate3'] = floatval($item->tax_rate3).'%';
                 $data[$key][$table_type.'.tax3'] = &$data[$key][$table_type.'.tax_rate3'];
             }
 

tests/Unit/RedisVsDatabaseTest.php

@@ -38,12 +38,11 @@ class RedisVsDatabaseTest extends TestCase
 
         $currencies = Cache::get('currencies');
 
-        $currency =  $currencies->filter(function ($item) {
+        $currencies->filter(function ($item) {
             return $item->id == 17;
         })->first();
 
-
-        $total_time = microtime(true) - $start;
+        nlog(microtime(true) - $start);
 
         $this->assertTrue(true);
         // nlog($total_time);
@@ -57,8 +56,7 @@ class RedisVsDatabaseTest extends TestCase
 
         $currency = Currency::find(17);
 
-        $total_time = microtime(true) - $start;
-
+        nlog(microtime(true) - $start);
 
         $this->assertTrue(true);
         // nlog($total_time);