Exporting to XAML doesn't use culture independent decimal sign.

Bug #380199 reported by jonathan.slenders
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Low
jazzynico

Bug Description

When in the "Regional and Language Options" dialog of Windows' control panel has been chosen for a number format with kommas instead of points as decimal, The generated XAML file will also contain kommas which are obviously not valid.

Tags: exporting xaml
Revision history for this message
jazzynico (jazzynico) wrote :

Can't reproduce it on Windows XP sp3, Inkscape 0.46 and rev. 21303.
Could you please report your Inkscape version and the system you use?
Could you also attach an example SVG file and the resulting XAML so that we can analyze it?
Thanks.

Changed in inkscape:
importance: Undecided → Low
status: New → Incomplete
tags: added: exporting xaml
Revision history for this message
jonathan.slenders (jonathan-slenders) wrote :

Find the example files in the attachment.

The SVG is converted to XAML with
- Window Vista Home Premium.
- Regional and Language options, Current Format="Dutch (Belgium)"
- Inkscape 0.46

Revision history for this message
jazzynico (jazzynico) wrote :

Reproduced on Windows XP sp3, Inkscape 0.46 and rev. 21424.
The bug affects circle, ellipse, polyline and polygon, but not path, rect and line elements.
I've tested with a point as decimal in the system options, and I get the same bug.

Changed in inkscape:
status: Incomplete → Confirmed
Revision history for this message
jazzynico (jazzynico) wrote :

Correction, only circle and ellipse are affected.

Revision history for this message
jazzynico (jazzynico) wrote :

Correction again: it IS affected by language options.
Sorry for the errors in my tests.
To sum up, when a comma is defined as a decimal in the system language options, points in circles and ellipses are converted to commas in the exported XAML.

Revision history for this message
jazzynico (jazzynico) wrote :

The attached patch seems to resolve this bug.
It's a very simple change in svg2xaml.xsl. I just replaced strings that result from a math operation by the format-number(number, "#.#") XSLT function . As I'm not the author of this extension and I can't connect to IRC, it would be nice if someone could check and validate this patch.
Thanks.

Changed in inkscape:
assignee: nobody → JazzyNico (jazzynico)
milestone: none → 0.47
status: Confirmed → In Progress
Revision history for this message
jazzynico (jazzynico) wrote :

Fix committed in rev. 21534. Please test and report.

Changed in inkscape:
status: In Progress → Fix Committed
ScislaC (scislac)
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.