sw/qa/extras/ooxmlexport/data/tdf112446_frameStyle.docx |binary
sw/qa/extras/ooxmlexport/ooxmlexport9.cxx | 5 +++++
writerfilter/source/dmapper/DomainMapper.cxx | 2 ++
3 files changed, 7 insertions(+)
New commits:
commit b642999beb5c404cb0ea3df3741506e512c19f55
Author: Justin Luth <justin_l...@sil.org>
Date: Tue Sep 19 14:53:41 2017 -0400
tdf#112446 ooxmlimport: Orient=NONE when distance is given
Prior to commit 9920a0bf9d783978cd6f7b97f7528d8aa2571143
the style could only contain the default of NONE. So when
a position was specified, it was always paired with
HoriOrient == NONE. So it never caused problems until
that commit when the Frame's style orientation started
overriding the unset paragraph default.
When a position is specified, that needs to be paired with an
orientation of NONE in order to take effect.
Change-Id: Iab0057810270ba708a8855c2ec6db291cef17cfb
Reviewed-on: https://gerrit.libreoffice.org/42499
Tested-by: Jenkins <c...@libreoffice.org>
Reviewed-by: Miklos Vajna <vmik...@collabora.co.uk>
diff --git a/sw/qa/extras/ooxmlexport/data/tdf112446_frameStyle.docx
b/sw/qa/extras/ooxmlexport/data/tdf112446_frameStyle.docx
new file mode 100644
index 000000000000..7c92ae36f411
Binary files /dev/null and
b/sw/qa/extras/ooxmlexport/data/tdf112446_frameStyle.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
b/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
index 99edeb2b8dcd..b4731f41de50 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
@@ -586,6 +586,11 @@ DECLARE_OOXMLEXPORT_TEST(testTdf37153,
"tdf37153_considerWrapOnObjPos.docx")
CPPUNIT_ASSERT_MESSAGE("TextTop should be 3856", nTextTop > 3000);
}
+DECLARE_OOXMLEXPORT_TEST(testTdf112446_frameStyle, "tdf112446_frameStyle.docx")
+{
+ CPPUNIT_ASSERT_EQUAL(text::HoriOrientation::NONE,
getProperty<sal_Int16>(getShape(1), "HoriOrient"));
+}
+
DECLARE_OOXMLEXPORT_TEST(testTdf82173_footnoteStyle,
"tdf82173_footnoteStyle.docx")
{
uno::Reference<text::XFootnotesSupplier> xFootnotesSupplier(mxComponent,
uno::UNO_QUERY);
diff --git a/writerfilter/source/dmapper/DomainMapper.cxx
b/writerfilter/source/dmapper/DomainMapper.cxx
index fc83d8d31af8..ab3e53c9d93d 100644
--- a/writerfilter/source/dmapper/DomainMapper.cxx
+++ b/writerfilter/source/dmapper/DomainMapper.cxx
@@ -765,6 +765,7 @@ void DomainMapper::lcl_attribute(Id nName, Value & val)
break;
case NS_ooxml::LN_CT_FramePr_x:
pParaProperties->Setx(
ConversionHelper::convertTwipToMM100(nIntValue ));
+ pParaProperties->SetxAlign(
text::HoriOrientation::NONE );
break;
case NS_ooxml::LN_CT_FramePr_xAlign:
switch( nIntValue )
@@ -780,6 +781,7 @@ void DomainMapper::lcl_attribute(Id nName, Value & val)
break;
case NS_ooxml::LN_CT_FramePr_y:
pParaProperties->Sety(
ConversionHelper::convertTwipToMM100(nIntValue ));
+ pParaProperties->SetyAlign(
text::VertOrientation::NONE );
break;
case NS_ooxml::LN_CT_FramePr_yAlign:
switch( nIntValue )
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
