[Bug 65061] New: Handle VmlDrawings containing spreadsheet-ml default namespace

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 65061] New: Handle VmlDrawings containing spreadsheet-ml default namespace

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=65061

            Bug ID: 65061
           Summary: Handle VmlDrawings containing spreadsheet-ml default
                    namespace
           Product: POI
           Version: 4.1.x-dev
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: XSSF
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

Created attachment 37691
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=37691&action=edit
excel containing vmldrawing with invalid default namespace

The regression test came up with a few failing documents containing
vmlDrawing1.xmls which have a spreadsheet-ml default namespace [1].

This is wrong and needs to be workaround - i.e. vml drawings have no root
namespace [2].

I can think of three ways to workaround it:
a) remove the namespace before parsing the file
b) replace the namespace in the loaded XmlObject
c) keep it inside and ignore it

I've picked a).
contra b): works, but the document is modified after it's validated.
contra c): writing the drawing back again keeps the erroneous declaration.
and the XmlBeans accessor doesn't work too.

I still need to test my modifications, but the fix will be applied soon.


[1] http://schemas.openxmlformats.org/spreadsheetml/2006/main

[2]
https://c-rex.net/projects/samples/ooxml/e1/Part1/OOXML_P1_Fundamentals_VML_topic_ID0EZ5EO.html

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 65061] Handle VmlDrawings containing spreadsheet-ml default namespace

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=65061

Andreas Beeker <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #1 from Andreas Beeker <[hidden email]> ---
applied via r1885197

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]