diff --git a/0001-writerfilter-DOCX-import-fix-handling-of-w-hideMark-.patch b/0001-writerfilter-DOCX-import-fix-handling-of-w-hideMark-.patch new file mode 100644 index 0000000..df138c3 --- /dev/null +++ b/0001-writerfilter-DOCX-import-fix-handling-of-w-hideMark-.patch @@ -0,0 +1,47 @@ +From baec88396e0f1910ac3d4d1a1f87b45a90ba0342 Mon Sep 17 00:00:00 2001 +From: Michael Stahl +Date: Thu, 30 Mar 2017 23:11:41 +0200 +Subject: [PATCH] writerfilter: DOCX import: fix handling of w:hideMark vs. + w:vMerge + +The problem is that Writer's layout can't handle the case where cells +are vertically merged and the last row has a fixed height; the vertically +merged cell will grow up to the height of the other cells in the non- +fixed rows plus the fixed row height, but no larger. + +So for now, avoid setting fixed row heights in this case. + +(regression from d1278ef4849661b9ae0eb7aaf4d74fbf91ccaf11) + +(cherry picked from commit c382c998ffdaf80c10a3f078fb4f0a37224d1158) +Reviewed-on: https://gerrit.libreoffice.org/35960 +Tested-by: Jenkins +Reviewed-by: Miklos Vajna +(cherry picked from commit 7d7d21cfa53c8e80fd4dd0938579d8377da5a840) + +Change-Id: Iac3689e0bb0d5b8a62115ca0fb1f2c553a6e6bbc +--- + writerfilter/source/dmapper/DomainMapperTableHandler.cxx | 6 +++++- + 1 file changed, 5 insertions(+), 1 deletion(-) + +diff --git a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx +index d21f5f7..dbe7cc5 100644 +--- a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx ++++ b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx +@@ -870,8 +870,12 @@ CellPropertyValuesSeq_t DomainMapperTableHandler::endTableGetCellProperties(Tabl + bool lcl_hideMarks(PropertyMapVector1& rCellProperties) + { + for (size_t nCell = 0; nCell < rCellProperties.size(); ++nCell) +- if (!rCellProperties[nCell]->isSet(PROP_CELL_HIDE_MARK)) ++ { ++ // if anything is vertically merged, the row must not be set to fixed ++ // as Writer's layout doesn't handle that well ++ if (!rCellProperties[nCell]->isSet(PROP_CELL_HIDE_MARK) || rCellProperties[nCell]->isSet(PROP_VERTICAL_MERGE)) + return false; ++ } + return true; + } + +-- +2.9.3 + diff --git a/libreoffice.spec b/libreoffice.spec index 5df798a..bb3a84c 100644 --- a/libreoffice.spec +++ b/libreoffice.spec @@ -61,7 +61,7 @@ Summary: Free Software Productivity Suite Name: libreoffice Epoch: 1 Version: %{libo_version}.2 -Release: 7%{?libo_prerelease}%{?dist} +Release: 8%{?libo_prerelease}%{?dist} License: (MPLv1.1 or LGPLv3+) and LGPLv3 and LGPLv2+ and BSD and (MPLv1.1 or GPLv2 or LGPLv2 or Netscape) and Public Domain and ASL 2.0 and Artistic and MPLv2.0 and CC0 URL: http://www.libreoffice.org/ @@ -310,6 +310,7 @@ Patch69: 0001-avoid-loss-of-text-in-edited-placeholder.patch Patch70: 0001-tdf-96522-ANIMATION-Animation-Spiral-in-does-not-wor.patch Patch71: 0001-Resolves-tdf-103493-copying-note-captions-needs-a-co.patch Patch72: 0001-Resolves-tdf-105968-handle-engineering-notation-roun.patch +Patch73: 0001-writerfilter-DOCX-import-fix-handling-of-w-hideMark-.patch %if ! 0%{?rhel} Patch400: 0001-Update-liborcus-to-0.11.0.patch @@ -2398,6 +2399,9 @@ done %endif %changelog +* Thu Feb 16 2017 Eike Rathke - 1:5.1.6.2-8.UNBUILT +- Resolves: DOCX import w:hideMark vs. w:vMerge table layout + * Thu Feb 16 2017 Eike Rathke - 1:5.1.6.2-7 - Resolves: tdf#103493 copying note captions needs a completed destination sheet - Resolves: tdf#105968 handle engineering notation rounded into next magnitude