sw/qa/extras/ww8import/data/tdf128605.doc |binary
sw/qa/extras/ww8import/ww8import.cxx | 11 +++++++++++
sw/source/filter/ww8/ww8par.cxx | 5 ++---
3 files changed, 13 insertions(+), 3 deletions(-)
New commits:
commit 89deeb8940c8bc9f1ad0c88075df1afdd73d441a
Author: Miklos Vajna <vmik...@collabora.com>
AuthorDate: Tue Nov 26 16:53:33 2019 +0100
Commit: Miklos Vajna <vmik...@collabora.com>
CommitDate: Wed Nov 27 10:23:58 2019 +0100
tdf#128605 DOC import: fix cont sect break with different top/bottom margin
Word has a feature to attach different page margins to sections, and
then refer to these section definitions via continuous section breaks.
The result is that the margin changes never introduce a page break, and
left/right margins are applied instantly, while top/bottom margins are
applied from the next page.
Now in Writer, either these continuous section breaks are mapped to a
page break and all margins show up as expected, or we don't do a page
break, and then the number of page breaks are as expected; but you can't
have both at the same time till the feature is missing in Writer core.
Commit c5ed52b1cd6f22787c94bec035ceecf9e1da3271 (ww8import create a
pagedesc if continuous section changes margins (bnc#875383), 2014-07-21)
tweaked this decision, so that the DOC import always creates a page
break, but left the DOCX import unchanged.
Fix the bug by going back to not creating page breaks on top/bottom
margin changes: this resolves the scenario and still keeps the original
situation fixed, too.
(cherry picked from commit 162722dfa96dc10243e9a07a3c123f22481d42cd)
Change-Id: Id0e8c3a4ce40db8da024a0a20005eb0445325a1a
Reviewed-on: https://gerrit.libreoffice.org/83864
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmik...@collabora.com>
diff --git a/sw/qa/extras/ww8import/data/tdf128605.doc
b/sw/qa/extras/ww8import/data/tdf128605.doc
new file mode 100644
index 000000000000..2bbfe6b73033
Binary files /dev/null and b/sw/qa/extras/ww8import/data/tdf128605.doc differ
diff --git a/sw/qa/extras/ww8import/ww8import.cxx
b/sw/qa/extras/ww8import/ww8import.cxx
index a79781d781d3..c67f2c9385b0 100644
--- a/sw/qa/extras/ww8import/ww8import.cxx
+++ b/sw/qa/extras/ww8import/ww8import.cxx
@@ -94,6 +94,17 @@ DECLARE_WW8IMPORT_TEST(testTdf124601, "tdf124601.doc")
CPPUNIT_ASSERT(getProperty<bool>(getShapeByName("Grafik 19"),
"IsFollowingTextFlow"));
}
+DECLARE_WW8IMPORT_TEST(testTdf128605, "tdf128605.doc")
+{
+ OUString aPara1PageStyleName = getProperty<OUString>(getParagraph(1),
"PageStyleName");
+ OUString aPara2PageStyleName = getProperty<OUString>(getParagraph(2),
"PageStyleName");
+ // Without the accompanying fix in place, this test would have failed with:
+ // - Expected: Standard
+ // - Actual : Convert 1
+ // i.e. the continuous section break resulted in an unwanted page break.
+ CPPUNIT_ASSERT_EQUAL(aPara1PageStyleName, aPara2PageStyleName);
+}
+
DECLARE_WW8IMPORT_TEST(testTdf112535, "tdf112535.doc")
{
SwXTextDocument* pTextDoc = dynamic_cast<SwXTextDocument
*>(mxComponent.get());
diff --git a/sw/source/filter/ww8/ww8par.cxx b/sw/source/filter/ww8/ww8par.cxx
index ca2c97b2125d..eb7733f2879b 100644
--- a/sw/source/filter/ww8/ww8par.cxx
+++ b/sw/source/filter/ww8/ww8par.cxx
@@ -4534,9 +4534,8 @@ void wwSectionManager::InsertSegments()
bool bInsertSection = (aIter != aStart) && aIter->IsContinuous() &&
bThisAndPreviousAreCompatible;
bool bInsertPageDesc = !bInsertSection;
- // HACK Force new pagedesc if margins change, otherwise e.g. floating
tables may be anchored improperly.
- if( aIter->maSep.dyaTop != aPrev->maSep.dyaTop ||
aIter->maSep.dyaBottom != aPrev->maSep.dyaBottom
- || aIter->maSep.dxaLeft != aPrev->maSep.dxaLeft ||
aIter->maSep.dxaRight != aPrev->maSep.dxaRight )
+ // HACK Force new pagedesc if left/right margins change, otherwise
e.g. floating tables may be anchored improperly.
+ if( aIter->maSep.dxaLeft != aPrev->maSep.dxaLeft ||
aIter->maSep.dxaRight != aPrev->maSep.dxaRight )
bInsertPageDesc = true;
bool bProtected = SectionIsProtected(*aIter); // do we really need
this ?? I guess I have a different logic in editshell which disables this...
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
