ENGCOM-1650: [Backport] Removed redundant else statement #15436

magento-engcom-team · magento-engcom-team · commit 233c80ac4324 · 2018-05-23T02:00:22.000-05:00
- Merge Pull Request #15436 from rogyar/magento2:2.1-configurable-redundant-else - Merged commits: 1. ec092de
diff --git a/app/code/Magento/ConfigurableImportExport/Model/Import/Product/Type/Configurable.php b/app/code/Magento/ConfigurableImportExport/Model/Import/Product/Type/Configurable.php
@@ -246,9 +246,8 @@ protected function _getSuperAttributeId($productId, $attributeId)
     {
         if (isset($this->_productSuperAttrs["{$productId}_{$attributeId}"])) {
             return $this->_productSuperAttrs["{$productId}_{$attributeId}"];
-        } else {
-            return null;
         }
+        return null;
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -246,9 +246,8 @@ protected function _getSuperAttributeId($productId, $attributeId)`
`246`	`246`	`{`
`247`	`247`	`if (isset($this->_productSuperAttrs["{$productId}_{$attributeId}"])) {`
`248`	`248`	`return $this->_productSuperAttrs["{$productId}_{$attributeId}"];`
`249`		`- } else {`
`250`		`- return null;`
`251`	`249`	`}`
	`250`	`+ return null;`
`252`	`251`	`}`
`253`	`252`
`254`	`253`	`/**`