You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by bu...@apache.org on 2021/01/06 11:49:17 UTC

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

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: dev@poi.apache.org
          Reporter: kiwiwings@apache.org
  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: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org


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

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=65061

Andreas Beeker <ki...@apache.org> changed:

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

--- Comment #1 from Andreas Beeker <ki...@apache.org> ---
applied via r1885197

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org