diff options
author | Regina Henschel <rb.henschel@t-online.de> | 2022-03-18 18:31:05 +0100 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2022-03-25 14:00:23 +0100 |
commit | 2029b2f6dd0109c5892e5ac5640022b31fe42fd2 (patch) | |
tree | cdb2b2776ea497f8bed2ffdcdaf100b2c2a4b64f /oox/qa/unit/data | |
parent | c4cb1d1dd581a5f120d9cf8b1d4274ec38f3eabe (diff) |
tdf#147978 export subpaths individually in custGeom
...and implement export of all missing commands,
use existing viewBox if suitable,
use one EnhancedCustomShape2d
move WriteCustomGeometryPoint to protected,
make GetCustomGeometryPointValue local
The fix solves tdf#100390 too.
Without the fix the entire enhanced-path was exported as one element
<a:path>. The command F was applied to the whole drawing but should
affect only the subpath. The implementation is changed so that each
subpath gets its own element <a:path> and command F acts only on its
subpath.
Support for export of handles and equations is still a long way off.
Thus there is no reason to tread shapes with and without handles
different. Shapes with handles had used method WritePolyPolygon, but
that is not able to handle subpaths. So have desided to use method
WriteCustomGeometry for all cases.
To get shapes exported regardless of path commands I have added the
export for the missing commands.
I have removed the no longer used method WritePolyPolygon.
The special treatment of shapes "moon" and "mso-spt89" (right up arrow)
in export is no longer needed. Related code parts are removed. The
unit test testFlipAndRotateCustomShape is adapted.
In case the method WriteCustomGeometry fails, the enhanced-path is
invalid. In that case an minimal custGeom is written in case of docx.
Shapes whose drawing does not touch all edges of the snap rectangle
were exported with wrong position and size of the drawing inside the
snap rectangle. That is fixed by using an existing ViewBox for the
OOXML path size. The old way of creating a path size from point
coordinates is only used if the shape has no suitable ViewBox.
The point values in unit test SdOOXMLExportTest2::testTdf111798 are
adapted to path size 21600 x 21600 and traverse direction of the points
is corrected. The resulting shape outline is still the same as before.
The expected xml is updated for file tdf92001.odp in
SdImportTest::testDocumentLayout. The resulting shape outline is the
same, because the shape touches the edges of the snap rectangle.
The case, that the shape outline does not touch a edge of the snap
rectangle is tested in SdOOXMLExportTest3::testEnhancedPathViewBox.
Still missing is the case, that ViewBox has other left,top than 0,0.
In that case all coordinates would have to be shifted because the path
size in OOXML has only width and height but not left,top. That will
not be included in this patch.
Change-Id: Ib1736d6a08371f4d98411d2769275f0580cd0030
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/131837
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Diffstat (limited to 'oox/qa/unit/data')
-rw-r--r-- | oox/qa/unit/data/tdf147978_endsubpath.odp | bin | 0 -> 14505 bytes |
-rw-r--r-- | oox/qa/unit/data/tdf147978_enhancedPath_commandA.odp | bin | 0 -> 12755 bytes |
-rw-r--r-- | oox/qa/unit/data/tdf147978_enhancedPath_commandHIJK.odp | bin | 0 -> 14922 bytes |
-rw-r--r-- | oox/qa/unit/data/tdf147978_enhancedPath_commandT.odp | bin | 0 -> 12869 bytes |
-rw-r--r-- | oox/qa/unit/data/tdf147978_enhancedPath_commandXY.odp | bin | 0 -> 13166 bytes |
-rw-r--r-- | oox/qa/unit/data/tdf147978_enhancedPath_subpath.pptx | bin | 0 -> 15040 bytes |
6 files changed, 0 insertions, 0 deletions
diff --git a/oox/qa/unit/data/tdf147978_endsubpath.odp b/oox/qa/unit/data/tdf147978_endsubpath.odp Binary files differnew file mode 100644 index 000000000000..2dfd55de1be3 --- /dev/null +++ b/oox/qa/unit/data/tdf147978_endsubpath.odp diff --git a/oox/qa/unit/data/tdf147978_enhancedPath_commandA.odp b/oox/qa/unit/data/tdf147978_enhancedPath_commandA.odp Binary files differnew file mode 100644 index 000000000000..99ddda7c132e --- /dev/null +++ b/oox/qa/unit/data/tdf147978_enhancedPath_commandA.odp diff --git a/oox/qa/unit/data/tdf147978_enhancedPath_commandHIJK.odp b/oox/qa/unit/data/tdf147978_enhancedPath_commandHIJK.odp Binary files differnew file mode 100644 index 000000000000..49e01bc0933a --- /dev/null +++ b/oox/qa/unit/data/tdf147978_enhancedPath_commandHIJK.odp diff --git a/oox/qa/unit/data/tdf147978_enhancedPath_commandT.odp b/oox/qa/unit/data/tdf147978_enhancedPath_commandT.odp Binary files differnew file mode 100644 index 000000000000..3dcd0d567545 --- /dev/null +++ b/oox/qa/unit/data/tdf147978_enhancedPath_commandT.odp diff --git a/oox/qa/unit/data/tdf147978_enhancedPath_commandXY.odp b/oox/qa/unit/data/tdf147978_enhancedPath_commandXY.odp Binary files differnew file mode 100644 index 000000000000..6112251783e1 --- /dev/null +++ b/oox/qa/unit/data/tdf147978_enhancedPath_commandXY.odp diff --git a/oox/qa/unit/data/tdf147978_enhancedPath_subpath.pptx b/oox/qa/unit/data/tdf147978_enhancedPath_subpath.pptx Binary files differnew file mode 100644 index 000000000000..bbedc7ab98f5 --- /dev/null +++ b/oox/qa/unit/data/tdf147978_enhancedPath_subpath.pptx |