MC-20490: Catalog product list widget doesn't work with custom attribute conditions

Oleksandr Dubovyk · Oleksandr Dubovyk · commit c989fcb8c962 · 2019-09-28T03:05:58.000-05:00
- static test fixed
diff --git a/app/code/Magento/CatalogWidget/Model/Rule/Condition/Product.php b/app/code/Magento/CatalogWidget/Model/Rule/Condition/Product.php
@@ -107,7 +107,7 @@ function ($attribute) {
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      *
      * @param array &$attributes
      * @return void
@@ -223,8 +223,7 @@ protected function addNotGlobalAttribute(
                         $validEntities[] = $entityId;
                     }
                 } else {
-                    if (
-                        isset($storeValues[Store::DEFAULT_STORE_ID]) &&
+                    if (isset($storeValues[Store::DEFAULT_STORE_ID]) &&
                         $this->validateAttribute($storeValues[Store::DEFAULT_STORE_ID])
                     ) {
                         $validEntities[] = $entityId;
@@ -244,7 +243,7 @@ protected function addNotGlobalAttribute(
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      *
      * @return string
      */
@@ -265,7 +264,7 @@ public function getMappedSqlField()
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      *
      * @param \Magento\Catalog\Model\ResourceModel\Product\Collection $productCollection
      * @return $this

Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ function ($attribute) {`
`107`	`107`	`}`
`108`	`108`
`109`	`109`	`/**`
`110`		`- * {@inheritdoc}`
	`110`	`+ * @inheritdoc`
`111`	`111`	`*`
`112`	`112`	`* @param array &$attributes`
`113`	`113`	`* @return void`
`@@ -223,8 +223,7 @@ protected function addNotGlobalAttribute(`
`223`	`223`	`$validEntities[] = $entityId;`
`224`	`224`	`}`
`225`	`225`	`} else {`
`226`		`- if (`
`227`		`- isset($storeValues[Store::DEFAULT_STORE_ID]) &&`
	`226`	`+ if (isset($storeValues[Store::DEFAULT_STORE_ID]) &&`
`228`	`227`	`$this->validateAttribute($storeValues[Store::DEFAULT_STORE_ID])`
`229`	`228`	`) {`
`230`	`229`	`$validEntities[] = $entityId;`
`@@ -244,7 +243,7 @@ protected function addNotGlobalAttribute(`
`244`	`243`	`}`
`245`	`244`
`246`	`245`	`/**`
`247`		`- * {@inheritdoc}`
	`246`	`+ * @inheritdoc`
`248`	`247`	`*`
`249`	`248`	`* @return string`
`250`	`249`	`*/`
`@@ -265,7 +264,7 @@ public function getMappedSqlField()`
`265`	`264`	`}`
`266`	`265`
`267`	`266`	`/**`
`268`		`- * {@inheritdoc}`
	`267`	`+ * @inheritdoc`
`269`	`268`	`*`
`270`	`269`	`* @param \Magento\Catalog\Model\ResourceModel\Product\Collection $productCollection`
`271`	`270`	`* @return $this`