MAGETWO-70761: Show values that are duplicate in attribute error msg #10213

Author: ishakhsuvarov

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

@@ -128,7 +128,7 @@ private function isUniqueAdminValues(array $optionsValues, array $deletedOptions
             }
         }
         $uniqueValues = array_unique($adminValues);
-        return ($uniqueValues === $adminValues);
+        return array_diff_assoc($adminValues, $uniqueValues);
     }
 
     /**
@@ -157,10 +157,16 @@ private function checkUniqueOption(DataObject $response, array $options = null)
         if (is_array($options)
             && !empty($options['value'])
             && !empty($options['delete'])
-            && !$this->isUniqueAdminValues($options['value'], $options['delete'])
         ) {
-            $this->setMessageToResponse($response, [__("The value of Admin must be unique.")]);
-            $response->setError(true);
+            $duplicates = $this->isUniqueAdminValues($options['value'], $options['delete']);
+            if ($duplicates) {
+                $this->setMessageToResponse(
+                    $response,
+                    [__('The value of Admin must be unique. (%1)', implode(', ', $duplicates))]
+                );
+
+                $response->setError(true);
+            }
         }
         return $this;
     }

app/code/Magento/Catalog/i18n/en_US.csv

@@ -717,3 +717,4 @@ none,none
 Overview,Overview
 Details,Details
 "The value of Admin must be unique.", "The value of Admin must be unique."
+"The value of Admin must be unique. (%1)", "The value of Admin must be unique. (%1)"

app/code/Magento/Catalog/view/adminhtml/web/catalog/product/attribute/unique-validate.js

@@ -10,11 +10,17 @@ define([
     'use strict';
 
     return function (config) {
-        var _config = jQuery.extend({
-            element: null,
-            message: '',
-            uniqueClass: 'required-unique'
-        }, config);
+        var msg = '',
+            _config = jQuery.extend({
+                element: null,
+                message: '',
+                uniqueClass: 'required-unique'
+            }, config),
+
+            /** @inheritdoc */
+            messager = function () {
+                return msg;
+            };
 
         if (typeof _config.element === 'string') {
             jQuery.validator.addMethod(
@@ -25,21 +31,27 @@ define([
                             .closest('table')
                             .find('.' + _config.uniqueClass + ':visible'),
                         valuesHash = {},
-                        isValid = true;
+                        isValid = true,
+                        duplicates = [];
 
                     inputs.each(function (el) {
                         var inputValue = inputs[el].value;
 
                         if (typeof valuesHash[inputValue] !== 'undefined') {
                             isValid = false;
+                            duplicates.push(inputValue);
                         }
                         valuesHash[inputValue] = el;
                     });
 
+                    if (!isValid) {
+                        msg = _config.message + ' (' + duplicates.join(', ') + ')';
+                    }
+
                     return isValid;
                 },
 
-                _config.message
+                messager
             );
         }
     };

app/code/Magento/Swatches/i18n/en_US.csv

@@ -34,4 +34,5 @@ Image,Image
 "Example format: 200x300.","Example format: 200x300."
 "Image Position","Image Position"
 "The value of Admin must be unique.","The value of Admin must be unique."
+"The value of Admin must be unique. (%1)","The value of Admin must be unique. (%1)"
 Description,Description