Skip to content

Commit e750fa2

Browse files
committed
MAGETWO-71255: Merge 2.2 into develop mainline (2.3)
- stabilize builds
1 parent 0a1d85c commit e750fa2

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

app/code/Magento/Catalog/Controller/Adminhtml/Product/Attribute/Validate.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -160,17 +160,17 @@ private function checkUniqueOption(DataObject $response, array $options = null)
160160
if (is_array($options)
161161
&& isset($options['value'])
162162
&& isset($options['delete'])
163-
&& !$this->isUniqueAdminValues($options['value'], $options['delete'])
163+
&& !empty($options['value'])
164+
&& !empty($options['delete'])
164165
) {
165166
$duplicates = $this->isUniqueAdminValues($options['value'], $options['delete']);
166-
if ($duplicates) {
167+
if (!empty($duplicates)) {
167168
$this->setMessageToResponse(
168169
$response,
169170
[__('The value of Admin must be unique. (%1)', implode(', ', $duplicates))]
170171
);
171-
172-
$response->setError(true);
173172
}
173+
$response->setError(true);
174174
}
175175
return $this;
176176
}

app/code/Magento/Catalog/Test/Unit/Controller/Adminhtml/Product/Attribute/ValidateTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,7 @@ public function provideUniqueData()
233233
"option_1" => "",
234234
"option_2" => "",
235235
]
236-
], false
236+
], true
237237
],
238238
'duplicate and deleted' => [
239239
[

0 commit comments

Comments
 (0)