Merge remote-tracking branch 'origin/MAGETWO-96241' into 2.2-develop-pr20

Author: serhii-balko

app/code/Magento/SalesRule/Model/ResourceModel/Rule/Collection.php

@@ -80,6 +80,8 @@ protected function _construct()
     }
 
     /**
+     * Map data for associated entities
+     *
      * @param string $entityType
      * @param string $objectField
      * @throws \Magento\Framework\Exception\LocalizedException
@@ -114,6 +116,8 @@ protected function mapAssociatedEntities($entityType, $objectField)
     }
 
     /**
+     *  Add website ids and customer group ids to rules data
+     *
      * @return $this
      * @throws \Exception
      * @since 100.1.0
@@ -158,60 +162,15 @@ public function setValidationFilter(
 
             $connection = $this->getConnection();
             if (strlen($couponCode)) {
-                $select->joinLeft(
-                    ['rule_coupons' => $this->getTable('salesrule_coupon')],
-                    $connection->quoteInto(
-                        'main_table.rule_id = rule_coupons.rule_id AND main_table.coupon_type != ?',
-                        \Magento\SalesRule\Model\Rule::COUPON_TYPE_NO_COUPON
-                    ),
-                    ['code']
-                );
-
                 $noCouponWhereCondition = $connection->quoteInto(
-                    'main_table.coupon_type = ? ',
+                    'main_table.coupon_type = ?',
                     \Magento\SalesRule\Model\Rule::COUPON_TYPE_NO_COUPON
                 );
-
-                $autoGeneratedCouponCondition = [
-                    $connection->quoteInto(
-                        "main_table.coupon_type = ?",
-                        \Magento\SalesRule\Model\Rule::COUPON_TYPE_AUTO
-                    ),
-                    $connection->quoteInto(
-                        "rule_coupons.type = ?",
-                        \Magento\SalesRule\Api\Data\CouponInterface::TYPE_GENERATED
-                    ),
-                ];
-
-                $orWhereConditions = [
-                    "(" . implode($autoGeneratedCouponCondition, " AND ") . ")",
-                    $connection->quoteInto(
-                        '(main_table.coupon_type = ? AND main_table.use_auto_generation = 1 AND rule_coupons.type = 1)',
-                        \Magento\SalesRule\Model\Rule::COUPON_TYPE_SPECIFIC
-                    ),
-                    $connection->quoteInto(
-                        '(main_table.coupon_type = ? AND main_table.use_auto_generation = 0 AND rule_coupons.type = 0)',
-                        \Magento\SalesRule\Model\Rule::COUPON_TYPE_SPECIFIC
-                    ),
-                ];
-
-                $andWhereConditions = [
-                    $connection->quoteInto(
-                        'rule_coupons.code = ?',
-                        $couponCode
-                    ),
-                    $connection->quoteInto(
-                        '(rule_coupons.expiration_date IS NULL OR rule_coupons.expiration_date >= ?)',
-                        $this->_date->date()->format('Y-m-d')
-                    ),
-                ];
-
-                $orWhereCondition = implode(' OR ', $orWhereConditions);
-                $andWhereCondition = implode(' AND ', $andWhereConditions);
+                $relatedRulesIds = $this->getCouponRelatedRuleIds($couponCode);
 
                 $select->where(
-                    $noCouponWhereCondition . ' OR ((' . $orWhereCondition . ') AND ' . $andWhereCondition . ')',
-                    null,
+                    $noCouponWhereCondition . ' OR main_table.rule_id IN (?)',
+                    $relatedRulesIds,
                     Select::TYPE_CONDITION
                 );
             } else {
@@ -227,6 +186,75 @@ public function setValidationFilter(
         return $this;
     }
 
+    /**
+     * Get rules ids related to coupon code
+     *
+     * @param string $couponCode
+     * @return array
+     */
+    private function getCouponRelatedRuleIds(string $couponCode): array
+    {
+        $connection = $this->getConnection();
+        $select = $connection->select()->from(
+            ['main_table' => $this->getTable('salesrule')],
+            'rule_id'
+        );
+        $select->joinLeft(
+            ['rule_coupons' => $this->getTable('salesrule_coupon')],
+            $connection->quoteInto(
+                'main_table.rule_id = rule_coupons.rule_id AND main_table.coupon_type != ?',
+                \Magento\SalesRule\Model\Rule::COUPON_TYPE_NO_COUPON,
+                null
+            )
+        );
+
+        $autoGeneratedCouponCondition = [
+            $connection->quoteInto(
+                "main_table.coupon_type = ?",
+                \Magento\SalesRule\Model\Rule::COUPON_TYPE_AUTO
+            ),
+            $connection->quoteInto(
+                "rule_coupons.type = ?",
+                \Magento\SalesRule\Api\Data\CouponInterface::TYPE_GENERATED
+            ),
+        ];
+
+        $orWhereConditions = [
+            "(" . implode($autoGeneratedCouponCondition, " AND ") . ")",
+            $connection->quoteInto(
+                '(main_table.coupon_type = ? AND main_table.use_auto_generation = 1 AND rule_coupons.type = 1)',
+                \Magento\SalesRule\Model\Rule::COUPON_TYPE_SPECIFIC
+            ),
+            $connection->quoteInto(
+                '(main_table.coupon_type = ? AND main_table.use_auto_generation = 0 AND rule_coupons.type = 0)',
+                \Magento\SalesRule\Model\Rule::COUPON_TYPE_SPECIFIC
+            ),
+        ];
+
+        $andWhereConditions = [
+            $connection->quoteInto(
+                'rule_coupons.code = ?',
+                $couponCode
+            ),
+            $connection->quoteInto(
+                '(rule_coupons.expiration_date IS NULL OR rule_coupons.expiration_date >= ?)',
+                $this->_date->date()->format('Y-m-d')
+            ),
+        ];
+
+        $orWhereCondition = implode(' OR ', $orWhereConditions);
+        $andWhereCondition = implode(' AND ', $andWhereConditions);
+
+        $select->where(
+            '(' . $orWhereCondition . ') AND ' . $andWhereCondition,
+            null,
+            Select::TYPE_CONDITION
+        );
+        $select->group('main_table.rule_id');
+
+        return $connection->fetchCol($select);
+    }
+
     /**
      * Filter collection by website(s), customer group(s) and date.
      * Filter collection to only active rules.
@@ -366,6 +394,8 @@ public function addCustomerGroupFilter($customerGroupId)
     }
 
     /**
+     * Getter for _associatedEntitiesMap property
+     *
      * @return array
      * @deprecated 100.1.0
      */
@@ -380,6 +410,8 @@ private function getAssociatedEntitiesMap()
     }
 
     /**
+     * Getter for dateApplier property
+     *
      * @return DateApplier
      * @deprecated 100.1.0
      */