From b6f52c5c6bc57d831e09126f802739589e5eea37 Mon Sep 17 00:00:00 2001
From: Mark Hung <marklh9@gmail.com>
Date: Wed, 23 Nov 2016 20:23:37 +0800
Subject: [PATCH] tdf#96052 - export the forbidden shape as path instead of
 preset shape.

OOXML noSmoking shape has different definition so that convert LO's
forbidden shape to OOXML noSmoking do not produce good result.
I put it on whitelist and now the geometry looks correct although handle
is lost.

This partially revert unit test for the shape in
2b4f9d0b2b0006fc7bebb9e696a32eabd1aeb993, where forbidden is exported as
a preset shape.

Change-Id: I2e134940fa4a36e6b7b814b008d859691fbcdd6a
Reviewed-on: https://gerrit.libreoffice.org/31110
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mark Hung <marklh9@gmail.com>
---
 oox/source/export/shapes.cxx       | 8 +-------
 sd/qa/unit/export-tests-ooxml2.cxx | 2 --
 2 files changed, 1 insertion(+), 9 deletions(-)

diff --git a/oox/source/export/shapes.cxx b/oox/source/export/shapes.cxx
index 820cb4ca56d3..7bd85a8cd39f 100644
--- a/oox/source/export/shapes.cxx
+++ b/oox/source/export/shapes.cxx
@@ -547,7 +547,6 @@ static bool lcl_IsOnBlacklist(OUString& rShapeType)
         OUStringLiteral("smiley"),
         OUStringLiteral("sun"),
         OUStringLiteral("flower"),
-        OUStringLiteral("forbidden"),
         OUStringLiteral("bracket-pair"),
         OUStringLiteral("brace-pair"),
         OUStringLiteral("col-60da8460"),
@@ -606,6 +605,7 @@ static bool lcl_IsOnWhitelist(OUString& rShapeType)
     static
 #endif
     const std::initializer_list<OUStringLiteral> vWhitelist = {
+        OUStringLiteral("forbidden"),
         OUStringLiteral("heart"),
         OUStringLiteral("puzzle")
     };
@@ -918,12 +918,6 @@ ShapeExport& ShapeExport::WriteCustomShape( const Reference< XShape >& xShape )
                 lcl_AppendAdjustmentValue( aAvList, 0, adj );
                 break;
             }
-            case mso_sptNoSmoking:
-            {
-                sal_Int32 adj =  double( nXPosition )/7200 *50000 ;
-                lcl_AppendAdjustmentValue( aAvList, 0, adj );
-                break;
-            }
             case mso_sptDonut:
             case mso_sptSun:
             case mso_sptMoon:
diff --git a/sd/qa/unit/export-tests-ooxml2.cxx b/sd/qa/unit/export-tests-ooxml2.cxx
index 05ef4aa700aa..540e80022d7d 100644
--- a/sd/qa/unit/export-tests-ooxml2.cxx
+++ b/sd/qa/unit/export-tests-ooxml2.cxx
@@ -467,8 +467,6 @@ void SdOOXMLExportTest2::testPresetShapesExport()
         "adj1","val 10490",
         "donut",
         "adj","val 9601",
-        "noSmoking",
-        "adj","val 16118",
         "bevel",
         "adj","val 42587",
         "foldedCorner",