COrrectif - priorité des promotions

Repository/Product/ProductFamilyStore.php

@@ -51,10 +51,11 @@ class ProductFamilyStore extends AbstractStore
     }
 
     public function getByParentCategory(
-            ProductCategoryInterface $parentCategory,
-            $user = null,
-            $query = null
-    ) {
+        ProductCategoryInterface $parentCategory,
+                                 $user = null,
+                                 $query = null
+    )
+    {
         $productFamiliesArray = [];
 
         foreach ($parentCategory->getChildrens() as $i => $category) {
@@ -69,14 +70,15 @@ class ProductFamilyStore extends AbstractStore
 
     // getProductFamiliesByCategory
     public function getByCategory(
-            ProductCategoryInterface $productCategory,
-            $query = null
-    ) {
+        ProductCategoryInterface $productCategory,
+                                 $query = null
+    )
+    {
         $query = $this->createDefaultQuery($query);
 
         $query
-                ->filterIsOnline()
-                ->filterByProductCategory($productCategory);
+            ->filterIsOnline()
+            ->filterByProductCategory($productCategory);
 
         return $query->find();
     }
@@ -85,7 +87,8 @@ class ProductFamilyStore extends AbstractStore
     public function getByCategoryOnlineAndOffline(
         ProductCategoryInterface $productCategory,
                                  $query = null
-    ) {
+    )
+    {
         $query = $this->createDefaultQuery($query);
 
         $query
@@ -100,8 +103,8 @@ class ProductFamilyStore extends AbstractStore
         $query = $this->createDefaultQuery($query);
 
         $query
-                ->filterByPropertyNoveltyExpirationDate()
-                ->filterIsOnline();
+            ->filterByPropertyNoveltyExpirationDate()
+            ->filterIsOnline();
 
         $results = $query->find();
 
@@ -114,8 +117,8 @@ class ProductFamilyStore extends AbstractStore
         $query = $this->createDefaultQuery($query);
 
         $query
-                ->filterIsOrganicLabel()
-                ->filterIsOnline();
+            ->filterIsOrganicLabel()
+            ->filterIsOnline();
 
         $results = $query->find();
 
@@ -133,10 +136,10 @@ class ProductFamilyStore extends AbstractStore
     {
         if ($user) {
             return $this->getWithReductions(
-                    $user->getFavoriteProductFamilies(),
-                    $user,
-                    false,
-                    $organizeByParentCategory
+                $user->getFavoriteProductFamilies(),
+                $user,
+                false,
+                $organizeByParentCategory
             );
         }
 
@@ -146,12 +149,13 @@ class ProductFamilyStore extends AbstractStore
 
     // findByTerms
     public function getByTerms(
-            $terms,
-            $maxResults = false,
-            $organizeByParentCategory = false,
-            $user = null,
-            $query = null
-    ) {
+        $terms,
+        $maxResults = false,
+        $organizeByParentCategory = false,
+        $user = null,
+        $query = null
+    )
+    {
         $query = $this->createDefaultQuery($query);
 
         $query->filterIsOnline();
@@ -169,26 +173,21 @@ class ProductFamilyStore extends AbstractStore
     }
 
     public function getBestReductionCatalog(
-            ProductFamilyInterface $productFamily,
-            ReductionCatalogInterface $reductionCatalog1,
-            ReductionCatalogInterface $reductionCatalog2
-    ) {
-        $price1 = $this->priceSolver->applyReductionCatalog(
-                $productFamily,
-                $this->priceSolver->getPrice($productFamily),
-                $this->priceSolver->getPriceWithTax($productFamily),
-                1,
-                $reductionCatalog1
+        ProductFamilyInterface    $productFamily,
+        ReductionCatalogInterface $reductionCatalog1,
+        ReductionCatalogInterface $reductionCatalog2
+    )
+    {
+        $price1 = $this->priceSolver->getPriceWithTaxByReduction(
+            $productFamily,
+            $reductionCatalog1
         );
-
-        $price2 = $this->priceSolver->applyReductionCatalog(
-                $productFamily,
-                $this->priceSolver->getPrice($productFamily),
-                $this->priceSolver->getPriceWithTax($productFamily),
-                1,
-                $reductionCatalog2
+        $price2 = $this->priceSolver->getPriceWithTaxByReduction(
+            $productFamily,
+            $reductionCatalog2
         );
 
+
         if ($price1 > $price2) {
             return $reductionCatalog2;
         } else {
@@ -198,29 +197,32 @@ class ProductFamilyStore extends AbstractStore
 
     // setReductionForProductFamilies
     public function getWithReductions(
-            $productFamilies,
-            UserInterface $user = null,
-            $organizeByCategory = false,
-            $organizeByParentCategory = false,
-            $onlyOnDiscount = false
-    ) {
+        $productFamilies,
+        UserInterface $user = null,
+        $organizeByCategory = false,
+        $organizeByParentCategory = false,
+        $onlyOnDiscount = false
+    )
+    {
         $conditions = [
-                'ids' => [],
-                'categories' => [],
+            'productFamiliesIds' => [],
+            'categories' => [],
+            'productFamilies' => [],
         ];
 
         foreach ($productFamilies as $productFamily) {
-            $conditions['ids'][] = $productFamily->getId();
+            $conditions['productFamiliesIds'][] = $productFamily->getId();
+            $conditions['productFamilies'][] = $productFamily;
             $conditions['categories'] = array_merge(
-                    $conditions['categories'],
-                    $productFamily->getProductCategories()->toArray()
+                $conditions['categories'],
+                $productFamily->getProductCategories()->toArray()
             );
         }
 
         if ($productFamilies) {
             $reductionCatalogs = $this->reductionCatalogStore->getByProductFamiliesConditions(
-                    $conditions,
-                    $user
+                $conditions,
+                $user
             );
         }
 
@@ -230,20 +232,19 @@ class ProductFamilyStore extends AbstractStore
                 $conditionProductFamilies = $conditionProductFamily = $conditionProductCategory = false;
 
                 if ($reductionCatalog->getProductFamilies()->contains(
-                                $productFamily
-                        ) || $reductionCatalog->getProductFamilies()->isEmpty()) {
+                        $productFamily
+                    ) || $reductionCatalog->getProductFamilies()->isEmpty()) {
                     $conditionProductFamilies = true;
                 }
 
-                if ($reductionCatalog->getProductFamily() == $productFamily || $reductionCatalog->getProductFamily(
-                        ) === null) {
+                if ($reductionCatalog->getProductFamily() == $productFamily || $reductionCatalog->getProductFamily() === null) {
                     $conditionProductFamily = true;
                 }
 
                 foreach ($productFamily->getProductCategories() as $productCategory) {
                     if ($reductionCatalog->getProductCategories()->contains(
-                                    $productCategory
-                            ) || $reductionCatalog->getProductCategories()->isEmpty()) {
+                            $productCategory
+                        ) || $reductionCatalog->getProductCategories()->isEmpty()) {
                         $conditionProductCategory = true;
                     }
                 }
@@ -251,11 +252,11 @@ class ProductFamilyStore extends AbstractStore
                 if ($conditionProductFamilies && $conditionProductFamily && $conditionProductCategory) {
                     if ($productFamily->getReductionCatalog()) {
                         $productFamily->setReductionCatalog(
-                                $this->getBestReductionCatalog(
-                                        $productFamily,
-                                        $reductionCatalog,
-                                        $productFamily->getReductionCatalog()
-                                )
+                            $this->getBestReductionCatalog(
+                                $productFamily,
+                                $reductionCatalog,
+                                $productFamily->getReductionCatalog()
+                            )
                         );
                     } else {
                         $productFamily->setReductionCatalog($reductionCatalog);
@@ -273,8 +274,8 @@ class ProductFamilyStore extends AbstractStore
                         if ($this->productCategorySolver->isDisplay($parentCategory, $user)) {
                             if (!isset($productFamiliesToReturn[$parentCategory->getId()])) {
                                 $productFamiliesToReturn[$parentCategory->getId()] = [
-                                        'category' => $parentCategory,
-                                        'products' => []
+                                    'category' => $parentCategory,
+                                    'products' => []
                                 ];
                             }
                             $productFamiliesToReturn[$parentCategory->getId()]['products'][] = $productFamily;

Repository/Reduction/ReductionCatalogRepositoryQuery.php

@@ -2,6 +2,7 @@
 
 namespace Lc\CaracoleBundle\Repository\Reduction;
 
+use Doctrine\Common\Collections\ArrayCollection;
 use Knp\Component\Pager\PaginatorInterface;
 use Lc\CaracoleBundle\Model\Product\ProductFamilyInterface;
 use Lc\CaracoleBundle\Repository\MerchantRepositoryQueryTrait;
@@ -29,8 +30,8 @@ class ReductionCatalogRepositoryQuery extends AbstractRepositoryQuery
             $this->isJoinUsers = true;
 
             return $this
-                    ->leftJoin('.users', 'pf_users')
-                    ->addSelect('pf_users') ;
+                ->leftJoin('.users', 'pf_users')
+                ->addSelect('pf_users');
         }
         return $this;
     }
@@ -41,8 +42,8 @@ class ReductionCatalogRepositoryQuery extends AbstractRepositoryQuery
             $this->isJoinGroupUsers = true;
 
             return $this
-                    ->leftJoin('.groupUsers', 'groupUsers')
-                    ->addSelect('groupUsers') ;
+                ->leftJoin('.groupUsers', 'groupUsers')
+                ->addSelect('groupUsers');
         }
         return $this;
     }
@@ -53,8 +54,8 @@ class ReductionCatalogRepositoryQuery extends AbstractRepositoryQuery
             $this->isJoinProductFamilies = true;
 
             return $this
-                    ->leftJoin('.productFamilies', 'productFamilies')
-                    ->addSelect('productFamilies') ;
+                ->leftJoin('.productFamilies', 'productFamilies')
+                ->addSelect('productFamilies');
         }
         return $this;
     }
@@ -65,8 +66,8 @@ class ReductionCatalogRepositoryQuery extends AbstractRepositoryQuery
             $this->isJoinProductFamily = true;
 
             return $this
-                    ->leftJoin('.productFamily', 'productFamily')
-                    ->addSelect('productFamily') ;
+                ->leftJoin('.productFamily', 'productFamily')
+                ->addSelect('productFamily');
         }
 
         return $this;
@@ -78,8 +79,8 @@ class ReductionCatalogRepositoryQuery extends AbstractRepositoryQuery
             $this->isJoinProductCategories = true;
 
             return $this
-                    ->leftJoin('.productCategories', 'productCategories')
-                    ->addSelect('productCategories') ;
+                ->leftJoin('.productCategories', 'productCategories')
+                ->addSelect('productCategories');
         }
 
         return $this;
@@ -88,15 +89,15 @@ class ReductionCatalogRepositoryQuery extends AbstractRepositoryQuery
     public function filterProductFamily(ProductFamilyInterface $productFamily)
     {
         return $this
-                ->andWhere('.productFamily = :productFamily')
-                ->setParameter(':productFamily', $productFamily);
+            ->andWhere('.productFamily = :productFamily')
+            ->setParameter(':productFamily', $productFamily);
     }
 
     public function filterConditionDate()
     {
         return $this
-                ->andWhere('.permanent = 1 OR ( .dateStart <= :now AND .dateEnd >= :now)')
-                ->setParameter(':now', new \DateTime());
+            ->andWhere('.permanent = 1 OR ( .dateStart <= :now AND .dateEnd >= :now)')
+            ->setParameter(':now', new \DateTime());
     }
 
     public function filterConditionUser(UserInterface $user = null)
@@ -105,11 +106,11 @@ class ReductionCatalogRepositoryQuery extends AbstractRepositoryQuery
 
         if ($user) {
             return $this
-                    ->andWhere(':user MEMBER OF .users OR .users is empty')
-                    ->setParameter('user', $user);
+                ->andWhere(':user MEMBER OF .users OR .users is empty')
+                ->setParameter('user', $user);
         } else {
             return $this
-                    ->andWhere('.users is empty');
+                ->andWhere('.users is empty');
         }
     }
 
@@ -119,21 +120,24 @@ class ReductionCatalogRepositoryQuery extends AbstractRepositoryQuery
 
         if ($user) {
             return $this
-                    ->andWhere(':groupUser MEMBER OF .groupUsers OR .groupUsers is empty')
-                    ->setParameter('groupUser', $user->getGroupUsers());
+                ->andWhere(':groupUser MEMBER OF .groupUsers OR .groupUsers is empty')
+                ->setParameter('groupUser', $user->getGroupUsers());
         } else {
             return $this
-                    ->andWhere('.groupUsers is empty');
+                ->andWhere('.groupUsers is empty');
         }
     }
 
-    public function filterConditionProductFamilies(array $productFamilies)
+    public function filterConditionProductFamilies(array $productFamiliesIds, array $productFamilies)
     {
         $this->joinProductFamilies();
+        $this->joinProductFamily();
 
         return $this
-                ->andWhere(':productFamilies MEMBER OF .productFamilies OR .productFamilies is empty')
-                ->setParameter('productFamilies', $productFamilies);
+            ->andWhere(':productFamiliesIds MEMBER OF .productFamilies OR .productFamilies is empty')
+            ->andWhere('.productFamily IN (:productFamilies) OR .productFamily is null')
+            ->setParameter('productFamiliesIds', $productFamiliesIds)
+            ->setParameter('productFamilies', $productFamilies);
     }
 
     public function filterConditionProductFamily(ProductFamilyInterface $productFamily)
@@ -141,8 +145,8 @@ class ReductionCatalogRepositoryQuery extends AbstractRepositoryQuery
         $this->joinProductFamily();
 
         return $this
-                ->andWhere(':productFamily MEMBER OF .productFamily OR .productFamily is empty')
-                ->setParameter('productFamily', $productFamily);
+            ->andWhere(':productFamily MEMBER OF .productFamily OR .productFamily is empty')
+            ->setParameter('productFamily', $productFamily);
     }
 
     public function filterConditionProductCategories(array $productCategories)
