import chart MSO 2007 streams correctly for docx files, tdf#82216

Change-Id: Icda809faf315dac5953d38781b2b401d51f7a40a

3 files changed, 29 insertions, 12 deletions

diff --git a/oox/source/core/xmlfilterbase.cxx b/oox/source/core/xmlfilterbase.cxx
index 5597a5865cf5..4ff011aa9161 100644
--- a/oox/source/core/xmlfilterbase.cxx
+++ b/oox/source/core/xmlfilterbase.cxx
@@ -223,33 +223,30 @@ XmlFilterBase::~XmlFilterBase()
     mxImpl->maFastParser.setDocumentHandler( 0 );
 }
 
-namespace {
-
-bool is2007MSODocument(Reference<XDocumentProperties> xDocProps)
+void XmlFilterBase::checkDocumentProperties(Reference<XDocumentProperties> xDocProps)
 {
+    mbMSO2007 = false;
     if (!xDocProps->getGenerator().startsWithIgnoreAsciiCase("Microsoft"))
-        return false;
+        return;
 
     uno::Reference<beans::XPropertyAccess> xUserDefProps(xDocProps->getUserDefinedProperties(), uno::UNO_QUERY);
     if (!xUserDefProps.is())
-        return false;
+        return;
 
     comphelper::SequenceAsHashMap aUserDefinedProperties(xUserDefProps->getPropertyValues());
     comphelper::SequenceAsHashMap::iterator it = aUserDefinedProperties.find("AppVersion");
     if (it == aUserDefinedProperties.end())
-        return false;
+        return;
 
     OUString aValue;
     if (!(it->second >>= aValue))
-        return false;
+        return;
 
     if (!aValue.startsWithIgnoreAsciiCase("12."))
-        return false;
+        return;
 
     SAL_WARN("oox", "a MSO 2007 document");
-    return true;
-}
-
+    mbMSO2007 = true;
 }
 
 void XmlFilterBase::importDocumentProperties()
@@ -270,7 +267,7 @@ void XmlFilterBase::importDocumentProperties()
     Reference< XDocumentPropertiesSupplier > xPropSupplier( xModel, UNO_QUERY);
     Reference< XDocumentProperties > xDocProps = xPropSupplier->getDocumentProperties();
     xImporter->importProperties( xDocumentStorage, xDocProps );
-    mbMSO2007 = is2007MSODocument(xDocProps);
+    checkDocumentProperties(xDocProps);
 }
 
 FastParser* XmlFilterBase::createParser() const
diff --git a/oox/source/shape/ShapeContextHandler.cxx b/oox/source/shape/ShapeContextHandler.cxx
index e4910ad037d0..39cc970f0be1 100644
--- a/oox/source/shape/ShapeContextHandler.cxx
+++ b/oox/source/shape/ShapeContextHandler.cxx
@@ -620,6 +620,19 @@ void SAL_CALL ShapeContextHandler::setPosition(const awt::Point& rPosition) thro
     maPosition = rPosition;
 }
 
+void SAL_CALL ShapeContextHandler::setDocumentProperties(const uno::Reference<document::XDocumentProperties>& xDocProps)
+    throw (css::uno::RuntimeException, std::exception)
+{
+    mxDocumentProperties = xDocProps;
+    mxFilterBase->checkDocumentProperties(mxDocumentProperties);
+}
+
+uno::Reference<document::XDocumentProperties> SAL_CALL ShapeContextHandler::getDocumentProperties()
+    throw (css::uno::RuntimeException, std::exception)
+{
+    return mxDocumentProperties;
+}
+
 OUString ShapeContextHandler::getImplementationName()
     throw (css::uno::RuntimeException, std::exception)
 {
diff --git a/oox/source/shape/ShapeContextHandler.hxx b/oox/source/shape/ShapeContextHandler.hxx
index 7697e2a6c4a8..ad57ad933416 100644
--- a/oox/source/shape/ShapeContextHandler.hxx
+++ b/oox/source/shape/ShapeContextHandler.hxx
@@ -30,6 +30,7 @@
 #include "oox/core/xmlfilterbase.hxx"
 #include "ShapeFilterBase.hxx"
 #include <com/sun/star/lang/XServiceInfo.hpp>
+#include <com/sun/star/document/XDocumentProperties.hpp>
 
 namespace oox { namespace shape {
 
@@ -138,6 +139,11 @@ public:
     virtual css::awt::Point SAL_CALL getPosition() throw (css::uno::RuntimeException, std::exception) SAL_OVERRIDE;
     virtual void SAL_CALL setPosition(const css::awt::Point& rPosition) throw (css::uno::RuntimeException, std::exception) SAL_OVERRIDE;
 
+    virtual void SAL_CALL setDocumentProperties(const css::uno::Reference<css::document::XDocumentProperties>& xDocProps)
+        throw (css::uno::RuntimeException, std::exception) SAL_OVERRIDE;
+    virtual css::uno::Reference<css::document::XDocumentProperties> SAL_CALL getDocumentProperties()
+        throw (css::uno::RuntimeException, std::exception) SAL_OVERRIDE;
+
 private:
     ShapeContextHandler(ShapeContextHandler &) SAL_DELETED_FUNCTION;
     void operator =(ShapeContextHandler &) SAL_DELETED_FUNCTION;
@@ -159,6 +165,7 @@ private:
     css::uno::Reference<css::drawing::XShape> mxSavedShape;
     css::uno::Reference<XFastContextHandler> mxWpgContext;
     css::uno::Reference<XFastContextHandler> mxChartShapeContext;
+    css::uno::Reference<css::document::XDocumentProperties> mxDocumentProperties;
 
     core::XmlFilterRef mxFilterBase;
     drawingml::ThemePtr mpThemePtr;