Merge pull request #47 from michalsn/fix-reconnect

michalsn · web-flow · commit 837736181c87 · 2024-09-19T07:44:51.000+02:00
Add an additional condition before `reconnect()`
diff --git a/src/Models/QueueJobModel.php b/src/Models/QueueJobModel.php
@@ -51,7 +51,7 @@ public function getFromQueue(string $name, array $priority): ?QueueJob
     {
         // For SQLite3 memory database this will cause problems
         // so check if we're not in the testing environment first.
-        if ($this->db->database !== ':memory:') {
+        if ($this->db->database !== ':memory:' && $this->db->connID !== false) {
             // Make sure we still have the connection
             $this->db->reconnect();
         }

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ public function getFromQueue(string $name, array $priority): ?QueueJob`
`51`	`51`	`{`
`52`	`52`	`// For SQLite3 memory database this will cause problems`
`53`	`53`	`// so check if we're not in the testing environment first.`
`54`		`- if ($this->db->database !== ':memory:') {`
	`54`	`+ if ($this->db->database !== ':memory:' && $this->db->connID !== false) {`
`55`	`55`	`// Make sure we still have the connection`
`56`	`56`	`$this->db->reconnect();`
`57`	`57`	`}`