@@ -150,8 +154,8 @@ class ReductionCatalogRepositoryQuery extends AbstractRepositoryQuery
         $this->joinProductCategories();
 
         return $this
-                ->andWhere(':productCategory MEMBER OF .productCategories OR .productCategories is empty')
-                ->setParameter('productCategory', $productCategories);
+            ->andWhere(':productCategory MEMBER OF .productCategories OR .productCategories is empty')
+            ->setParameter('productCategory', $productCategories);
     }
 
 

Solver/Price/ProductPriceSolver.php

@@ -5,6 +5,7 @@ namespace Lc\CaracoleBundle\Solver\Price;
 use Lc\CaracoleBundle\Doctrine\Extension\ProductPropertyInterface;
 use Lc\CaracoleBundle\Model\Product\ProductFamilyInterface;
 use Lc\CaracoleBundle\Model\Product\ProductInterface;
+use Lc\CaracoleBundle\Model\Reduction\ReductionCatalogInterface;
 use Lc\CaracoleBundle\Solver\Product\ProductFamilySolver;
 use Lc\CaracoleBundle\Solver\Product\ProductSolver;
 
@@ -87,6 +88,18 @@ class ProductPriceSolver
         );
     }
 
+    //Bridge pour applyReductionCatalog qui ne peut pas être appeler à cause du call
+    public function getPriceWithTaxByReduction(ProductPropertyInterface $product, ReductionCatalogInterface $reductionCatalog)
+    {
+        return $this->applyReductionCatalog(
+            $product,
+            $this->getPrice($product),
+            $this->getPriceWithTax($product),
+            1,
+            $reductionCatalog
+        );
+    }
+
     public function getPriceByRefUnitWithTaxAndReduction(ProductPropertyInterface $product)
     {
         return ($this->getPriceByRefUnitWithTax($product) * $this->getPriceWithTaxAndReduction($product))