Windows: '/' in relative image links rewritten as '\' on save

confirmed on Windows 10, Inkscape 0.92.0 r15299

if I load a jpg file using a link, then the XML editor initially reports the link as:
xlink:href = file:///C:/Windows/Temp/cathedral.jpg

after saving the file, the link is shown as:
sodipodi:absref="C:\Windows\Temp\cathedral.jpg"
xlink:href="cathedral.jpg"

Please note that the core of the report focuses on the value in the xlink:href attribute. The sodipodi:absref attribute in the custom namespace in my understanding mostly matters on the local platform of the author (absolute links to files on a local hard drive rarely work on other systems). A relative image link in the xlink:href attribute on the other hand is not only used by Inkscape (not restricted to a private namespace used by Inkscape), and AFAIU should not use a OS-specific path separator.

Also reproduced on Windows XP (32-bit), lp:inkscape/0.92.x rev. 15346.

Typo introduced rev. 10199 (see http://bazaar.launchpad.net/~inkscape.dev/inkscape/0.92.x/revision/10199#src/xml/rebase-hrefs.cpp).
Patch in progress.

Patch attached, please test.

I'm afraid it's a bit too late for 0.92.1, so I'm targeting it to 0.92.2 for now.

+1 Approved for landing on 0.92.x branch for inclusion in 0.92.1.

I would like to see testing of this to ensure the issue is fully resolved, but the fix looks straightforward enough.

Remember to land the change with a commit message that explains the user-visible issue, and ref this bug #.

> I would like to see testing of this to ensure the issue is fully resolved.

It was fully tested on Windows XP (32-bit), with the 0.92.x branch (rev. 15346).

Seems to work fine on Windows 10, Inkscape 0.92.1pre0 15346 (64-bit)

There are two minor glitches (but I doubt they're caused by this change?):
1. Assume an image file "E:\test\raster.png"
2. Open a new document, embed raster.png and save to "E:\test\drawing.svg". Although the file is in the same directory the full path is saved 'xlink:href="file:///E:/test/raster.png"'. One can avoid this by saving the file once before embedding raster.png
3. Immediately after saving to "E:\test\drawing.svg" in 2. save the file again to ""E:\drawing.svg". You will now end up with 'sodipodi:absref="E:\test/raster.png"' in the saved file.

@jazzynico: You had a look at the relevant code lately. Maybe you can check it out? I guess at least 2 is worth figuring out (for future versions) while most likely not a regression. 3. is obviously only a minor thing.

Thanks for your test Eduard.
If I remember correctly, the path in 2 is automatically fixed when saving the file a second time (to be confirmed). I'm going to grab the remaining issues (some where reported in Bug #170225 'relative image paths instead of absolute' - which was never fixed in a released version due to the regression introduced rev.10199).

Fixed in lp:inkscape rev. 15459.

Fixed in lp:inkscape/0.92.x rev. 15347.

@Eduard, just tested again issue #2. The file is correctly saved (checked the SVG directly with a text editor), and thus it's just that the path is not correctly refreshed in the application.

Issue #2 again: the SVG file is correct, so there's no need to save twice (you can just reload), but the xlink is still invalid in the Object attribute dialog and in the XML editor.

Regarding comment #9, all related bugs from Bug #170225 seem to be fixed in 0.92.

I definitely saw #2 in the saved SVG file (not the XML editor). Are you sure you did not save twice?

Yes, I checked on a Windows 8 computer, and checked again on Xubuntu to be sure. To be sure, try to save once and revert, and you should have the expected values.