MAGETWO-50460: User Defined qty available for 'Checkbox' bundle option

Author: Oleksandr Iegorov

app/code/Magento/Bundle/Ui/DataProvider/Product/Form/Modifier/BundlePanel.php

@@ -458,8 +458,8 @@ protected function getOptionInfo()
                                 'component' => 'Magento_Bundle/js/components/bundle-input-type',
                                 'parentContainer' => 'product_bundle_container',
                                 'selections' => 'bundle_selections',
-                                'targetIndex' => 'is_default',
-                                'targetCheckbox' => 'selection_can_change_qty',
+                                'isDefaultIndex' => 'is_default',
+                                'userDefinedIndex' => 'selection_can_change_qty',
                                 'dataScope' => 'type',
                                 'label' => __('Input Type'),
                                 'sortOrder' => 20,

app/code/Magento/Bundle/view/adminhtml/web/js/components/bundle-input-type.js

@@ -26,54 +26,42 @@ define([
 
             if (type !== this.previousType) {
                 this.previousType = type;
-
-                if (type === 'radio') {
-                    this.clearValues();
-                }
+                this.processSelections(type === 'radio');
             }
 
-            this.toggleCheckbox(type === 'radio');
-
             this._super();
         },
 
         /**
-         * Toggle 'User Defined' checkbox in selections
-         * @param {Boolean} visibility
+         * Toggle 'User Defined' column and clears values
+         * @param {Boolean} isRadio
          */
-        toggleCheckbox: function (visibility) {
-            var records = registry.get(this.retrieveParentName(this.parentContainer) + '.' + this.selections);
+        processSelections: function (isRadio) {
+            var records = registry.get(this.retrieveParentName(this.parentContainer) + '.' + this.selections),
+                checkedFound = false;
 
             records.elems.each(function (record) {
                 record.elems.filter(function (comp) {
-                    return comp.index === this.targetCheckbox;
+                    return comp.index === this.userDefinedIndex;
                 }, this).each(function (comp) {
-                    comp.visible(visibility);
+                    comp.visible(isRadio);
                 });
-            }, this);
-        },
 
-        /**
-         * Clears values in components like this.
-         */
-        clearValues: function () {
-            var records = registry.get(this.retrieveParentName(this.parentContainer) + '.' + this.selections),
-                checkedFound = false;
+                if (isRadio) {
+                    record.elems.filter(function (comp) {
+                        return comp.index === this.isDefaultIndex;
+                    }, this).each(function (comp) {
+                        if (comp.checked()) {
+                            if (checkedFound) {
+                                comp.clearing = true;
+                                comp.clear();
+                                comp.clearing = false;
+                            }
 
-            records.elems.each(function (record) {
-                record.elems.filter(function (comp) {
-                    return comp.index === this.targetIndex;
-                }, this).each(function (comp) {
-                    if (comp.checked()) {
-                        if (checkedFound) {
-                            comp.clearing = true;
-                            comp.clear();
-                            comp.clearing = false;
+                            checkedFound = true;
                         }
-
-                        checkedFound = true;
-                    }
-                });
+                    });
+                }
             }, this);
         },