Merge pull request #286 from magento-south/MAGETWO-45429-mb

slavvka · slavvka · commit 9bc199631f1e · 2016-01-13T12:58:32.000+02:00
[SOUTH] MAGETWO-45429
diff --git a/app/code/Magento/Bundle/view/frontend/templates/catalog/product/view/type/bundle/option/checkbox.phtml b/app/code/Magento/Bundle/view/frontend/templates/catalog/product/view/type/bundle/option/checkbox.phtml
@@ -31,6 +31,7 @@
                                type="checkbox"
                                <?php if ($_option->getRequired()) echo 'data-validate="{\'validate-one-required-by-name\':\'input[name^=&quot;bundle_option[' . $_option->getId() . ']&quot;]:checked\'}"'?>
                                name="bundle_option[<?php echo $_option->getId() ?>][<?php echo $_selection->getId() ?>]"
+                               data-selector="bundle_option[<?php echo $_option->getId() ?>][<?php echo $_selection->getId() ?>]"
                                <?php if ($block->isSelected($_selection)) echo ' checked="checked"' ?>
                                <?php if (!$_selection->isSaleable()) echo ' disabled="disabled"' ?>
                                value="<?php echo $_selection->getSelectionId() ?>"/>
diff --git a/app/code/Magento/Bundle/view/frontend/templates/catalog/product/view/type/bundle/option/multi.phtml b/app/code/Magento/Bundle/view/frontend/templates/catalog/product/view/type/bundle/option/multi.phtml
@@ -25,6 +25,7 @@
                     size="5"
                     id="bundle-option-<?php echo $_option->getId() ?>"
                     name="bundle_option[<?php echo $_option->getId() ?>][]"
+                    data-selector="bundle_option[<?php echo $_option->getId() ?>][]"
                     class="bundle-option-<?php echo $_option->getId() ?> multiselect product bundle option change-container-classname"
                     <?php if ($_option->getRequired()) echo 'data-validate={required:true}' ?>>
                 <?php if(!$_option->getRequired()): ?>
diff --git a/app/code/Magento/Bundle/view/frontend/templates/catalog/product/view/type/bundle/option/radio.phtml b/app/code/Magento/Bundle/view/frontend/templates/catalog/product/view/type/bundle/option/radio.phtml
@@ -32,6 +32,7 @@
                                class="radio product bundle option"
                                id="bundle-option-<?php echo $_option->getId() ?>"
                                name="bundle_option[<?php echo $_option->getId() ?>]"
+                               data-selector="bundle_option[<?php echo $_option->getId() ?>]"
                                <?php echo ($_default && $_default->isSalable())?'':' checked="checked" ' ?>
                                value=""/>
                         <label class="label" for="bundle-option-<?php echo $_option->getId() ?>">
@@ -46,6 +47,7 @@
                                id="bundle-option-<?php echo $_option->getId() ?>-<?php echo $_selection->getSelectionId() ?>"
                                <?php if ($_option->getRequired()) echo 'data-validate="{\'validate-one-required-by-name\':true}"'?>
                                name="bundle_option[<?php echo $_option->getId() ?>]"
+                               data-selector="bundle_option[<?php echo $_option->getId() ?>]"
                                <?php if ($block->isSelected($_selection)) echo ' checked="checked"' ?>
                                <?php if (!$_selection->isSaleable()) echo ' disabled="disabled"' ?>
                                value="<?php echo $_selection->getSelectionId() ?>"/>
@@ -67,6 +69,7 @@
                            class="input-text qty<?php if (!$_canChangeQty) echo ' qty-disabled' ?>"
                            type="number"
                            name="bundle_option_qty[<?php echo $_option->getId() ?>]"
+                           data-selector="bundle_option_qty[<?php echo $_option->getId() ?>]"
                            value="<?php echo $_defaultQty ?>"/>
                 </div>
             </div>
diff --git a/app/code/Magento/Bundle/view/frontend/templates/catalog/product/view/type/bundle/option/select.phtml b/app/code/Magento/Bundle/view/frontend/templates/catalog/product/view/type/bundle/option/select.phtml
@@ -27,6 +27,7 @@
         <?php else:?>
             <select id="bundle-option-<?php echo $_option->getId() ?>"
                     name="bundle_option[<?php echo $_option->getId() ?>]"
