Move escapeSql to driver implementations. (#104)

terabytesoftw · web-flow · commit 03474c9cfca7 · 2022-11-21T15:20:08.000-03:00
* Move escapeSql to driver implementations.
diff --git a/src/Builder/LikeConditionBuilder.php b/src/Builder/LikeConditionBuilder.php
@@ -14,7 +14,7 @@
 
 final class LikeConditionBuilder extends AbstractLikeConditionBuilder
 {
-    protected string|null $escapeCharacter = '!';
+    private string|null $escapeCharacter = '!';
 
     /**
      * `\` is initialized in {@see buildLikeCondition()} method since we need to choose replacement value based on
@@ -28,7 +28,7 @@ final class LikeConditionBuilder extends AbstractLikeConditionBuilder
 
     public function __construct(private QueryBuilderInterface $queryBuilder)
     {
-        parent::__construct($queryBuilder);
+        parent::__construct($queryBuilder, $this->getEscapeSql());
     }
 
     /**
@@ -46,4 +46,17 @@ public function build(LikeConditionInterface $expression, array &$params = []):
 
         return parent::build($expression, $params);
     }
+
+    /**
+     * @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 '';
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`
`15`	`15`	`final class LikeConditionBuilder extends AbstractLikeConditionBuilder`
`16`	`16`	`{`
`17`		`- protected string\|null $escapeCharacter = '!';`
	`17`	`+ private string\|null $escapeCharacter = '!';`
`18`	`18`
`19`	`19`	`/**`
`20`	`20`	* `\` is initialized in {@see buildLikeCondition()} method since we need to choose replacement value based on
`@@ -28,7 +28,7 @@ final class LikeConditionBuilder extends AbstractLikeConditionBuilder`
`28`	`28`
`29`	`29`	`public function __construct(private QueryBuilderInterface $queryBuilder)`
`30`	`30`	`{`
`31`		`- parent::__construct($queryBuilder);`
	`31`	`+ parent::__construct($queryBuilder, $this->getEscapeSql());`
`32`	`32`	`}`
`33`	`33`
`34`	`34`	`/**`
`@@ -46,4 +46,17 @@ public function build(LikeConditionInterface $expression, array &$params = []):`
`46`	`46`
`47`	`47`	`return parent::build($expression, $params);`
`48`	`48`	`}`
	`49`	`+`
	`50`	`+ /**`
	`51`	`+ * @return string character used to escape special characters in LIKE conditions.`
	`52`	+ * By default, it's assumed to be `!`.
	`53`	`+ */`
	`54`	`+ private function getEscapeSql(): string`
	`55`	`+ {`
	`56`	`+ if ($this->escapeCharacter !== null) {`
	`57`	`+ return " ESCAPE '{$this->escapeCharacter}'";`
	`58`	`+ }`
	`59`	`+`
	`60`	`+ return '';`
	`61`	`+ }`
`49`	`62`	`}`