sc/qa/unit/scshapetest.cxx | 8 ------- sc/qa/unit/uicalc/uicalc.cxx | 4 --- schema/libreoffice/OpenDocument-v1.3+libreoffice-schema.rng | 13 ++++++++++++ 3 files changed, 13 insertions(+), 12 deletions(-)
New commits: commit 1d2830b936d02ed0882d9cc1234ddafb3bb7e8dd Author: Regina Henschel <rb.hensc...@t-online.de> AuthorDate: Mon May 22 15:57:38 2023 +0200 Commit: Regina Henschel <rb.hensc...@t-online.de> CommitDate: Mon May 22 19:40:45 2023 +0200 tdf#155271 add drawooo:display to libreoffice-schema Commit 33ef17ed had introduced attribute drawooo:display at 2010-03-02. The entry in libreoffice-schema.rng was missing and therefore validation in unit tests fails, when this attribute occures. The attribute is written in case an object in Calc is hidden by hiding the cell to which it is anchored, for example. The attribute belongs to properties 'Visibile' and 'Printable' in 'Shape Service' in API. Change-Id: Id3d2e7984aed146fca9e2819eb183d7d7bfaf701 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/152119 Reviewed-by: Michael Stahl <michael.st...@allotropia.de> Tested-by: Jenkins Reviewed-by: Regina Henschel <rb.hensc...@t-online.de> diff --git a/sc/qa/unit/scshapetest.cxx b/sc/qa/unit/scshapetest.cxx index b658e44f03d5..b5083544f114 100644 --- a/sc/qa/unit/scshapetest.cxx +++ b/sc/qa/unit/scshapetest.cxx @@ -890,10 +890,6 @@ CPPUNIT_TEST_FIXTURE(ScShapeTest, testTdf115655_HideDetail) pViewShell->GetViewData().GetDispatcher().Execute(SID_OUTLINE_HIDE); CPPUNIT_ASSERT_MESSAGE("Collapse: Image should not be visible", !pObj->IsVisible()); - // FIXME: validation fails with - // Error: unexpected attribute "drawooo:display" - skipValidation(); - // Save and reload saveAndReload("calc8"); @@ -1191,10 +1187,6 @@ CPPUNIT_TEST_FIXTURE(ScShapeTest, testTdf154821_shape_in_group) pViewShell->GetViewData().SetCurY(1); pViewShell->GetViewData().GetDispatcher().Execute(SID_OUTLINE_HIDE); - // Save and reload - // FIXME: validation fails with - // Error: unexpected attribute "drawooo:display" - skipValidation(); saveAndReload("calc8"); // Expand the lower group diff --git a/sc/qa/unit/uicalc/uicalc.cxx b/sc/qa/unit/uicalc/uicalc.cxx index fae0b389584e..daccc08c2600 100644 --- a/sc/qa/unit/uicalc/uicalc.cxx +++ b/sc/qa/unit/uicalc/uicalc.cxx @@ -904,10 +904,6 @@ CPPUNIT_TEST_FIXTURE(ScUiCalcTest, testTdf144244) CPPUNIT_ASSERT_EQUAL(OUString("x"), pDoc->GetString(ScAddress(0, 0, 0))); - // FIXME: validation fails with - // Error: unexpected attribute "drawooo:display" - skipValidation(); - // Without the fix in place, this test would have crashed saveAndReload("calc8"); pModelObj = comphelper::getFromUnoTunnel<ScModelObj>(mxComponent); diff --git a/schema/libreoffice/OpenDocument-v1.3+libreoffice-schema.rng b/schema/libreoffice/OpenDocument-v1.3+libreoffice-schema.rng index 0436448644bb..2466b797933c 100644 --- a/schema/libreoffice/OpenDocument-v1.3+libreoffice-schema.rng +++ b/schema/libreoffice/OpenDocument-v1.3+libreoffice-schema.rng @@ -3620,4 +3620,17 @@ xmlns:loext="urn:org:documentfoundation:names:experimental:office:xmlns:loext:1. </rng:element> </rng:define> + <!-- TODO no proposal, belongs to commit 33ef17ed, 2010-03-02 --> + <rng:define name="common-draw-shape-with-styles-attlist" combine="interleave"> + <rng:optional> + <rng:attribute name="drawooo:display"> + <rng:choice> + <rng:value>always</rng:value> + <rng:value>screen</rng:value> + <rng:value>printer</rng:value> + <rng:value>none</rng:value> + </rng:choice> + </rng:attribute> + </rng:optional> + </rng:define> </rng:grammar>