Hello Carl, All, I am branching the thread from here in order to prune less interesting parts.
On Sun, Nov 14, 2021 at 10:12:05PM -0500, Carl Marcum wrote: > Hi All, > > On 11/14/21 5:05 PM, Matthias Seidel wrote: > > Hi Carl, > > > > Am 14.11.21 um 22:45 schrieb Carl Marcum: > > > Hi Matthias, > > > > > > > > > On 11/14/21 11:44 AM, Matthias Seidel wrote: > > > > Hi Carl, > > > > > > > > Am 14.11.21 um 17:36 schrieb Matthias Seidel: > > > > > Hi Carl, > > > > > > > > > > Am 14.11.21 um 17:31 schrieb Carl Marcum: > > > > > > Hi Matthias and All, > > > > > > > > > > > > On 11/14/21 9:10 AM, Matthias Seidel wrote: > > > > > > > Hi Carl, > > > > > > > > > > > > > > I only have a VM available with Xubuntu 18.04.6 and AOO42X, but > > > > > > > have no > > > > > > > problems opening and saving *.STW. > > > > > > Okay I'll build AOO42X and see if I have same issues as trunk. > > > > > > > > > > > > I recompiled trunk with > > > > > > --enable-crashdump=yes \ > > > > > > --enable-dbgutil \ > > > > > > > > > > > > and now I get a better error dialog for opening these file types: > > > > > > testSaveNewSTC.stc > > > > > > testSaveNewSXC.sxc > > > > > > ------------------ > > > > > > Error: uno exception caught while importing: > > > > > > attempt to insert duplicate XML tag attribute: xmlns:svg > > > > > > From File > > > > > > <build-path-to>/openoffice/main/sc/source/filter/xml/xmlwrap.cxx at > > > > > > Line 365 > > > > > > Abort ? (Yes=abort / No=ignore / Cancel=core dump) > > > > > I remember that we have a commit in trunk but not in AOO42X: > > > > > > > > > > https://github.com/apache/openoffice/pull/122 > > > > > > > > > > Until now it has not been tested... Maybe that is the problem? > > > > I can confirm Read-Errors with AOO450 (trunk) on Windows with STW and > > > > SXW. > > > > No problems with AOO420 (AOO42X). > > > > > > > > This merge may need to be reverted... > > > > > > Thanks for checking. > > > > > > It might be better at this point to fix the code. > > That's why I asked for testing... ;-) > > > I'll do some bisecting and try to narrow it down. > > I am not sure if the code was ready, Arrigo might know better. > > I bisected it down to this one where it starts giving me the error which > makes sense according the the commit message: > > ebd28ad3a7fedadb378ee1596803008e8c6b383b is the first bad commit > commit ebd28ad3a7fedadb378ee1596803008e8c6b383b > > Raise an exception when attempting to add duplicate attributes The error is raised during parsing of file styles.xml, initial tag <office:document>, attribute xmlns:svg="http://openoffice.org/2000/svg"; Method XMLDocumentTransformerContext_Impl::StartElement() considers individual XML attributes and, at the end, it adds the ones it considers missing. The xmlns:svg is considered missing because it does not have the expected _content_. Yes, the check is made on attribute contents, rather than attribute names. That attributs's content _should_ be the one indicated above, but the caller method XMLTransformerBase::startElement() has _just changed its content_ into "urn:oasis:names:tc:opendocument:xmlns:svg-compatible:1.0" For this reason, method XMLDocumentTransformerContext_Impl::StartElement() tries to re-add the "xmlns:svg" element. The first solutions that come into my mind are: 1- have XMLDocumentTransformerContext_Impl::StartElement() consider element _names_ rather than values, or 2- have XMLDocumentTransformerContext_Impl::StartElement() look for the ``changed'' content (urn:etcetera) instead of the ``original'' one (http://Etcetera) What is the best way to go? Best regards, -- Arrigo --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org
