editeng/source/editeng/impedit2.cxx | 6 ++++--
editeng/source/xml/xmltxtimp.cxx | 5 ++++-
sd/qa/unit/tiledrendering/tiledrendering.cxx | 14 ++++++++++----
3 files changed, 18 insertions(+), 7 deletions(-)
New commits:
commit 66ed51712bb323bba7d1a0b0d21b256e5f5f9ccf
Author: Katarina Behrens <katarina.behr...@cib.de>
AuthorDate: Fri Jan 17 14:30:56 2020 +0100
Commit: Katarina Behrens <katarina.behr...@cib.de>
CommitDate: Thu Jan 23 15:01:33 2020 +0100
tdf#128046: preserve list levels on paste
done in two steps:
1) bullet and numbering properties must be included in the property
list of SvxUnoText on import from ODF clipboard (SvxReadXML)
2) in ConnectParagraphs, don't merge para attributes if the paragraph
to be deleted is empty. Paragraph breaks are inserted before paste so
this would make the last list level of the pasted text take the properties
of the empty paragraph and thus lose its indentation
Also updated tdf#103083 unit test to be more specific and test that
bullet properties don't get lost after paste. Previously only count
of SfxItems was tested which is somehow fragile and fails bc paste
adds a paragraph style
Change-Id: Ia162adb7abf145a8217f21c7ebfe06b69731e2a5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/86983
Tested-by: Jenkins
Reviewed-by: Katarina Behrens <katarina.behr...@cib.de>
diff --git a/editeng/source/editeng/impedit2.cxx
b/editeng/source/editeng/impedit2.cxx
index 64c9f10f5135..30e12bec9a56 100644
--- a/editeng/source/editeng/impedit2.cxx
+++ b/editeng/source/editeng/impedit2.cxx
@@ -2232,7 +2232,9 @@ EditPaM ImpEditEngine::ImpConnectParagraphs( ContentNode*
pLeft, ContentNode* pR
if ( bBackward )
{
pLeft->SetStyleSheet( pRight->GetStyleSheet() );
- pLeft->GetContentAttribs().GetItems().Set(
pRight->GetContentAttribs().GetItems() );
+ // it feels wrong to set pLeft's attribs if pRight is empty, tdf#128046
+ if ( pRight->Len() )
+ pLeft->GetContentAttribs().GetItems().Set(
pRight->GetContentAttribs().GetItems() );
pLeft->GetCharAttribs().GetDefFont() =
pRight->GetCharAttribs().GetDefFont();
}
@@ -3489,7 +3491,7 @@ uno::Reference< datatransfer::XTransferable >
ImpEditEngine::CreateTransferable(
std::filebuf afilebuf;
afilebuf.open ("gsoc17_clipboard_test.xml",std::ios::out);
std::ostream os(&afilebuf);
- os.write((const char*)(pDataObj->GetODFStream().GetBuffer()),
pDataObj->GetODFStream().remainingSize());
+ os.write((const char*)(pDataObj->GetODFStream().GetData()),
pDataObj->GetODFStream().remainingSize());
afilebuf.close();
*/
//dumping ends
diff --git a/editeng/source/xml/xmltxtimp.cxx b/editeng/source/xml/xmltxtimp.cxx
index acd86b77cfee..f0bb5d171ceb 100644
--- a/editeng/source/xml/xmltxtimp.cxx
+++ b/editeng/source/xml/xmltxtimp.cxx
@@ -138,7 +138,10 @@ EditPaM SvxReadXML( EditEngine& rEditEngine, SvStream&
rStream, const ESelection
{
SVX_UNOEDIT_CHAR_PROPERTIES,
SVX_UNOEDIT_FONT_PROPERTIES,
-// SVX_UNOEDIT_OUTLINER_PROPERTIES,
+// bullets & numbering props, tdf#128046
+ { OUString(UNO_NAME_NUMBERING_RULES), EE_PARA_NUMBULLET,
cppu::UnoType<css::container::XIndexReplace>::get(), 0, 0 },
+ { OUString(UNO_NAME_NUMBERING),
EE_PARA_BULLETSTATE,cppu::UnoType<bool>::get(), 0, 0 },
+ { OUString(UNO_NAME_NUMBERING_LEVEL), EE_PARA_OUTLLEVEL,
::cppu::UnoType<sal_Int16>::get(), 0, 0 },
SVX_UNOEDIT_PARA_PROPERTIES,
{ OUString(), 0, css::uno::Type(), 0, 0 }
};
diff --git a/sd/qa/unit/tiledrendering/tiledrendering.cxx
b/sd/qa/unit/tiledrendering/tiledrendering.cxx
index 1e40d9645627..2fe192f66de9 100644
--- a/sd/qa/unit/tiledrendering/tiledrendering.cxx
+++ b/sd/qa/unit/tiledrendering/tiledrendering.cxx
@@ -24,6 +24,7 @@
#include <editeng/editids.hrc>
#include <editeng/editobj.hxx>
#include <editeng/editview.hxx>
+#include <editeng/numitem.hxx>
#include <editeng/outliner.hxx>
#include <editeng/fhgtitem.hxx>
#include <editeng/outlobj.hxx>
@@ -1428,8 +1429,9 @@ void SdTiledRenderingTest::testTdf103083()
SdrOutliner* pOutliner = pView->GetTextEditOutliner();
CPPUNIT_ASSERT_EQUAL(OUString("No-Logo Content~LT~Gliederung 2"),
pOutliner->GetStyleSheet(2)->GetName());
- const SfxItemSet& rParagraphItemSet1 =
pTextObject->GetOutlinerParaObject()->GetTextObject().GetParaAttribs(2);
- CPPUNIT_ASSERT_EQUAL(sal_uInt16(3), rParagraphItemSet1.Count());
+ const EditTextObject& aEdit =
pTextObject->GetOutlinerParaObject()->GetTextObject();
+ const SvxNumBulletItem* pNumFmt =
aEdit.GetParaAttribs(2).GetItem(EE_PARA_NUMBULLET);
+ SvxNumberFormat aNumFmt(pNumFmt->GetNumRule()->GetLevel(2));
// cut contents of bullet item
comphelper::dispatchCommand(".uno:Cut",
uno::Sequence<beans::PropertyValue>());
@@ -1456,8 +1458,12 @@ void SdTiledRenderingTest::testTdf103083()
CPPUNIT_ASSERT_EQUAL(OUString("No-Logo Content~LT~Gliederung 2"),
pOutliner->GetStyleSheet(2)->GetName());
- const SfxItemSet& rParagraphItemSet2 =
pTextObject->GetOutlinerParaObject()->GetTextObject().GetParaAttribs(2);
- CPPUNIT_ASSERT_EQUAL(sal_uInt16(3), rParagraphItemSet2.Count());
+ const EditTextObject& aEdit2 =
pTextObject->GetOutlinerParaObject()->GetTextObject();
+ const SvxNumBulletItem* pNumFmt2 =
aEdit2.GetParaAttribs(2).GetItem(EE_PARA_NUMBULLET);
+ SvxNumberFormat aNumFmt2(pNumFmt2->GetNumRule()->GetLevel(2));
+
+ bool bEqual(aNumFmt2 == aNumFmt);
+ CPPUNIT_ASSERT_MESSAGE("Bullet properties changed after paste", bEqual);
}
/**
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
