Skip to content

Commit 20b0f26

Browse files
committed
Replaced string values with constants (#24726)
1 parent de29ee2 commit 20b0f26

File tree

2 files changed

+24
-24
lines changed

2 files changed

+24
-24
lines changed

app/code/Magento/Catalog/Model/Product/Option.php

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -163,22 +163,22 @@ public function __construct(
163163
$this->customOptionValuesFactory = $customOptionValuesFactory ?:
164164
\Magento\Framework\App\ObjectManager::getInstance()->get(ProductCustomOptionValuesInterfaceFactory::class);
165165
$this->optionGroups = $optionGroups ?: [
166-
'date' => Date::class,
167-
'file' => File::class,
168-
'select' => Select::class,
169-
'text' => Text::class,
166+
self::OPTION_GROUP_DATE => Date::class,
167+
self::OPTION_GROUP_FILE => File::class,
168+
self::OPTION_GROUP_SELECT => Select::class,
169+
self::OPTION_GROUP_TEXT => Text::class,
170170
];
171171
$this->optionGroupsToTypes = $optionGroupsToTypes ?: [
172-
'field' => 'text',
173-
'area' => 'text',
174-
'file' => 'file',
175-
'drop_down' => 'select',
176-
'radio' => 'select',
177-
'checkbox' => 'select',
178-
'multiple' => 'select',
179-
'date' => 'date',
180-
'date_time' => 'date',
181-
'time' => 'date',
172+
self::OPTION_TYPE_FIELD => self::OPTION_GROUP_TEXT,
173+
self::OPTION_TYPE_AREA => self::OPTION_GROUP_TEXT,
174+
self::OPTION_TYPE_FILE => self::OPTION_GROUP_FILE,
175+
self::OPTION_TYPE_DROP_DOWN => self::OPTION_GROUP_SELECT,
176+
self::OPTION_TYPE_RADIO => self::OPTION_GROUP_SELECT,
177+
self::OPTION_TYPE_CHECKBOX => self::OPTION_GROUP_SELECT,
178+
self::OPTION_TYPE_MULTIPLE => self::OPTION_GROUP_SELECT,
179+
self::OPTION_TYPE_DATE => self::OPTION_GROUP_DATE,
180+
self::OPTION_TYPE_DATE_TIME => self::OPTION_GROUP_DATE,
181+
self::OPTION_TYPE_TIME => self::OPTION_GROUP_DATE,
182182
];
183183

184184
parent::__construct(

app/code/Magento/Catalog/etc/di.xml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -420,16 +420,16 @@
420420
<item name="text" xsi:type="string">Magento\Catalog\Model\Product\Option\Type\Text</item>
421421
</argument>
422422
<argument name="optionGroupsToTypes" xsi:type="array">
423-
<item name="field" xsi:type="string">text</item>
424-
<item name="area" xsi:type="string">text</item>
425-
<item name="file" xsi:type="string">file</item>
426-
<item name="drop_down" xsi:type="string">select</item>
427-
<item name="radio" xsi:type="string">select</item>
428-
<item name="checkbox" xsi:type="string">select</item>
429-
<item name="multiple" xsi:type="string">select</item>
430-
<item name="date" xsi:type="string">date</item>
431-
<item name="date_time" xsi:type="string">date</item>
432-
<item name="time" xsi:type="string">date</item>
423+
<item name="field" xsi:type="const">Magento\Catalog\Api\Data\ProductCustomOptionInterface::OPTION_GROUP_TEXT</item>
424+
<item name="area" xsi:type="const">Magento\Catalog\Api\Data\ProductCustomOptionInterface::OPTION_GROUP_TEXT</item>
425+
<item name="file" xsi:type="const">Magento\Catalog\Api\Data\ProductCustomOptionInterface::OPTION_GROUP_FILE</item>
426+
<item name="drop_down" xsi:type="const">Magento\Catalog\Api\Data\ProductCustomOptionInterface::OPTION_GROUP_SELECT</item>
427+
<item name="radio" xsi:type="const">Magento\Catalog\Api\Data\ProductCustomOptionInterface::OPTION_GROUP_SELECT</item>
428+
<item name="checkbox" xsi:type="const">Magento\Catalog\Api\Data\ProductCustomOptionInterface::OPTION_GROUP_SELECT</item>
429+
<item name="multiple" xsi:type="const">Magento\Catalog\Api\Data\ProductCustomOptionInterface::OPTION_GROUP_SELECT</item>
430+
<item name="date" xsi:type="const">Magento\Catalog\Api\Data\ProductCustomOptionInterface::OPTION_GROUP_DATE</item>
431+
<item name="date_time" xsi:type="const">Magento\Catalog\Api\Data\ProductCustomOptionInterface::OPTION_GROUP_DATE</item>
432+
<item name="time" xsi:type="const">Magento\Catalog\Api\Data\ProductCustomOptionInterface::OPTION_GROUP_DATE</item>
433433
</argument>
434434
</arguments>
435435
</type>

0 commit comments

Comments
 (0)