[Libreoffice-commits] core.git: oox/README.vars oox/source

[Libreoffice Gerrit user]

 oox/README.vars              |    5 +++++
 oox/source/ppt/pptimport.cxx |    3 ++-
 2 files changed, 7 insertions(+), 1 deletion(-)

New commits:
commit 4fb3e08a9a9f057708c530faec853993e8972029
Author:     Miklos Vajna <vmiklos at collabora.co.uk>
AuthorDate: Wed Oct 17 15:20:37 2018 +0200
Commit:     Miklos Vajna <vmiklos at collabora.co.uk>
CommitDate: Wed Oct 17 21:19:37 2018 +0200

    oox: add an environment variable to bypass the SmartArt warning
    
    That dialog is counter-productive if you develop that feature itself.
    
    Change-Id: Ia635a2ff0ecc29a0d11be5c621a7c2a76469f9ba
    Reviewed-on: https://gerrit.libreoffice.org/61880
    Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>
    Tested-by: Jenkins

diff --git a/oox/README.vars b/oox/README.vars
new file mode 100644
index 000000000000..a2aba2023826
--- /dev/null
+++ b/oox/README.vars
@@ -0,0 +1,5 @@
+Environment variables in oox:
+
+- ppt:
+
+  - OOX_NO_SMARTART_WARNING - don't warn on SmartArts missing their DrawingML fallback
diff --git a/oox/source/ppt/pptimport.cxx b/oox/source/ppt/pptimport.cxx
index bfc58430840d..2ce9ac4ea0ea 100644
--- a/oox/source/ppt/pptimport.cxx
+++ b/oox/source/ppt/pptimport.cxx
@@ -138,7 +138,8 @@ bool PowerPointImport::importDocument()
 
     bool bRet = importFragment(xPresentationFragmentHandler);
 
-    if (mbMissingExtDrawing)
+    static bool bNoSmartartWarning = getenv("OOX_NO_SMARTART_WARNING");
+    if (!bNoSmartartWarning && mbMissingExtDrawing)
     {
         // Construct a warning message.
         INetURLObject aURL(getFileUrl());