Merge pull request #140 from bem/issue-138

Add ignoring of duplicate 'css' classes

Author: eGavr

README.md

@@ -81,7 +81,7 @@ For example, the following two code samples will be considered to be equivalent:
 ```
 
 ```html
-<span class=" cd  ab bc">Text</span>
+<span class=" cd  ab bc bc">Text</span>
 ```
 
 **CAUTION!**<br>

README.ru.md

@@ -81,7 +81,7 @@
 ```
 
 ```html
-<span class=" cd  ab bc">Text</span>
+<span class=" cd  ab bc bc">Text</span>
 ```
 
 **ВНИМАНИЕ!**<br>

lib/utils/utils.js

@@ -51,7 +51,11 @@ function sortCssClasses(attrs) {
     function _sortCssClassesValues(cssClasses) {
         var classList = (cssClasses || '').split(' ');
 
-        return _.filter(classList).sort().join(' ');
+        return _(classList)
+            .filter()
+            .uniq()
+            .sort()
+            .join(' ');
     }
 
     var classIndexes = _getIndexesInArray(attrs, 'class');

test/differ/fixtures/second/sort-classes.html

@@ -1 +1 @@
-<input class=" cd  ab bc">
+<input class=" cd  ab bc bc">

test/unit/utils.js

@@ -29,11 +29,13 @@ describe('\'utils\'', function () {
                 { name: 'class', value: ' b c a' },
                 { name: 'blah', value: 'b c a' },
                 { name: 'class', value: '' },
+                { name: 'class', value: 'a a a' }
             ],
             output = [
                 { name: 'class', value: 'a b c' },
                 { name: 'blah', value: 'b c a' },
-                { name: 'class', value: '' }
+                { name: 'class', value: '' },
+                { name: 'class', value: 'a' }
             ];
 
         utils.sortCssClasses(input).must.be.eql(output);