| Summary: | "style:contextual-spacing" is not valid ODF 1.2 | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Rainer Bielefeld Retired <LibreOffice> |
| Component: | Writer | Assignee: | Eike Rathke <erack> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | eymux2009, jmadero.dev, LibreOffice, vmiklos |
| Priority: | medium | ||
| Version: | 3.6.4.3 release | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: | https://bugs.freedesktop.org/show_bug.cgi?id=45447 | ||
| Whiteboard: | BSA odf odf_validation target:4.3.0 target:4.2.0.2 target:4.1.5 | ||
| Crash report or crash signature: | Regression By: | ||
| Attachments: |
Test Kit with all mentioned documents
Validation Results with LibO 333 Falidation Results with LibO 3.6.4.3 |
||
|
Description
Rainer Bielefeld Retired
2012-12-11 05:34:01 UTC
I have no skills to check the document source. @Joel, eymux: can we do anything else before we forward to developers? Created attachment 71307 [details]
Validation Results with LibO 333
Document saved from server installation of "LibreOffice 3.5.7.2 rc German UI/Locale [Build-ID: 3215f89-f603614-ab984f2-7348103-1225a5b] on German WIN7 Home Premium (64bit) have been rated VALID, so this one is a regression Created attachment 71309 [details]
Falidation Results with LibO 3.6.4.3
For me the 3.6.4.3 results look very similar to the ones from 4.0, so I think the current invalidity has been introduced with 3.6.
Bibisecting of this would be really hard, so first we should be sure that the Validators recognized a real problem.
i get the same error here with ODFToolkit validator as the office-o-tron one in the attachment: Documents/issues/fdo#58112/out_master_12ext.odt/styles.xml[2,5103]: Error: unexpected attribute "style:contextual-spacing" this only happens when using ODF 1.2 extended (default) format; not when saving as 1.1 or 1.2. the attribute "style:contextual-spacing" is indeed not in ODF 1.2 but since LO doesn't write it when set to ODF 1.2 it's not really a problem. @Michael Stahl Thank you for the detailed analysis. I can confirm, Save as ODF 1.2 produces a valid document. I simply used the "1.2 Extended (recommended)" mode, and due to help this should only allow comments in Draw and Impress, I wonder why this also causes a syntax error in Writer. See http://wiki.documentfoundation.org/Development/ODF_Implementer_Notes#LibreOffice_ODF_extensions Contextual spacing is proposed to be part of the next ODF version, till then we just write it in extended ODF mode, so should be OK. *** Bug 58795 has been marked as a duplicate of this bug. *** Eike Rathke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=ef5e7b69440baa9f222d1ec870668d31d08268f0 prepare to accept loext:contextual-spacing, fdo#58112 The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Eike Rathke committed a patch related to this issue. It has been pushed to "libreoffice-4-2": http://cgit.freedesktop.org/libreoffice/core/commit/?id=116e214a82dcdce9d4764646ff43b60d5c7c9392&h=libreoffice-4-2 prepare to accept loext:contextual-spacing, fdo#58112 It will be available in LibreOffice 4.2. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Pending review for 4-1 at https://gerrit.libreoffice.org/7151 Eike Rathke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=d1ae39e484f888ddcca15f584cddd2b3ccae1024 fdo#58112 write loext:contextual-spacing accept style:contextual-spacing The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Eike Rathke committed a patch related to this issue. It has been pushed to "libreoffice-4-1": http://cgit.freedesktop.org/libreoffice/core/commit/?id=6c90074b8dcfbb10ab03acfa082ac746ea38df6e&h=libreoffice-4-1 prepare to accept loext:contextual-spacing, fdo#58112 It will be available in LibreOffice 4.1.5. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. |