Merge pull request #1749 from JCID/patch-1

l3pp4rd · web-flow · commit d437a75c468f · 2017-02-13T16:49:08.000+02:00
[Sluggable] Fix query when identifier is a embedded value holder
diff --git a/lib/Gedmo/Sluggable/Mapping/Event/Adapter/ORM.php b/lib/Gedmo/Sluggable/Mapping/Event/Adapter/ORM.php
@@ -62,8 +62,9 @@ public function getSimilarSlugs($object, $meta, array $config, $slug)
         // include identifiers
         foreach ((array) $wrapped->getIdentifier(false) as $id => $value) {
             if (!$meta->isIdentifier($config['slug'])) {
-                $qb->andWhere($qb->expr()->neq('rec.'.$id, ':'.$id));
-                $qb->setParameter($id, $value);
+                $namedId = str_replace('.', '_', $id);
+                $qb->andWhere($qb->expr()->neq('rec.'.$id, ':'.$namedId));
+                $qb->setParameter($namedId, $value);
             }
         }
         $q = $qb->getQuery();

Original file line number	Diff line number	Diff line change
`@@ -62,8 +62,9 @@ public function getSimilarSlugs($object, $meta, array $config, $slug)`
`62`	`62`	`// include identifiers`
`63`	`63`	`foreach ((array) $wrapped->getIdentifier(false) as $id => $value) {`
`64`	`64`	`if (!$meta->isIdentifier($config['slug'])) {`
`65`		`- $qb->andWhere($qb->expr()->neq('rec.'.$id, ':'.$id));`
`66`		`- $qb->setParameter($id, $value);`
	`65`	`+ $namedId = str_replace('.', '_', $id);`
	`66`	`+ $qb->andWhere($qb->expr()->neq('rec.'.$id, ':'.$namedId));`
	`67`	`+ $qb->setParameter($namedId, $value);`
`67`	`68`	`}`
`68`	`69`	`}`
`69`	`70`	`$q = $qb->getQuery();`