Merge pull request svg#529 from ricardobeat/addClassesNoRepeat

GreLI · web-flow · commit 517449fc52f7 · 2016-08-25T23:52:55.000+04:00
addClassesToSVGElement: avoid repeating classnames
diff --git a/plugins/addClassesToSVGElement.js b/plugins/addClassesToSVGElement.js
@@ -42,11 +42,13 @@ exports.fn = function(data, params) {
 
     if (svg.isElem('svg')) {
         if (svg.hasAttr('class')) {
-            svg.attr('class').value =
-                svg.attr('class').value
-                    .split(' ')
-                    .concat(classNames)
-                    .join(' ');
+            var classes = svg.attr('class').value.split(' ');
+            classNames.forEach(function(className){
+                if (classes.indexOf(className) < 0) {
+                    classes.push(className);
+                }
+            });
+            svg.attr('class').value = classes.join(' ');
         } else {
             svg.addAttr({
                 name: 'class',
