Better support for \\

Author: VincentLanglet

src/Doctrine/Orm/Filter/PartialSearchFilter.php

@@ -40,21 +40,16 @@ public function apply(QueryBuilder $queryBuilder, QueryNameGeneratorInterface $q
         if (!is_iterable($values)) {
             $queryBuilder->setParameter($parameterName, $this->formatLikeValue(strtolower($values)));
 
-            $queryBuilder->{$context['whereClause'] ?? 'andWhere'}($queryBuilder->expr()->like(
-                'LOWER('.$field.')',
-                ':'.$parameterName
-            ));
+            $likeExpression = 'LOWER('.$field.') LIKE :'.$parameterName.' ESCAPE \'\\\'';
+            $queryBuilder->{$context['whereClause'] ?? 'andWhere'}($likeExpression);
 
             return;
         }
 
         $likeExpressions = [];
         foreach ($values as $val) {
             $parameterName = $queryNameGenerator->generateParameterName($property);
-            $likeExpressions[] = $queryBuilder->expr()->like(
-                'LOWER('.$field.')',
-                ':'.$parameterName
-            );
+            $likeExpressions[] = 'LOWER('.$field.') LIKE :'.$parameterName.' ESCAPE \'\\\'';
             $queryBuilder->setParameter($parameterName, $this->formatLikeValue(strtolower($val)));
         }
 
@@ -65,6 +60,6 @@ public function apply(QueryBuilder $queryBuilder, QueryNameGeneratorInterface $q
 
     private function formatLikeValue(string $value): string
     {
-        return '%'.addcslashes($value, '%_').'%';
+        return '%'.addcslashes($value, '\\%_').'%';
     }
 }

tests/Functional/Parameters/PartialSearchFilterTest.php

@@ -119,19 +119,25 @@ public static function partialSearchFilterProvider(): \Generator
         yield 'filter by partial name "%"' => [
             '/chickens?namePartial=%25',
             1,
-            ['xx_%_\_%_xx'],
+            ['xx_%_\\_%_xx'],
         ];
 
         yield 'filter by partial name "_"' => [
-            '/chickens?namePartial=_',
+            '/chickens?namePartial=%5F',
             1,
-            ['xx_%_\_%_xx'],
+            ['xx_%_\\_%_xx'],
         ];
 
         yield 'filter by partial name "\"' => [
             '/chickens?namePartial=%5C',
             1,
-            ['xx_%_\_%_xx'],
+            ['xx_%_\\_%_xx'],
+        ];
+
+        yield 'filter by partial name "\_"' => [
+            '/chickens?namePartial=%5C%5F',
+            1,
+            ['xx_%_\\_%_xx'],
         ];
     }
 
@@ -158,7 +164,7 @@ private function loadFixtures(): void
         $chicken2->setChickenCoop($chickenCoop2);
 
         $chicken3 = new $chickenClass();
-        $chicken3->setName('xx_%_\_%_xx');
+        $chicken3->setName('xx_%_\\_%_xx');
         $chicken3->setChickenCoop($chickenCoop1);
 
         $chickenCoop1->addChicken($chicken1);