Update EavAttribute.php

angelo983 · web-flow · commit 84e649e34d2d · 2017-10-25T16:15:16.000+02:00
The existence of $optionsArray['delete'] does not guarantee the existence of $optionsArray['delete'][$optionId]
diff --git a/app/code/Magento/Swatches/Model/Plugin/EavAttribute.php b/app/code/Magento/Swatches/Model/Plugin/EavAttribute.php
@@ -174,7 +174,7 @@ protected function prepareOptionIds(array $optionsArray)
     {
         if (isset($optionsArray['value']) && is_array($optionsArray['value'])) {
             foreach (array_keys($optionsArray['value']) as $optionId) {
-                if (isset($optionsArray['delete']) && $optionsArray['delete'][$optionId] == 1) {
+                if (isset($optionsArray['delete'][$optionId]) && $optionsArray['delete'][$optionId] == 1) {
                     unset($optionsArray['value'][$optionId]);
                 }
             }

Original file line number	Diff line number	Diff line change
`@@ -174,7 +174,7 @@ protected function prepareOptionIds(array $optionsArray)`
`174`	`174`	`{`
`175`	`175`	`if (isset($optionsArray['value']) && is_array($optionsArray['value'])) {`
`176`	`176`	`foreach (array_keys($optionsArray['value']) as $optionId) {`
`177`		`- if (isset($optionsArray['delete']) && $optionsArray['delete'][$optionId] == 1) {`
	`177`	`+ if (isset($optionsArray['delete'][$optionId]) && $optionsArray['delete'][$optionId] == 1) {`
`178`	`178`	`unset($optionsArray['value'][$optionId]);`
`179`	`179`	`}`
`180`	`180`	`}`