serialising xlink attributes broken in latest Chrome

[bartvde]

This came up in #23 originally

see also: https://code.google.com/p/chromium/issues/detail?id=390879 and openlayers/openlayers#1359 are related

[highsource]

I think I've fixed this:
http://jsfiddle.net/lexi/6ecm5kfa/
(rawgit still shows the old version at the moment)

[highsource]

Please test/confirm.

[bartvde]

I've updated with bower and can see the new code in jsonix-all.js but the function (isXlinkFixRequired) never gets called in my case.

The issue is that the prefix is still p2 in Chrome and not xlink:

            <p0:Reference p2:href="http://geoserver/wps" method="POST"
                mimeType="text/xml; subtype=gml/3.1.1">

[bartvde]

I did a debugging session and it seems getPrefix returns the incorrect prefix because this.namespacePrefixes seems incorrect, I'll dig deeper why it's not populated correctly.

[bartvde]

No actually this is not the issue, as the Object is exactly the same in Firefox where it works

[bartvde]

okay I think I've identified the problem will come up with a fix

[bartvde]

so we should not check for the prefix, but instead check for the namespaceURI instead

[highsource]

Yes, you're right. Fix #26 is now applied - could you please recheck?

On Tue, Sep 2, 2014 at 11:01 AM, Bart van den Eijnden <
notifications@github.com> wrote:

so we should not check for the prefix, but instead check for the
namespaceURI instead

—
Reply to this email directly or view it on GitHub
#25 (comment).

[bartvde]

yes now it works, thanks

[highsource]

Hm. Sorry, I've made one mistake, wrong prefix.

On Tue, Sep 2, 2014 at 12:45 PM, Bart van den Eijnden <
notifications@github.com> wrote:

yes now it works, thanks

—
Reply to this email directly or view it on GitHub
#25 (comment).

[bartvde]

Right, I tested with: "jsonix": "highsource/jsonix#55c0667f6f6ce19d45e548c39f9923e14015f310" so the merge commit which worked

[highsource]

Ok, now with the right prefix. Now we're done. :)

On Tue, Sep 2, 2014 at 12:48 PM, Bart van den Eijnden <
notifications@github.com> wrote:

Right, I tested with: "jsonix":
"highsource/jsonix#55c0667f6f6ce19d45e548c39f9923e14015f310" so the merge
commit which worked

—
Reply to this email directly or view it on GitHub
#25 (comment).