svgdraw creates wrong item

[ThomasWaldmann]

Original report by Reimar Bauer (Bitbucket: ReimarBauer, GitHub: ReimarBauer).

---

by some change svgdraw (may be this counts for all drawing items)
now saves to item name "undefined" and not to the given one.

[ThomasWaldmann]

Original comment by Sam Toyer (Bitbucket: qxcv, GitHub: qxcv).

---

I've figured out the problem here - env/lib/python2.7/site-packages/xstatic/pkg/svgedit_moin/data/editor/extensions/ext-server_moinsave.js has a line:

var name = qstr.substr(9).split('/+get/')[1];

Which should be:

var name = qstr.substr(9).split('/%2Bget/')[1];

As Flask's url_for quotes the URL so it can be included in the form.

Presumably it was broken by an upstream change in Flask or Werkzeug.

Also: the "Unable to load from URL:\nundefined" error message is unrelated, undefined refers to the fact that jQuery did not return an error message when it got a 404 on the /+get/<item name> page. See line 4813 of svg-editor.js.

[ThomasWaldmann]

Original comment by Thomas Waldmann (Bitbucket: thomaswaldmann, GitHub: thomaswaldmann).

---

can you check whether this bug is introduced by werkzeug 0.8.2 and does not happen with 0.8.1?

[ThomasWaldmann]

Original comment by Sam Toyer (Bitbucket: qxcv, GitHub: qxcv).

---

Ah, I see your issue at pallets/werkzeug#146

Installed Werkzeug 0.8.1 and the issue no longer happened. I shall amend my original fix to:

var name = qstr.substr(9).split(/\/(?:\+|%2B)get\//)[1];

This should handle both situations until Werkzeug >0.8.2 is released.

[ThomasWaldmann]

Original comment by xiaq (Bitbucket: xiaq, GitHub: xiaq).

---

As of today, this bug is still present...

[ThomasWaldmann]

Original comment by Reimar Bauer (Bitbucket: ReimarBauer, GitHub: ReimarBauer).

---

2012-07-15 nothing changed yet

[ThomasWaldmann]

Original comment by Reimar Bauer (Bitbucket: ReimarBauer, GitHub: ReimarBauer).

---

The workaround of Sam Toyer was added to the save extension,

https://bitbucket.org/ReimarBauer/xstatic-svg-edit-moin/changeset/fe5ca63c6adf795d85461da3305153b0

[RogerHaase]

closing as dup of #355

svgdraw is very broken, needs newer version of https://github.com/SVG-Edit/svgedit