duplicate xmlns:link attributes about gulp-svgstore HOT 7 CLOSED

@legostud, can you provide source files? I just need <svg> elements and their attributes, <svg> contents are not required.

from gulp-svgstore.

@legostud please provide a test case or I will have to close this as "can't reproduce".

from gulp-svgstore.

@legostud I added the test (that is passing) to this branch: https://github.com/w0rm/gulp-svgstore/tree/51-duplicate-namespaces

from gulp-svgstore.

/cc @ghetolay what do you think of this issue?

from gulp-svgstore.

Well an example would be great because duplicate of attribute names seems very unlikely.
Because we could totally do something wrong on the detect/check of namespace procedure but then the part about adding attribute is very simple :

for (var nsName in namespaces)
  $combinedSvg.attr(nsName, namespaces[nsName])

namespaces being an object it's technically impossible to add the same nsName twice. And EVEN if we did call attr() twice with same attribute name it should replace old value not duplicate the attribute.

So this is very odd for me and I have absolutely no idea about how this is possible.

from gulp-svgstore.

@legostud Also quick guess, if it works on gulp-svgstore 5.0.4, and since this version is not adding any namespace, this is probably because you are adding it yourself.
This is maybe why you end up with duplicate namespace on version 5.0.5 : first one added by gulp-svgstore and second one by you.

from gulp-svgstore.

@ghetolay yeah, could be it. Anyway, I am closing this because @legostud is not responding.

from gulp-svgstore.