Add ability to limit products and taxons when looking for name part

Sylius · Jun 13, 2019 · 95d841e · 95d841e
1 parent 901be26
commit 95d841e
Show file tree

Hide file tree

Showing 6 changed files with 9 additions and 5 deletions.
diff --git a/src/Sylius/Bundle/AdminBundle/Resources/config/routing/ajax/product.yml b/src/Sylius/Bundle/AdminBundle/Resources/config/routing/ajax/product.yml
@@ -19,6 +19,7 @@ sylius_admin_ajax_product_by_name_phrase:
                 arguments:
                     phrase: $phrase
                     locale: expr:service('sylius.context.locale').getLocaleCode()
+                    limit: 25
 
 sylius_admin_ajax_product_by_code:
     path: /code

diff --git a/src/Sylius/Bundle/AdminBundle/Resources/config/routing/ajax/taxon.yml b/src/Sylius/Bundle/AdminBundle/Resources/config/routing/ajax/taxon.yml
@@ -50,6 +50,8 @@ sylius_admin_ajax_taxon_by_name_phrase:
                 method: findByNamePart
                 arguments:
                     phrase: $phrase
+                    locale: expr:service('sylius.context.locale').getLocaleCode()
+                    limit: 25
 
 sylius_admin_ajax_generate_taxon_slug:
     path: /generate-slug/

diff --git a/src/Sylius/Bundle/ProductBundle/Doctrine/ORM/ProductRepository.php b/src/Sylius/Bundle/ProductBundle/Doctrine/ORM/ProductRepository.php
@@ -36,14 +36,14 @@ public function findByName(string $name, string $locale): array
     /**
      * {@inheritdoc}
      */
-    public function findByNamePart(string $phrase, string $locale): array
+    public function findByNamePart(string $phrase, string $locale, ?int $limit = null): array
     {
         return $this->createQueryBuilder('o')
             ->innerJoin('o.translations', 'translation', 'WITH', 'translation.locale = :locale')
             ->andWhere('translation.name LIKE :name')
             ->setParameter('name', '%' . $phrase . '%')
             ->setParameter('locale', $locale)
-            ->setMaxResults(25)
+            ->setMaxResults($limit)
             ->getQuery()
             ->getResult()
         ;

diff --git a/src/Sylius/Bundle/TaxonomyBundle/Doctrine/ORM/TaxonRepository.php b/src/Sylius/Bundle/TaxonomyBundle/Doctrine/ORM/TaxonRepository.php
@@ -87,11 +87,12 @@ public function findRootNodes(): array
     /**
      * {@inheritdoc}
      */
-    public function findByNamePart(string $phrase, ?string $locale = null): array
+    public function findByNamePart(string $phrase, ?string $locale = null, ?int $limit = null): array
     {
         return $this->createTranslationBasedQueryBuilder($locale)
             ->andWhere('translation.name LIKE :name')
             ->setParameter('name', '%' . $phrase . '%')
+            ->setMaxResults($limit)
             ->getQuery()
             ->getResult()
         ;

diff --git a/src/Sylius/Component/Product/Repository/ProductRepositoryInterface.php b/src/Sylius/Component/Product/Repository/ProductRepositoryInterface.php
@@ -26,5 +26,5 @@ public function findByName(string $name, string $locale): array;
     /**
      * @return array|ProductInterface[]
      */
-    public function findByNamePart(string $phrase, string $locale): array;
+    public function findByNamePart(string $phrase, string $locale, ?int $limit = null): array;
 }
diff --git a/src/Sylius/Component/Taxonomy/Repository/TaxonRepositoryInterface.php b/src/Sylius/Component/Taxonomy/Repository/TaxonRepositoryInterface.php
@@ -39,7 +39,7 @@ public function findByName(string $name, string $locale): array;
     /**
      * @return array|TaxonInterface[]
      */
-    public function findByNamePart(string $phrase, ?string $locale = null): array;
+    public function findByNamePart(string $phrase, ?string $locale = null, ?int $limit = null): array;
 
     public function createListQueryBuilder(): QueryBuilder;
 }