+                    data-selector="bundle_option[<?php echo $_option->getId() ?>]"
                     class="bundle-option-<?php echo $_option->getId() ?> product bundle option bundle-option-select change-container-classname"
                 <?php if ($_option->getRequired()) echo 'data-validate = {required:true}' ?>>
                 <option value=""><?php echo __('Choose a selection...') ?></option>
@@ -50,6 +51,7 @@
                            class="input-text qty<?php if (!$_canChangeQty) echo ' qty-disabled' ?>"
                            type="number"
                            name="bundle_option_qty[<?php echo $_option->getId() ?>]"
+                           data-selector="bundle_option_qty[<?php echo $_option->getId() ?>]"
                            value="<?php echo $_defaultQty ?>"/>
                 </div>
             </div>
diff --git a/app/code/Magento/Catalog/Block/Product/View/Options/Type/Date.php b/app/code/Magento/Catalog/Block/Product/View/Options/Type/Date.php
@@ -199,6 +199,7 @@ protected function _getHtmlSelect($name, $value = null)
             $extraParams .= ' onchange="opConfig.reloadPrice()"';
         }
         $extraParams .= ' data-role="calendar-dropdown" data-calendar-role="' . $name . '"';
+        $extraParams .= ' data-selector="' . $select->getName() . '"';
         $select->setExtraParams($extraParams);
 
         if ($value === null) {
diff --git a/app/code/Magento/Catalog/Block/Product/View/Options/Type/Select.php b/app/code/Magento/Catalog/Block/Product/View/Options/Type/Select.php
@@ -69,6 +69,7 @@ public function getValuesHtml()
             if (!$this->getSkipJsReloadPrice()) {
                 $extraParams .= ' onchange="opConfig.reloadPrice()"';
             }
+            $extraParams .= ' data-selector="' . $select->getName() . '"';
             $select->setExtraParams($extraParams);
 
             if ($configValue) {
@@ -97,6 +98,7 @@ public function getValuesHtml()
                             ' product-custom-option" name="options[' .
                             $_option->getId() .
                             ']"' .
+                            ' data-selector="options[' . $_option->getId() . ']"' .
                             ($this->getSkipJsReloadPrice() ? '' : ' onclick="opConfig.reloadPrice()"') .
                             ' value="" checked="checked" /><label class="label admin__field-label" for="options_' .
                             $_option->getId() .
@@ -128,6 +130,11 @@ public function getValuesHtml()
                     $checked = $configValue == $htmlValue ? 'checked' : '';
                 }
 
+                $dataSelector = 'options[' . $_option->getId() . ']';
+                if ($arraySign) {
+                    $dataSelector .= '[' . $htmlValue . ']';
+                }
+
                 $selectHtml .= '<div class="field choice admin__field admin__field-option' .
                     $require .
                     '">' .
@@ -151,6 +158,7 @@ public function getValuesHtml()
                     $htmlValue .
                     '" ' .
                     $checked .
+                    ' data-selector="' . $dataSelector . '"' .
                     ' price="' .
                     $this->pricingHelper->currencyByStore($_value->getPrice(true), $store, false) .
                     '" />' .
diff --git a/app/code/Magento/Catalog/view/frontend/templates/product/view/options/type/text.phtml b/app/code/Magento/Catalog/view/frontend/templates/product/view/options/type/text.phtml
@@ -37,6 +37,7 @@ $class = ($_option->getIsRequire()) ? ' required' : '';
                    data-validate="<?php echo  $block->escapeHtml(json_encode($_textValidate));?>"
                    <?php } ?>
                    name="options[<?php echo $_option->getId() ?>]"
+                   data-selector="options[<?php echo $_option->getId() ?>]"
                    value="<?php echo $block->escapeHtml($block->getDefaultValue()) ?>"/>
         <?php elseif ($_option->getType() == \Magento\Catalog\Model\Product\Option::OPTION_TYPE_AREA): ?>
             <?php $_textAreaValidate = null;
@@ -53,6 +54,7 @@ $class = ($_option->getIsRequire()) ? ' required' : '';
                         data-validate="<?php echo $block->escapeHtml(json_encode($_textAreaValidate));?>"
                     <?php } ?>
                       name="options[<?php echo $_option->getId() ?>]"
+                      data-selector="options[<?php echo $_option->getId() ?>]"
                       rows="5"
                       cols="25"><?php echo $block->escapeHtml($block->getDefaultValue()) ?></textarea>
         <?php endif; ?>
