Fix 7: Move escapeSql to driver implementations. (#390)

terabytesoftw · web-flow · commit 763c33f4565f · 2022-11-21T15:38:43.000-03:00
Move escapeSql to driver implementations.
diff --git a/src/QueryBuilder/Conditions/Builder/LikeConditionBuilder.php b/src/QueryBuilder/Conditions/Builder/LikeConditionBuilder.php
@@ -23,8 +23,10 @@
  */
 class LikeConditionBuilder implements ExpressionBuilderInterface
 {
-    public function __construct(private QueryBuilderInterface $queryBuilder)
-    {
+    public function __construct(
+        private QueryBuilderInterface $queryBuilder,
+        private string|null $escapeSql = null
+    ) {
     }
 
     /**
@@ -36,7 +38,6 @@ public function __construct(private QueryBuilderInterface $queryBuilder)
         '_' => '\_',
         '\\' => '\\\\',
     ];
-    protected string|null $escapeCharacter = null;
 
     /**
      * @throws Exception|InvalidArgumentException|InvalidConfigException|NotSupportedException
@@ -68,7 +69,6 @@ public function build(LikeConditionInterface $expression, array &$params = []):
             $column = $this->queryBuilder->quoter()->quoteColumnName($column);
         }
 
-        $escapeSql = $this->getEscapeSql();
         $parts = [];
 
         /** @psalm-var string[] $values */
@@ -81,7 +81,7 @@ public function build(LikeConditionInterface $expression, array &$params = []):
                     $params
                 );
             }
-            $parts[] = "{$column} {$operator} {$phName}{$escapeSql}";
+            $parts[] = "{$column} {$operator} {$phName}{$this->escapeSql}";
         }
 
         return implode($andor, $parts);
@@ -95,7 +95,7 @@ public function build(LikeConditionInterface $expression, array &$params = []):
     protected function parseOperator(string $operator): array
     {
         if (!preg_match('/^(AND |OR |)((NOT |)I?LIKE)/', $operator, $matches)) {
-            throw new InvalidArgumentException("Invalid operator '$operator'.");
+            throw new InvalidArgumentException("Invalid operator in like condition: \"{$operator}\"");
         }
 
         $andor = ' ' . (!empty($matches[1]) ? $matches[1] : 'AND ');
@@ -104,17 +104,4 @@ protected function parseOperator(string $operator): array
 
         return [$andor, $not, $operator];
     }
-
-    /**
-     * @return string character used to escape special characters in LIKE conditions. By default,
-     * it's assumed to be `\`.
-     */
-    private function getEscapeSql(): string
-    {
-        if ($this->escapeCharacter !== null) {
-            return " ESCAPE '{$this->escapeCharacter}'";
-        }
-
-        return '';
-    }
 }
