Skip to content
Snippets Groups Projects
Commit fe83f692 authored by Andrii Lugovyi's avatar Andrii Lugovyi
Browse files

MAGETWO-44055: [GITHUB] Admin product «Custom Options» tab controls («Import...

MAGETWO-44055: [GITHUB] Admin product «Custom Options» tab controls («Import Options», «Add New Option» buttons) do not work when non-English locale is used #1971
parent bdc295b7
No related merge requests found
...@@ -9,7 +9,7 @@ ...@@ -9,7 +9,7 @@
?> ?>
<?php /** @var $block \Magento\Catalog\Block\Adminhtml\Product\Edit\Tab\Options */ ?> <?php /** @var $block \Magento\Catalog\Block\Adminhtml\Product\Edit\Tab\Options */ ?>
<div class="fieldset-wrapper" id="product-custom-options-wrapper"> <div class="fieldset-wrapper" id="product-custom-options-wrapper" data-block="product-custom-options">
<div class="fieldset-wrapper-title"> <div class="fieldset-wrapper-title">
<strong class="title"> <strong class="title">
<span><?php /* @escapeNotVerified */ echo __('Custom Options') ?></span> <span><?php /* @escapeNotVerified */ echo __('Custom Options') ?></span>
......
...@@ -88,7 +88,7 @@ require([ ...@@ -88,7 +88,7 @@ require([
], function(jQuery){ ], function(jQuery){
jQuery(function ($) { jQuery(function ($) {
var fieldSet = $('#Custom_Options'); var fieldSet = $('[data-block=product-custom-options]');
fieldSet.customOptions(<?php /* @escapeNotVerified */ echo $this->helper('Magento\Framework\Json\Helper\Data')->jsonEncode( fieldSet.customOptions(<?php /* @escapeNotVerified */ echo $this->helper('Magento\Framework\Json\Helper\Data')->jsonEncode(
[ [
'fieldId' => $block->getFieldId(), 'fieldId' => $block->getFieldId(),
......
...@@ -216,7 +216,7 @@ define([ ...@@ -216,7 +216,7 @@ define([
var widget = this, var widget = this,
currentElement = $(event.target), currentElement = $(event.target),
parentId = '#' + currentElement.closest('.fieldset-alt').attr('id'), parentId = '#' + currentElement.closest('.fieldset-alt').attr('id'),
group = currentElement.find('[value="' + currentElement.val() + '"]').closest('optgroup').attr('label'), group = currentElement.find('[value="' + currentElement.val() + '"]').closest('optgroup').attr('data-name'),
previousGroup = $(parentId + '_previous_group').val(), previousGroup = $(parentId + '_previous_group').val(),
previousBlock = $(parentId + '_type_' + previousGroup), previousBlock = $(parentId + '_type_' + previousGroup),
tmpl; tmpl;
......
...@@ -154,16 +154,18 @@ class Select extends \Magento\Framework\View\Element\AbstractBlock ...@@ -154,16 +154,18 @@ class Select extends \Magento\Framework\View\Element\AbstractBlock
if ($isArrayOption && is_array($option)) { if ($isArrayOption && is_array($option)) {
$value = $option['value']; $value = $option['value'];
$label = (string)$option['label']; $label = (string)$option['label'];
$name = $option['label'] instanceof \Magento\Framework\Phrase ? $option['label']->getText() : $label;
$params = !empty($option['params']) ? $option['params'] : []; $params = !empty($option['params']) ? $option['params'] : [];
} else { } else {
$value = (string)$key; $value = (string)$key;
$label = (string)$option; $label = (string)$option;
$name = $label;
$isArrayOption = false; $isArrayOption = false;
$params = []; $params = [];
} }
if (is_array($value)) { if (is_array($value)) {
$html .= '<optgroup label="' . $label . '">'; $html .= '<optgroup label="' . $label . '" data-name="' . $name . '">';
foreach ($value as $keyGroup => $optionGroup) { foreach ($value as $keyGroup => $optionGroup) {
if (!is_array($optionGroup)) { if (!is_array($optionGroup)) {
$optionGroup = ['value' => $keyGroup, 'label' => $optionGroup]; $optionGroup = ['value' => $keyGroup, 'label' => $optionGroup];
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment