fix: explicitly stage/clear transaction to correctly run afterCommit …

…jobs [laravel v10.32.0](laravel/framework#48859) requires explicit staging of transactions
oprudkyi · Nov 17, 2023 · 3bc31f4 · 3bc31f4
1 parent b57fa3d
commit 3bc31f4
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,9 @@ Changed
 - [Breaking] SessionNotFoundErrorMode was removed and will always run clear session pool. (#132) (#130)
 - [Breaking] Auth cache and Session pool now share the same file cache adapter (#139)
 
+Fixed
+- Explicitly stage/clear transaction on commit to correctly run afterCommit jobs in Laravel >= [v10.32.0](https://github.com/laravel/framework/pull/48859) (#144)
+
 # v5.2.2 (2023-08-22)
 
 Fixed

diff --git a/composer.json b/composer.json
@@ -11,7 +11,7 @@
     "php": "^8.1",
     "ext-grpc": "*",
     "ext-json": "*",
-    "laravel/framework": "~10.0",
+    "laravel/framework": "^10.32.0",
     "google/cloud-spanner": "^1.58.4",
     "grpc/grpc": "^1.42",
     "symfony/cache": "~6",

diff --git a/src/Concerns/ManagesTransactions.php b/src/Concerns/ManagesTransactions.php
@@ -141,6 +141,8 @@ protected function performSpannerCommit(): void
             $this->currentTransaction->commit();
         }
 
+        $this->transactionsManager?->stageTransactions($this->getName());
+
         $this->transactions = max(0, $this->transactions - 1);
         if ($this->isTransactionFinished()) {
             $this->currentTransaction = null;