diff --git a/app/code/Magento/ConfigurableProduct/view/frontend/templates/product/view/type/options/configurable.phtml b/app/code/Magento/ConfigurableProduct/view/frontend/templates/product/view/type/options/configurable.phtml
@@ -21,6 +21,7 @@ $_attributes = $block->decorateArray($block->getAllowAttributes());
             </label>
             <div class="control">
                 <select name="super_attribute[<?php echo $_attribute->getAttributeId() ?>]"
+                        data-selector="super_attribute[<?php echo $_attribute->getAttributeId() ?>]"
                         data-validate="{required:true}"
                         id="attribute<?php echo $_attribute->getAttributeId() ?>"
                         class="super-attribute-select">
diff --git a/app/code/Magento/GroupedProduct/view/frontend/templates/product/view/type/grouped.phtml b/app/code/Magento/GroupedProduct/view/frontend/templates/product/view/type/grouped.phtml
@@ -47,6 +47,7 @@
                 <?php if ($_item->isSaleable()) : ?>
                     <div class="control qty">
                         <input type="number" name="super_group[<?php echo $_item->getId() ?>]"
+                               data-selector="super_group[<?php echo $_item->getId() ?>]"
                                maxlength="12"
                                value="<?php echo $_item->getQty() * 1 ?>"
                                title="<?php echo __('Qty') ?>"
diff --git a/app/code/Magento/Wishlist/view/frontend/web/js/add-to-wishlist.js b/app/code/Magento/Wishlist/view/frontend/web/js/add-to-wishlist.js
@@ -11,7 +11,7 @@ define([
     
     $.widget('mage.addToWishlist', {
         options: {
-            bundleInfo: 'div.control [name^=bundle_option]:not([name*=qty])',
+            bundleInfo: 'div.control [name^=bundle_option]',
             configurableInfo: '.super-attribute-select',
             groupedInfo: '#super-product-table input',
             downloadableInfo: '#downloadable-links-list input',
@@ -43,10 +43,12 @@ define([
             $(event.handleObj.selector).each(function(index, element){
                 if ($(element).is('input[type=text]')
                     || $(element).is('input[type=email]')
+                    || $(element).is('input[type=number]')
+                    || $(element).is('input[type=hidden]')
                     || $(element).is('input[type=checkbox]:checked')
                     || $(element).is('input[type=radio]:checked')
-                    || $('#' + element.id + ' option:selected').length
                     || $(element).is('textarea')
+                    || $('#' + element.id + ' option:selected').length
                 ) {
                     dataToAdd = $.extend({}, dataToAdd, self._getElementData(element));
                     return;
@@ -66,7 +68,7 @@ define([
             $('[data-action="add-to-wishlist"]').each(function(index, element) {
                 var params = $(element).data('post');
                 if (!params)
-                    params = {};
+                    params = {'data': {}};
 
                 if (!$.isEmptyObject(dataToAdd)) {
                     self._removeExcessiveData(params, dataToAdd);
@@ -89,7 +91,7 @@ define([
         },
         _getElementData: function(element) {
             var data = {},
-                elementName = $(element).attr('name'),
+                elementName = $(element).data('selector'),
                 elementValue = $(element).val();
             if ($(element).is('select[multiple]') && elementValue !== null) {
                 if (elementName.substr(elementName.length - 2) == '[]') {
@@ -121,6 +123,13 @@ define([
                     params = $(event.currentTarget).data('post'),
                     form = $(element).closest('form'),
                     action = params.action;
+                if (params.data.id) {
+                    $('<input>', {
+                        type: 'hidden',
+                        name: 'id',
+                        value: params.data.id
+                    }).appendTo(form);
+                }
                 if (params.data.uenc) {
                     action += 'uenc/' + params.data.uenc;
                 }

Original file line number	Diff line number	Diff line change
`@@ -199,6 +199,7 @@ protected function _getHtmlSelect($name, $value = null)`
`199`	`199`	`$extraParams .= ' onchange="opConfig.reloadPrice()"';`
`200`	`200`	`}`
`201`	`201`	`$extraParams .= ' data-role="calendar-dropdown" data-calendar-role="' . $name . '"';`
	`202`	`+ $extraParams .= ' data-selector="' . $select->getName() . '"';`
`202`	`203`	`$select->setExtraParams($extraParams);`
`203`	`204`
`204`	`205`	`if ($value === null) {`