diff options
author | Mike Kaganski <mike.kaganski@collabora.com> | 2021-07-15 01:14:15 +0300 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2021-07-15 08:58:21 +0200 |
commit | 6c411afae7f6cd9819c1b65d7b4212019a7b3cfe (patch) | |
tree | e50e105bd8f41f3f173a1d1f193f32a258db061f /sd/qa | |
parent | 33e4ad909b54de433e7f73a87348505bd3fd6624 (diff) |
tdf#143315: restore the stylesheet after clearing outliner object
Prior to commit c4f615b359be56e88e4fbf9aaaf30affb29d57e2, pObj passed
to SvxMSDffManager::ReadObjText was not initializing its properties
(no call to AttributeProperties::GetObjectItemSet() happened) until
call to CreateParaObject in the end of the function. Then it finally
initialized, and called applyDefaultStyleSheetFromSdrModel, which
applied the obtained stylesheet to the nodes of the outliner object,
and resulted in OutlinerParaObject having correct style name in its
ContentInfo structures, which then went into pObj.
The mentioned commit added calls to SdrTextObj::GetTextColumns* in
SdrOutliner::SetTextObj, called by SdrTextObj::ImpGetDrawOutliner
indirectly. These calls initialize the object properties, and the
stylesheet gets applied to the SdrOutliner and its empty node early.
Then the call to Outliner::Init and Outliner::Clear resulted in the
node being destroyed and re-created without stylesheet info. Then in
AttributeProperties::GetObjectItemSet called from CreateParaObject,
re-initialization does not happen, because the item set exists.
When exporting to OOXML, the missing stylesheet required to use pool
default value, which led to bullet being exported.
The previous behavior relied on fragile and unspecified assumption.
Instead, now we call SetStyleSheet after all operations, to make sure
that the stylesheet information is present in the final data.
Change-Id: I14de9017e4af92a2eaf12b3a0e090b0db7fcc759
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118961
Tested-by: Mike Kaganski <mike.kaganski@collabora.com>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118963
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Diffstat (limited to 'sd/qa')
-rw-r--r-- | sd/qa/unit/data/ppt/tdf143315-WordartWithoutBullet.ppt | bin | 0 -> 68608 bytes | |||
-rw-r--r-- | sd/qa/unit/export-tests-ooxml2.cxx | 26 |
2 files changed, 26 insertions, 0 deletions
diff --git a/sd/qa/unit/data/ppt/tdf143315-WordartWithoutBullet.ppt b/sd/qa/unit/data/ppt/tdf143315-WordartWithoutBullet.ppt Binary files differnew file mode 100644 index 000000000000..54111bb28a7a --- /dev/null +++ b/sd/qa/unit/data/ppt/tdf143315-WordartWithoutBullet.ppt diff --git a/sd/qa/unit/export-tests-ooxml2.cxx b/sd/qa/unit/export-tests-ooxml2.cxx index b80078fe7844..759fa8e203e5 100644 --- a/sd/qa/unit/export-tests-ooxml2.cxx +++ b/sd/qa/unit/export-tests-ooxml2.cxx @@ -215,6 +215,7 @@ public: void testTextColumns_3columns(); void testTdf59323_slideFooters(); void testTdf143222_embeddedWorksheet(); + void testTdf143315(); CPPUNIT_TEST_SUITE(SdOOXMLExportTest2); @@ -342,6 +343,7 @@ public: CPPUNIT_TEST(testTextColumns_3columns); CPPUNIT_TEST(testTdf59323_slideFooters); CPPUNIT_TEST(testTdf143222_embeddedWorksheet); + CPPUNIT_TEST(testTdf143315); CPPUNIT_TEST_SUITE_END(); @@ -3319,6 +3321,30 @@ void SdOOXMLExportTest2::testTdf143222_embeddedWorksheet() xDocShRef->DoClose(); } +void SdOOXMLExportTest2::testTdf143315() +{ + auto xDocShRef = loadURL( + m_directories.getURLFromSrc(u"sd/qa/unit/data/ppt/tdf143315-WordartWithoutBullet.ppt"), + PPT); + + utl::TempFile tmpfile; + xDocShRef = saveAndReload(xDocShRef.get(), PPTX, &tmpfile); + xDocShRef->DoClose(); + + xmlDocUniquePtr pXml = parseExport(tmpfile, "ppt/slides/slide1.xml"); + + // Without the fix in place, whis would have failed with + // - Expected: + // - Actual : 216000 + // - In <file:///tmp/lu161922zcvd.tmp>, XPath '/p:sld/p:cSld/p:spTree/p:sp/p:txBody/a:p/a:pPr' unexpected 'marL' attribute + + assertXPathNoAttribute(pXml, "/p:sld/p:cSld/p:spTree/p:sp/p:txBody/a:p/a:pPr", "marL"); + assertXPathNoAttribute(pXml, "/p:sld/p:cSld/p:spTree/p:sp/p:txBody/a:p/a:pPr", "indent"); + assertXPath(pXml, "/p:sld/p:cSld/p:spTree/p:sp/p:txBody/a:p/a:pPr/a:buClr", 0); + assertXPath(pXml, "/p:sld/p:cSld/p:spTree/p:sp/p:txBody/a:p/a:pPr/a:buSzPct", 0); + assertXPath(pXml, "/p:sld/p:cSld/p:spTree/p:sp/p:txBody/a:p/a:pPr/a:buFont", 0); + assertXPath(pXml, "/p:sld/p:cSld/p:spTree/p:sp/p:txBody/a:p/a:pPr/a:buChar", 0); +} CPPUNIT_TEST_SUITE_REGISTRATION(SdOOXMLExportTest2); |