tdf#155682 sw floattable: fix DOCX with big pictures causes endless loop
The bugdoc had a floating table in the header that overlapped with
full-page, as-char anchored images in the body text, leading to a layout
loop.
The trouble is that the body text was broken into 2 lines: the first
line just had fly portions for the area where the in-header floating
table was rendered and then the second line hosted the actual image. But
then this image didn't fit the remaining space, so it moves to the next
page. And this happened again and again.
Fix the problem by keeping in-header anchored floating tables inside the
boundary of the header, this way the body text still doesn't overlap
with the floating table, but the space available to the body text
doesn't have to contain fly portions. Which means we don't try to move
the image to a next page, so there is no loop.
Note that the IsFollowingTextFlow flag is already ignored in some cases
(e.g. sw/qa/extras/ww8export/data/tdf128700_relativeTableWidth.doc), so
this doesn't break the use-case when a footer provides an anchor
position for some text on the left/right margin. In that case the footer
height is still unchanged as it should be.
Change-Id: Id9e80140af3123d52b0fea2f96fc19c150c8e736
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/158413
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
diff --git a/writerfilter/qa/cppunittests/dmapper/DomainMapperTableHandler.cxx b/writerfilter/qa/cppunittests/dmapper/DomainMapperTableHandler.cxx
index 34fee71..5046be8 100644
--- a/writerfilter/qa/cppunittests/dmapper/DomainMapperTableHandler.cxx
+++ b/writerfilter/qa/cppunittests/dmapper/DomainMapperTableHandler.cxx
@@ -17,6 +17,8 @@
#include <com/sun/star/drawing/XDrawPageSupplier.hpp>
#include <com/sun/star/text/XTextDocument.hpp>
#include <com/sun/star/style/BreakType.hpp>
#include <com/sun/star/text/XTextViewCursorSupplier.hpp>
#include <com/sun/star/text/XPageCursor.hpp>
using namespace ::com::sun::star;
@@ -160,6 +162,27 @@ CPPUNIT_TEST_FIXTURE(Test, testDOCXFloatingTableNested)
// split.
CPPUNIT_ASSERT(bIsSplitAllowed);
}
CPPUNIT_TEST_FIXTURE(Test, testDOCXFloatingTableHeader)
{
// Given a document with a header that has a floating table and some large images in the body
// text:
loadFromURL(u"floattable-header.docx");
// When breaking that document into pages:
uno::Reference<frame::XModel> xModel(mxComponent, uno::UNO_QUERY);
uno::Reference<text::XTextViewCursorSupplier> xTextViewCursorSupplier(
xModel->getCurrentController(), uno::UNO_QUERY);
uno::Reference<text::XPageCursor> xCursor(xTextViewCursorSupplier->getViewCursor(),
uno::UNO_QUERY);
xCursor->jumpToLastPage();
// Then make sure we get 2 pages:
sal_Int32 nLastPage = xCursor->getPage();
// Without the accompanying fix in place, this test would have failed, the page count went to
// 2233 pages and then there was a layout loop.
CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(2), nLastPage);
}
}
CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/writerfilter/qa/cppunittests/dmapper/data/floattable-header.docx b/writerfilter/qa/cppunittests/dmapper/data/floattable-header.docx
new file mode 100644
index 0000000..1042dfc
--- /dev/null
+++ b/writerfilter/qa/cppunittests/dmapper/data/floattable-header.docx
Binary files differ
diff --git a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
index 2d1c4f3..2092ba4 100644
--- a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
@@ -1580,9 +1580,11 @@ void DomainMapperTableHandler::endTable(unsigned int nestedTableLevel)
xTableProperties->setPropertyValue("BreakType", uno::Any(style::BreakType_NONE));
}
if (nestedTableLevel >= 2)
if (nestedTableLevel >= 2 || m_rDMapper_Impl.IsInHeaderFooter())
{
// Floating tables inside a table always stay inside the cell.
// Also extend the header/footer area if needed, so an in-header floating table
// typically doesn't overlap with body test.
aFrameProperties.push_back(
comphelper::makePropertyValue("IsFollowingTextFlow", true));
}
