Merge pull request #2 from justbetter/feature/only-update-open

Also update where the operation status is still null
justbetter · Aug 27, 2024 · c6b43fd · c6b43fd
2 parents d938910 + 5e05a9a
commit c6b43fd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/Actions/UpdateBulkStatuses.php b/src/Actions/UpdateBulkStatuses.php
@@ -15,7 +15,9 @@ public function update(): void
     {
         BulkRequest::query()
             ->whereHas('operations', function (Builder $query): void {
-                $query->where('status', '=', OperationStatus::Open);
+                $query
+                    ->where('status', '=', OperationStatus::Open)
+                    ->orWhereNull('status');
             })
             ->get()
             ->each(fn (BulkRequest $bulkRequest): PendingDispatch => UpdateBulkStatusJob::dispatch($bulkRequest));

diff --git a/tests/Actions/UpdateBulkStatusesTest.php b/tests/Actions/UpdateBulkStatusesTest.php
@@ -54,7 +54,7 @@ public function it_can_update_bulk_statuses(): void
 
         $status2->operations()->create([
             'operation_id' => 1,
-            'status' => OperationStatus::Open,
+            'status' => null,
         ]);
 
         $status3->operations()->create([