From 3e33a2551532806b99f6c8ca415136cd3f2f9256 Mon Sep 17 00:00:00 2001 From: Paul Trojahn Date: Tue, 1 Aug 2017 15:56:48 +0200 Subject: tdf#109317 Fix incorrect rotation of flipped custom OOXML shapes MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The shape wasn't imported correctly because IsPostRotateAngle wasn't set. As a result the shape was flipped after it was rotated instead of before. This commit removes 1c44b263 and converts the rotation to the internal format instead. This also fixes that ooxml preset shapes can't be exported to odf correctly and that ooxml preset shapes rotate in the wrong direction when rotated in LO. Change-Id: I9691902fdfb2cfecc8fe50d6eb66ac3880ddd4e8 Reviewed-on: https://gerrit.libreoffice.org/40789 Reviewed-by: Tamás Zolnai Tested-by: Tamás Zolnai --- sd/qa/unit/data/tdf109317.pptx | Bin 0 -> 39405 bytes sd/qa/unit/data/xml/fdo47434_0.xml | 4 - sd/qa/unit/data/xml/n593612_0.xml | 1 - sd/qa/unit/data/xml/n762695_0.xml | 8 +- sd/qa/unit/data/xml/n762695_1.xml | 3 - sd/qa/unit/data/xml/n819614_0.xml | 56 ---- sd/qa/unit/data/xml/n820786_0.xml | 51 ---- sd/qa/unit/data/xml/n902652_0.xml | 3 - sd/qa/unit/data/xml/tdf109317_0.xml | 492 ++++++++++++++++++++++++++++++++++++ sd/qa/unit/import-tests.cxx | 1 + 10 files changed, 496 insertions(+), 123 deletions(-) create mode 100644 sd/qa/unit/data/tdf109317.pptx create mode 100644 sd/qa/unit/data/xml/tdf109317_0.xml (limited to 'sd') diff --git a/sd/qa/unit/data/tdf109317.pptx b/sd/qa/unit/data/tdf109317.pptx new file mode 100644 index 000000000000..427108069b0e Binary files /dev/null and b/sd/qa/unit/data/tdf109317.pptx differ diff --git a/sd/qa/unit/data/xml/fdo47434_0.xml b/sd/qa/unit/data/xml/fdo47434_0.xml index a37b6cb91c12..776797057bdc 100644 --- a/sd/qa/unit/data/xml/fdo47434_0.xml +++ b/sd/qa/unit/data/xml/fdo47434_0.xml @@ -27,7 +27,6 @@ - @@ -90,7 +89,6 @@ - @@ -153,7 +151,6 @@ - @@ -216,7 +213,6 @@ - diff --git a/sd/qa/unit/data/xml/n593612_0.xml b/sd/qa/unit/data/xml/n593612_0.xml index 5d03e5887794..b595a2bb2a97 100644 --- a/sd/qa/unit/data/xml/n593612_0.xml +++ b/sd/qa/unit/data/xml/n593612_0.xml @@ -21,7 +21,6 @@ - diff --git a/sd/qa/unit/data/xml/n762695_0.xml b/sd/qa/unit/data/xml/n762695_0.xml index 2725854df0cc..0b216210097d 100644 --- a/sd/qa/unit/data/xml/n762695_0.xml +++ b/sd/qa/unit/data/xml/n762695_0.xml @@ -108,7 +108,6 @@ - @@ -163,7 +162,7 @@ - + @@ -172,8 +171,8 @@ - - + + @@ -185,7 +184,6 @@ - diff --git a/sd/qa/unit/data/xml/n762695_1.xml b/sd/qa/unit/data/xml/n762695_1.xml index 91ab1172478b..7cf8d4d59b0b 100644 --- a/sd/qa/unit/data/xml/n762695_1.xml +++ b/sd/qa/unit/data/xml/n762695_1.xml @@ -195,7 +195,6 @@ - @@ -348,7 +347,6 @@ - @@ -501,7 +499,6 @@ - diff --git a/sd/qa/unit/data/xml/n819614_0.xml b/sd/qa/unit/data/xml/n819614_0.xml index aff9c324344c..368a2fafcd2b 100644 --- a/sd/qa/unit/data/xml/n819614_0.xml +++ b/sd/qa/unit/data/xml/n819614_0.xml @@ -3310,7 +3310,6 @@ - @@ -3374,7 +3373,6 @@ - @@ -3438,7 +3436,6 @@ - @@ -3502,7 +3499,6 @@ - @@ -3566,7 +3562,6 @@ - @@ -3630,7 +3625,6 @@ - @@ -3694,7 +3688,6 @@ - @@ -3758,7 +3751,6 @@ - @@ -3822,7 +3814,6 @@ - @@ -3886,7 +3877,6 @@ - @@ -3950,7 +3940,6 @@ - @@ -4014,7 +4003,6 @@ - @@ -4078,7 +4066,6 @@ - @@ -4142,7 +4129,6 @@ - @@ -4206,7 +4192,6 @@ - @@ -4270,7 +4255,6 @@ - @@ -4334,7 +4318,6 @@ - @@ -4398,7 +4381,6 @@ - @@ -4462,7 +4444,6 @@ - @@ -4526,7 +4507,6 @@ - @@ -4590,7 +4570,6 @@ - @@ -4654,7 +4633,6 @@ - @@ -4718,7 +4696,6 @@ - @@ -4782,7 +4759,6 @@ - @@ -4846,7 +4822,6 @@ - @@ -4910,7 +4885,6 @@ - @@ -4974,7 +4948,6 @@ - @@ -5038,7 +5011,6 @@ - @@ -5102,7 +5074,6 @@ - @@ -5166,7 +5137,6 @@ - @@ -5230,7 +5200,6 @@ - @@ -5294,7 +5263,6 @@ - @@ -5358,7 +5326,6 @@ - @@ -5422,7 +5389,6 @@ - @@ -5486,7 +5452,6 @@ - @@ -5550,7 +5515,6 @@ - @@ -5614,7 +5578,6 @@ - @@ -5678,7 +5641,6 @@ - @@ -5742,7 +5704,6 @@ - @@ -5806,7 +5767,6 @@ - @@ -5870,7 +5830,6 @@ - @@ -5934,7 +5893,6 @@ - @@ -5998,7 +5956,6 @@ - @@ -6062,7 +6019,6 @@ - @@ -6126,7 +6082,6 @@ - @@ -6190,7 +6145,6 @@ - @@ -6254,7 +6208,6 @@ - @@ -6318,7 +6271,6 @@ - @@ -6382,7 +6334,6 @@ - @@ -6446,7 +6397,6 @@ - @@ -6510,7 +6460,6 @@ - @@ -6574,7 +6523,6 @@ - @@ -6638,7 +6586,6 @@ - @@ -6702,7 +6649,6 @@ - @@ -6766,7 +6712,6 @@ - @@ -6830,7 +6775,6 @@ - diff --git a/sd/qa/unit/data/xml/n820786_0.xml b/sd/qa/unit/data/xml/n820786_0.xml index 7f8d44d55db1..53a8ee93610d 100644 --- a/sd/qa/unit/data/xml/n820786_0.xml +++ b/sd/qa/unit/data/xml/n820786_0.xml @@ -21,7 +21,6 @@ - @@ -87,7 +86,6 @@ - @@ -171,7 +169,6 @@ - @@ -262,7 +259,6 @@ - @@ -326,7 +322,6 @@ - @@ -390,7 +385,6 @@ - @@ -454,7 +448,6 @@ - @@ -518,7 +511,6 @@ - @@ -582,7 +574,6 @@ - @@ -646,7 +637,6 @@ - @@ -710,7 +700,6 @@ - @@ -774,7 +763,6 @@ - @@ -838,7 +826,6 @@ - @@ -902,7 +889,6 @@ - @@ -966,7 +952,6 @@ - @@ -1030,7 +1015,6 @@ - @@ -1094,7 +1078,6 @@ - @@ -1158,7 +1141,6 @@ - @@ -1222,7 +1204,6 @@ - @@ -1286,7 +1267,6 @@ - @@ -1350,7 +1330,6 @@ - @@ -1414,7 +1393,6 @@ - @@ -1478,7 +1456,6 @@ - @@ -1542,7 +1519,6 @@ - @@ -1606,7 +1582,6 @@ - @@ -1670,7 +1645,6 @@ - @@ -1734,7 +1708,6 @@ - @@ -1798,7 +1771,6 @@ - @@ -1862,7 +1834,6 @@ - @@ -1926,7 +1897,6 @@ - @@ -1990,7 +1960,6 @@ - @@ -2054,7 +2023,6 @@ - @@ -2118,7 +2086,6 @@ - @@ -2182,7 +2149,6 @@ - @@ -2246,7 +2212,6 @@ - @@ -2310,7 +2275,6 @@ - @@ -2374,7 +2338,6 @@ - @@ -2438,7 +2401,6 @@ - @@ -2502,7 +2464,6 @@ - @@ -2566,7 +2527,6 @@ - @@ -2630,7 +2590,6 @@ - @@ -2694,7 +2653,6 @@ - @@ -2758,7 +2716,6 @@ - @@ -2822,7 +2779,6 @@ - @@ -2886,7 +2842,6 @@ - @@ -2950,7 +2905,6 @@ - @@ -3014,7 +2968,6 @@ - @@ -3078,7 +3031,6 @@ - @@ -3142,7 +3094,6 @@ - @@ -3206,7 +3157,6 @@ - @@ -3270,7 +3220,6 @@ - diff --git a/sd/qa/unit/data/xml/n902652_0.xml b/sd/qa/unit/data/xml/n902652_0.xml index de8ed91092ba..dacda959f063 100644 --- a/sd/qa/unit/data/xml/n902652_0.xml +++ b/sd/qa/unit/data/xml/n902652_0.xml @@ -21,7 +21,6 @@ - @@ -123,7 +122,6 @@ - @@ -225,7 +223,6 @@ - diff --git a/sd/qa/unit/data/xml/tdf109317_0.xml b/sd/qa/unit/data/xml/tdf109317_0.xml new file mode 100644 index 000000000000..b3b22a42ad32 --- /dev/null +++ b/sd/qa/unit/data/xml/tdf109317_0.xml @@ -0,0 +1,492 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/sd/qa/unit/import-tests.cxx b/sd/qa/unit/import-tests.cxx index 726df149b40e..7cf536eb3bb1 100644 --- a/sd/qa/unit/import-tests.cxx +++ b/sd/qa/unit/import-tests.cxx @@ -293,6 +293,7 @@ void SdImportTest::testDocumentLayout() { "tdf90338.odp", "xml/tdf90338_", ODP, PPTX }, { "tdf92001.odp", "xml/tdf92001_", ODP, PPTX }, { "tdf100491.pptx", "xml/tdf100491_", PPTX, -1 }, + { "tdf109317.pptx", "xml/tdf109317_", PPTX, ODP}, // { "pptx/n828390.pptx", "pptx/xml/n828390_", PPTX, PPTX }, // Example }; -- cgit