Issue #25676 Method checkIsAllRequiredOptions for Bundle product must not allow option without actual value

karyna-t · karyna-t · commit a573bfb22d69 · 2019-11-20T20:57:26.000+02:00
diff --git a/app/code/Magento/Bundle/Model/Product/Type.php b/app/code/Magento/Bundle/Model/Product/Type.php
@@ -1263,7 +1263,7 @@ protected function checkIsAllRequiredOptions($product, $isStrictProcessMode, $op
     {
         if (!$product->getSkipCheckRequiredOption() && $isStrictProcessMode) {
             foreach ($optionsCollection->getItems() as $option) {
-                if ($option->getRequired() && !isset($options[$option->getId()])) {
+                if ($option->getRequired() && empty($options[$option->getId()])) {
                     throw new \Magento\Framework\Exception\LocalizedException(
                         __('Please select all required options.')
                     );

Original file line number	Diff line number	Diff line change
`@@ -1263,7 +1263,7 @@ protected function checkIsAllRequiredOptions($product, $isStrictProcessMode, $op`
`1263`	`1263`	`{`
`1264`	`1264`	`if (!$product->getSkipCheckRequiredOption() && $isStrictProcessMode) {`
`1265`	`1265`	`foreach ($optionsCollection->getItems() as $option) {`
`1266`		`- if ($option->getRequired() && !isset($options[$option->getId()])) {`
	`1266`	`+ if ($option->getRequired() && empty($options[$option->getId()])) {`
`1267`	`1267`	`throw new \Magento\Framework\Exception\LocalizedException(`
`1268`	`1268`	`__('Please select all required options.')`
`1269`	`1269`	`);`