oox: move DataTableConverter into own file

Change-Id: I3dce87ecd5b8f69862f26edb66fe54c6acee8660
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/133117
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>

3 files changed, 53 insertions, 21 deletions

diff --git a/oox/source/drawingml/chart/datatablecontext.cxx b/oox/source/drawingml/chart/datatablecontext.cxx
index c40a4f782450..0813a6424312 100644
--- a/oox/source/drawingml/chart/datatablecontext.cxx
+++ b/oox/source/drawingml/chart/datatablecontext.cxx
@@ -55,7 +55,7 @@ ContextHandlerRef DataTableContext::onCreateContext(sal_Int32 nElement,
                     mrModel.mbShowOutline = rAttribs.getBool(XML_val, false);
                     break;
                 case C_TOKEN(showKeys):
-                    //mrModel.mbShowKeys = rAttribs.getBool( XML_val, false );
+                    mrModel.mbShowKeys = rAttribs.getBool(XML_val, false);
                     break;
             }
             break;
diff --git a/oox/source/drawingml/chart/datatableconverter.cxx b/oox/source/drawingml/chart/datatableconverter.cxx
new file mode 100644
index 000000000000..802edc9464ac
--- /dev/null
+++ b/oox/source/drawingml/chart/datatableconverter.cxx
@@ -0,0 +1,50 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ *
+ * This file incorporates work covered by the following license notice:
+ *
+ *   Licensed to the Apache Software Foundation (ASF) under one or more
+ *   contributor license agreements. See the NOTICE file distributed
+ *   with this work for additional information regarding copyright
+ *   ownership. The ASF licenses this file to you under the Apache
+ *   License, Version 2.0 (the "License"); you may not use this file
+ *   except in compliance with the License. You may obtain a copy of
+ *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
+ */
+
+#include <drawingml/chart/datatableconverter.hxx>
+
+#include <com/sun/star/chart2/XDiagram.hpp>
+#include <drawingml/chart/plotareamodel.hxx>
+#include <oox/token/properties.hxx>
+
+using namespace css;
+
+namespace oox::drawingml::chart
+{
+DataTableConverter::DataTableConverter(const ConverterRoot& rParent, DataTableModel& rModel)
+    : ConverterBase<DataTableModel>(rParent, rModel)
+{
+}
+
+DataTableConverter::~DataTableConverter() = default;
+
+void DataTableConverter::convertFromModel(uno::Reference<chart2::XDiagram> const& rxDiagram)
+{
+    PropertySet aPropSet(rxDiagram);
+    if (mrModel.mbShowHBorder)
+        aPropSet.setProperty(PROP_DataTableHBorder, mrModel.mbShowHBorder);
+    if (mrModel.mbShowVBorder)
+        aPropSet.setProperty(PROP_DataTableVBorder, mrModel.mbShowVBorder);
+    if (mrModel.mbShowOutline)
+        aPropSet.setProperty(PROP_DataTableOutline, mrModel.mbShowOutline);
+}
+
+} // namespace oox
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/oox/source/drawingml/chart/plotareaconverter.cxx b/oox/source/drawingml/chart/plotareaconverter.cxx
index 9a0f9d337c8a..afbb28adee03 100644
--- a/oox/source/drawingml/chart/plotareaconverter.cxx
+++ b/oox/source/drawingml/chart/plotareaconverter.cxx
@@ -28,6 +28,7 @@
 #include <com/sun/star/drawing/ProjectionMode.hpp>
 #include <com/sun/star/drawing/ShadeMode.hpp>
 #include <osl/diagnose.h>
+#include <drawingml/chart/datatableconverter.hxx>
 #include <drawingml/chart/axisconverter.hxx>
 #include <drawingml/chart/plotareamodel.hxx>
 #include <drawingml/chart/typegroupconverter.hxx>
@@ -292,26 +293,6 @@ void WallFloorConverter::convertFromModel( const Reference< XDiagram >& rxDiagra
     }
 }
 
-DataTableConverter::DataTableConverter( const ConverterRoot& rParent, DataTableModel& rModel ) :
-        ConverterBase< DataTableModel >( rParent, rModel )
-{
-}
-
-DataTableConverter::~DataTableConverter()
-{
-}
-
-void DataTableConverter::convertFromModel( const Reference< XDiagram >& rxDiagram )
-{
-    PropertySet aPropSet( rxDiagram );
-    if (mrModel.mbShowHBorder)
-        aPropSet.setProperty( PROP_DataTableHBorder, mrModel.mbShowHBorder );
-    if (mrModel.mbShowVBorder)
-        aPropSet.setProperty( PROP_DataTableVBorder, mrModel.mbShowVBorder);
-    if (mrModel.mbShowOutline)
-        aPropSet.setProperty( PROP_DataTableOutline, mrModel.mbShowOutline );
-}
-
 PlotAreaConverter::PlotAreaConverter( const ConverterRoot& rParent, PlotAreaModel& rModel ) :
     ConverterBase< PlotAreaModel >( rParent, rModel ),
     mb3dChart( false ),
@@ -454,6 +435,7 @@ void PlotAreaConverter::convertFromModel( View3DModel& rView3DModel )
 
     DataTableConverter dataTableConverter (*this, mrModel.mxDataTable.getOrCreate());
     dataTableConverter.convertFromModel(xDiagram);
+
     // plot area formatting
     if( xDiagram.is() && !mb3dChart )
     {