refactor(sql): extract getSQLConditionsForCollection helper

The find/count/sum methods all repeated the same try/finally block to
set $currentQueryCollection around getSQLConditions(). Three identical
copies of an exception-safety pattern is one mistake away from a leak
the next time the pattern is added — extract into a single helper so
the set/reset is in one place.

No behaviour change.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: 2 people

src/Database/Adapter/SQL.php

@@ -59,6 +59,25 @@ protected function getFloatPrecision(float $value): string
         return sprintf('%.'. $this->floatPrecision . 'F', $value);
     }
 
+    /**
+     * Build SQL conditions for a query while exposing the active collection
+     * via $currentQueryCollection. The state is always reset, even when
+     * getSQLConditions throws — adapter overrides that read it (e.g.
+     * SQLite's FTS5 routing) rely on it being absent outside this scope.
+     *
+     * @param array<Query> $queries
+     * @param array<string,mixed> $binds
+     */
+    protected function getSQLConditionsForCollection(string $name, array $queries, array &$binds, string $separator = 'AND'): string
+    {
+        $this->currentQueryCollection = $name;
+        try {
+            return $this->getSQLConditions($queries, $binds, $separator);
+        } finally {
+            $this->currentQueryCollection = null;
+        }
+    }
+
     /**
      * Constructor.
      *
@@ -3165,12 +3184,7 @@ public function find(Document $collection, array $queries = [], ?int $limit = 25
             $where[] = '(' . implode(' OR ', $cursorWhere) . ')';
         }
 
-        $this->currentQueryCollection = $name;
-        try {
-            $conditions = $this->getSQLConditions($queries, $binds);
-        } finally {
-            $this->currentQueryCollection = null;
-        }
+        $conditions = $this->getSQLConditionsForCollection($name, $queries, $binds);
         if (!empty($conditions)) {
             $where[] = $conditions;
         }
@@ -3314,12 +3328,7 @@ public function count(Document $collection, array $queries = [], ?int $max = nul
             }
         }
 
-        $this->currentQueryCollection = $name;
-        try {
-            $conditions = $this->getSQLConditions($otherQueries, $binds);
-        } finally {
-            $this->currentQueryCollection = null;
-        }
+        $conditions = $this->getSQLConditionsForCollection($name, $otherQueries, $binds);
         if (!empty($conditions)) {
             $where[] = $conditions;
         }
@@ -3405,12 +3414,7 @@ public function sum(Document $collection, string $attribute, array $queries = []
             }
         }
 
-        $this->currentQueryCollection = $name;
-        try {
-            $conditions = $this->getSQLConditions($otherQueries, $binds);
-        } finally {
-            $this->currentQueryCollection = null;
-        }
+        $conditions = $this->getSQLConditionsForCollection($name, $otherQueries, $binds);
         if (!empty($conditions)) {
             $where[] = $conditions;
         }