chart2/source/controller/inc/DataPointItemConverter.hxx                     |  
  1 
 chart2/source/controller/inc/ErrorBarItemConverter.hxx                      |  
  7 +-
 chart2/source/controller/inc/GraphicPropertyItemConverter.hxx               |  
  3 -
 chart2/source/controller/inc/LegendItemConverter.hxx                        |  
  4 +
 chart2/source/controller/inc/MultipleChartConverters.hxx                    |  
  9 +--
 chart2/source/controller/inc/RegressionCurveItemConverter.hxx               |  
  2 
 chart2/source/controller/inc/RegressionEquationItemConverter.hxx            |  
  4 +
 chart2/source/controller/inc/TitleItemConverter.hxx                         |  
  4 +
 chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx          |  
  3 -
 chart2/source/controller/itemsetwrapper/ErrorBarItemConverter.cxx           |  
 10 +--
 chart2/source/controller/itemsetwrapper/GraphicPropertyItemConverter.cxx    |  
 27 ++++------
 chart2/source/controller/itemsetwrapper/LegendItemConverter.cxx             |  
  5 +
 chart2/source/controller/itemsetwrapper/MultipleChartConverters.cxx         |  
 15 ++---
 chart2/source/controller/itemsetwrapper/RegressionCurveItemConverter.cxx    |  
  5 +
 chart2/source/controller/itemsetwrapper/RegressionEquationItemConverter.cxx |  
  5 +
 chart2/source/controller/itemsetwrapper/TitleItemConverter.cxx              |  
  5 +
 chart2/source/controller/main/ChartController_Insert.cxx                    |  
  6 --
 chart2/source/controller/main/ChartController_Properties.cxx                |  
  8 +-
 18 files changed, 59 insertions(+), 64 deletions(-)

New commits:
commit 0b900b8683398666f8c5d84587bc2b4dc91a4a03
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Tue Jun 17 13:05:52 2025 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Wed Jun 18 19:52:40 2025 +0200

    use more concrete UNO classes in chart2
    
    Change-Id: Ia8ccb01b6ae4b586da0d6f11d0593840af4c356f
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/186668
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>
    Tested-by: Jenkins

diff --git a/chart2/source/controller/inc/DataPointItemConverter.hxx 
b/chart2/source/controller/inc/DataPointItemConverter.hxx
index 720cc0fef2e7..5d75b3057cfd 100644
--- a/chart2/source/controller/inc/DataPointItemConverter.hxx
+++ b/chart2/source/controller/inc/DataPointItemConverter.hxx
@@ -47,7 +47,6 @@ public:
         const rtl::Reference<::chart::DataSeries>& xSeries,
         SfxItemPool& rItemPool,
         SdrModel& rDrawModel,
-        const css::uno::Reference<css::lang::XMultiServiceFactory>& 
xNamedPropertyContainerFactory,
         GraphicObjectType eMapTo,
         const std::optional<css::awt::Size>& pRefSize = std::nullopt,
         bool bDataSeries = false,
diff --git a/chart2/source/controller/inc/ErrorBarItemConverter.hxx 
b/chart2/source/controller/inc/ErrorBarItemConverter.hxx
index a53fbe18ef60..50b2fbe3f73b 100644
--- a/chart2/source/controller/inc/ErrorBarItemConverter.hxx
+++ b/chart2/source/controller/inc/ErrorBarItemConverter.hxx
@@ -20,11 +20,13 @@
 #pragma once
 
 #include "ItemConverter.hxx"
+#include <rtl/ref.hxx>
 
 namespace com::sun::star::frame { class XModel; }
 namespace com::sun::star::lang { class XMultiServiceFactory; }
 
 class SdrModel;
+namespace chart { class ChartModel; }
 
 namespace chart::wrapper
 {
@@ -33,11 +35,10 @@ class ErrorBarItemConverter final : public ItemConverter
 {
 public:
     ErrorBarItemConverter(
-        css::uno::Reference< css::frame::XModel > xChartModel,
+        const rtl::Reference< ChartModel > & xChartModel,
         const css::uno::Reference< css::beans::XPropertySet > & rPropertySet,
         SfxItemPool& rItemPool,
-        SdrModel& rDrawModel,
-        const css::uno::Reference< css::lang::XMultiServiceFactory > & 
xNamedPropertyContainerFactory );
+        SdrModel& rDrawModel );
     virtual ~ErrorBarItemConverter() override;
 
     virtual void FillItemSet( SfxItemSet & rOutItemSet ) const override;
diff --git a/chart2/source/controller/inc/GraphicPropertyItemConverter.hxx 
b/chart2/source/controller/inc/GraphicPropertyItemConverter.hxx
index c8a78944c96b..afb9dd0a0c40 100644
--- a/chart2/source/controller/inc/GraphicPropertyItemConverter.hxx
+++ b/chart2/source/controller/inc/GraphicPropertyItemConverter.hxx
@@ -46,7 +46,7 @@ public:
         const css::uno::Reference< css::beans::XPropertySet > & rPropertySet,
         SfxItemPool& rItemPool,
         SdrModel& rDrawModel,
-        css::uno::Reference< css::lang::XMultiServiceFactory > 
xNamedPropertyContainerFactory,
+        rtl::Reference< ChartModel > xChartModel,
         GraphicObjectType eObjectType );
     virtual ~GraphicPropertyItemConverter() override;
 
@@ -60,7 +60,6 @@ protected:
 private:
     GraphicObjectType              m_GraphicObjectType;
     SdrModel &                      m_rDrawModel;
-    css::uno::Reference< css::lang::XMultiServiceFactory >  
m_xNamedPropertyTableFactory;
     rtl::Reference<ChartModel> m_xChartModel;
 };
 
diff --git a/chart2/source/controller/inc/LegendItemConverter.hxx 
b/chart2/source/controller/inc/LegendItemConverter.hxx
index cbb657edf0d3..4d3d8fd87e59 100644
--- a/chart2/source/controller/inc/LegendItemConverter.hxx
+++ b/chart2/source/controller/inc/LegendItemConverter.hxx
@@ -20,12 +20,14 @@
 
 #include "ItemConverter.hxx"
 #include <com/sun/star/awt/Size.hpp>
+#include <rtl/ref.hxx>
 #include <optional>
 #include <vector>
 
 namespace com::sun::star::lang { class XMultiServiceFactory; }
 
 class SdrModel;
+namespace chart { class ChartModel; }
 
 namespace chart::wrapper
 {
@@ -37,7 +39,7 @@ public:
         const css::uno::Reference< css::beans::XPropertySet > & rPropertySet,
         SfxItemPool& rItemPool,
         SdrModel& rDrawModel,
-        const css::uno::Reference< css::lang::XMultiServiceFactory > & 
xNamedPropertyContainerFactory,
+        const rtl::Reference< ChartModel > & xChartModel,
         const std::optional<css::awt::Size>& pRefSize );
 
     virtual ~LegendItemConverter() override;
diff --git a/chart2/source/controller/inc/MultipleChartConverters.hxx 
b/chart2/source/controller/inc/MultipleChartConverters.hxx
index 51b3f70e190a..ce24aba49309 100644
--- a/chart2/source/controller/inc/MultipleChartConverters.hxx
+++ b/chart2/source/controller/inc/MultipleChartConverters.hxx
@@ -50,8 +50,7 @@ public:
     AllGridItemConverter(
         const rtl::Reference<::chart::ChartModel>& xChartModel,
         SfxItemPool& rItemPool,
-        SdrModel& rDrawModel,
-        const css::uno::Reference<css::lang::XMultiServiceFactory> & 
xNamedPropertyContainerFactory );
+        SdrModel& rDrawModel );
     virtual ~AllGridItemConverter() override;
 
 protected:
@@ -64,8 +63,7 @@ public:
     AllDataLabelItemConverter(
         const rtl::Reference<::chart::ChartModel>& xChartModel,
         SfxItemPool& rItemPool,
-        SdrModel& rDrawModel,
-        const css::uno::Reference<css::lang::XMultiServiceFactory>& 
xNamedPropertyContainerFactory );
+        SdrModel& rDrawModel );
 
     virtual ~AllDataLabelItemConverter() override;
 
@@ -78,8 +76,7 @@ class AllTitleItemConverter final : public 
MultipleItemConverter
 public:
     AllTitleItemConverter(
         const rtl::Reference<::chart::ChartModel>& xChartModel,
-        SfxItemPool& rItemPool, SdrModel& rDrawModel,
-        const css::uno::Reference<css::lang::XMultiServiceFactory>& 
xNamedPropertyContainerFactory );
+        SfxItemPool& rItemPool, SdrModel& rDrawModel );
 
     virtual ~AllTitleItemConverter() override;
 
diff --git a/chart2/source/controller/inc/RegressionCurveItemConverter.hxx 
b/chart2/source/controller/inc/RegressionCurveItemConverter.hxx
index cce795f9789d..3849a2179387 100644
--- a/chart2/source/controller/inc/RegressionCurveItemConverter.hxx
+++ b/chart2/source/controller/inc/RegressionCurveItemConverter.hxx
@@ -36,7 +36,7 @@ public:
         rtl::Reference< ::chart::DataSeries > xRegCurveCnt,
         SfxItemPool& rItemPool,
         SdrModel& rDrawModel,
-        const css::uno::Reference< css::lang::XMultiServiceFactory > & 
xNamedPropertyContainerFactory );
+        const rtl::Reference< ChartModel > & xChartModel );
     virtual ~RegressionCurveItemConverter() override;
 
     virtual void FillItemSet( SfxItemSet & rOutItemSet ) const override;
diff --git a/chart2/source/controller/inc/RegressionEquationItemConverter.hxx 
b/chart2/source/controller/inc/RegressionEquationItemConverter.hxx
index 83394b1ea45d..e0b0bac2bae9 100644
--- a/chart2/source/controller/inc/RegressionEquationItemConverter.hxx
+++ b/chart2/source/controller/inc/RegressionEquationItemConverter.hxx
@@ -20,6 +20,7 @@
 
 #include "ItemConverter.hxx"
 #include <com/sun/star/awt/Size.hpp>
+#include <rtl/ref.hxx>
 #include <optional>
 #include <vector>
 
@@ -27,6 +28,7 @@ namespace com::sun::star::beans { class XPropertySet; }
 namespace com::sun::star::lang { class XMultiServiceFactory; }
 
 class SdrModel;
+namespace chart { class ChartModel; }
 
 namespace chart::wrapper {
 
@@ -37,7 +39,7 @@ public:
         const css::uno::Reference< css::beans::XPropertySet > & rPropertySet,
         SfxItemPool& rItemPool,
         SdrModel& rDrawModel,
-        const css::uno::Reference< css::lang::XMultiServiceFactory > & 
xNamedPropertyContainerFactory,
+        const rtl::Reference< ChartModel > & xChartModel,
         const std::optional<css::awt::Size>& pRefSize );
 
     virtual ~RegressionEquationItemConverter() override;
diff --git a/chart2/source/controller/inc/TitleItemConverter.hxx 
b/chart2/source/controller/inc/TitleItemConverter.hxx
index 4a306b690363..917db7ef9cb6 100644
--- a/chart2/source/controller/inc/TitleItemConverter.hxx
+++ b/chart2/source/controller/inc/TitleItemConverter.hxx
@@ -21,12 +21,14 @@
 #include "ItemConverter.hxx"
 
 #include <com/sun/star/awt/Size.hpp>
+#include <rtl/ref.hxx>
 #include <optional>
 #include <vector>
 
 namespace com::sun::star::lang { class XMultiServiceFactory; }
 
 class SdrModel;
+namespace chart { class ChartModel; }
 
 namespace chart::wrapper {
 
@@ -36,7 +38,7 @@ public:
     TitleItemConverter(
         const css::uno::Reference<css::beans::XPropertySet>& rPropertySet,
         SfxItemPool& rItemPool, SdrModel& rDrawModel,
-        const css::uno::Reference<css::lang::XMultiServiceFactory>& 
xNamedPropertyContainerFactory,
+        const rtl::Reference<ChartModel>& xChartModel,
         const std::optional<css::awt::Size>& pRefSize );
 
     virtual ~TitleItemConverter() override;
diff --git a/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx 
b/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx
index 57fe3fd6f3d9..aa8f21231120 100644
--- a/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx
+++ b/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx
@@ -206,7 +206,6 @@ DataPointItemConverter::DataPointItemConverter(
     const rtl::Reference< DataSeries > & xSeries,
     SfxItemPool& rItemPool,
     SdrModel& rDrawModel,
-    const uno::Reference<lang::XMultiServiceFactory>& 
xNamedPropertyContainerFactory,
     GraphicObjectType eMapTo,
     const std::optional<awt::Size>& pRefSize,
     bool bDataSeries,
@@ -229,7 +228,7 @@ DataPointItemConverter::DataPointItemConverter(
         m_xSeries(xSeries)
 {
     m_aConverters.emplace_back( new GraphicPropertyItemConverter(
-                                 rPropertySet, rItemPool, rDrawModel, 
xNamedPropertyContainerFactory, eMapTo ));
+                                 rPropertySet, rItemPool, rDrawModel, 
xChartModel, eMapTo ));
     m_aConverters.emplace_back( new 
CharacterPropertyItemConverter(rPropertySet, rItemPool, pRefSize, 
u"ReferencePageSize"_ustr));
     if( bDataSeries )
     {
diff --git a/chart2/source/controller/itemsetwrapper/ErrorBarItemConverter.cxx 
b/chart2/source/controller/itemsetwrapper/ErrorBarItemConverter.cxx
index 082494182a01..f97ae631ee02 100644
--- a/chart2/source/controller/itemsetwrapper/ErrorBarItemConverter.cxx
+++ b/chart2/source/controller/itemsetwrapper/ErrorBarItemConverter.cxx
@@ -22,6 +22,7 @@
 #include <StatisticsHelper.hxx>
 
 #include <GraphicPropertyItemConverter.hxx>
+#include <ChartModel.hxx>
 
 #include <svl/stritem.hxx>
 #include <svx/chrtitem.hxx>
@@ -79,17 +80,16 @@ namespace chart::wrapper
 {
 
 ErrorBarItemConverter::ErrorBarItemConverter(
-    uno::Reference< frame::XModel > xModel,
+    const rtl::Reference< ChartModel > & xChartModel,
     const uno::Reference< beans::XPropertySet > & rPropertySet,
     SfxItemPool& rItemPool,
-    SdrModel& rDrawModel,
-    const uno::Reference< lang::XMultiServiceFactory > & 
xNamedPropertyContainerFactory ) :
+    SdrModel& rDrawModel ) :
         ItemConverter( rPropertySet, rItemPool ),
         m_spGraphicConverter( std::make_shared<GraphicPropertyItemConverter>(
                                   rPropertySet, rItemPool, rDrawModel,
-                                  xNamedPropertyContainerFactory,
+                                  xChartModel,
                                   GraphicObjectType::LineProperties )),
-        m_xModel(std::move( xModel ))
+        m_xModel(xChartModel)
 {}
 
 ErrorBarItemConverter::~ErrorBarItemConverter()
diff --git 
a/chart2/source/controller/itemsetwrapper/GraphicPropertyItemConverter.cxx 
b/chart2/source/controller/itemsetwrapper/GraphicPropertyItemConverter.cxx
index a6fecfb39f85..301abe03397c 100644
--- a/chart2/source/controller/itemsetwrapper/GraphicPropertyItemConverter.cxx
+++ b/chart2/source/controller/itemsetwrapper/GraphicPropertyItemConverter.cxx
@@ -140,16 +140,13 @@ 
GraphicPropertyItemConverter::GraphicPropertyItemConverter(
     beans::XPropertySet > & rPropertySet,
     SfxItemPool& rItemPool,
     SdrModel& rDrawModel,
-    uno::Reference< lang::XMultiServiceFactory > 
xNamedPropertyContainerFactory,
+    rtl::Reference< ChartModel > xChartModel,
     GraphicObjectType eObjectType /* = FILL_PROPERTIES */ ) :
         ItemConverter( rPropertySet, rItemPool ),
         m_GraphicObjectType( eObjectType ),
         m_rDrawModel( rDrawModel ),
-        m_xNamedPropertyTableFactory(std::move( xNamedPropertyContainerFactory 
))
+        m_xChartModel(std::move( xChartModel ))
 {
-    m_xChartModel = 
dynamic_cast<ChartModel*>(m_xNamedPropertyTableFactory.get());
-    DBG_ASSERT(m_xChartModel.is(),
-        "GraphicPropertyItemConverter ctor: passed XMultiServiceFactory 
parameter is not a ChartModel instance.");
 }
 
 GraphicPropertyItemConverter::~GraphicPropertyItemConverter()
@@ -253,7 +250,7 @@ void GraphicPropertyItemConverter::FillSpecialItem(
                         aItem.PutValue( aValue, MID_NAME );
 
                         lcl_SetContentForNamedProperty(
-                            m_xNamedPropertyTableFactory, 
u"com.sun.star.drawing.TransparencyGradientTable"_ustr ,
+                            m_xChartModel, 
u"com.sun.star.drawing.TransparencyGradientTable"_ustr ,
                             aItem, MID_FILLGRADIENT );
 
                         // this is important to enable the item
@@ -301,7 +298,7 @@ void GraphicPropertyItemConverter::FillSpecialItem(
             aItem.PutValue( GetPropertySet()->getPropertyValue( aPropName ), 
MID_NAME );
 
             lcl_SetContentForNamedProperty(
-                m_xNamedPropertyTableFactory, 
u"com.sun.star.drawing.DashTable"_ustr ,
+                m_xChartModel, u"com.sun.star.drawing.DashTable"_ustr ,
                 aItem, MID_LINEDASH );
 
             // translate model name to UI-name for predefined entries, so
@@ -327,7 +324,7 @@ void GraphicPropertyItemConverter::FillSpecialItem(
                 aItem.PutValue( GetPropertySet()->getPropertyValue( aPropName 
), MID_NAME );
 
                 lcl_SetContentForNamedProperty(
-                    m_xNamedPropertyTableFactory, 
u"com.sun.star.drawing.GradientTable"_ustr ,
+                    m_xChartModel, u"com.sun.star.drawing.GradientTable"_ustr ,
                     aItem, MID_FILLGRADIENT );
 
                 // translate model name to UI-name for predefined entries, so
@@ -353,7 +350,7 @@ void GraphicPropertyItemConverter::FillSpecialItem(
                 aItem.PutValue( GetPropertySet()->getPropertyValue( aPropName 
), MID_NAME );
 
                 lcl_SetContentForNamedProperty(
-                    m_xNamedPropertyTableFactory, 
u"com.sun.star.drawing.HatchTable"_ustr ,
+                    m_xChartModel, u"com.sun.star.drawing.HatchTable"_ustr ,
                     aItem, MID_FILLHATCH );
 
                 // translate model name to UI-name for predefined entries, so
@@ -374,7 +371,7 @@ void GraphicPropertyItemConverter::FillSpecialItem(
                 aItem.PutValue( GetPropertySet()->getPropertyValue( 
u"FillBitmapName"_ustr ), MID_NAME );
 
                 lcl_SetContentForNamedProperty(
-                    m_xNamedPropertyTableFactory, 
u"com.sun.star.drawing.BitmapTable"_ustr ,
+                    m_xChartModel, u"com.sun.star.drawing.BitmapTable"_ustr ,
                     aItem, MID_BITMAP );
 
                 // translate model name to UI-name for predefined entries, so
@@ -497,7 +494,7 @@ bool GraphicPropertyItemConverter::ApplySpecialItem(
                         OUString aPreferredName;
                         aValue >>= aPreferredName;
                         aValue <<= 
PropertyHelper::addTransparencyGradientUniqueNameToTable(
-                            aGradient, m_xNamedPropertyTableFactory, 
aPreferredName );
+                            aGradient, m_xChartModel, aPreferredName );
 
                         if( aValue != GetPropertySet()->getPropertyValue( 
aPropName ))
                         {
@@ -568,7 +565,7 @@ bool GraphicPropertyItemConverter::ApplySpecialItem(
                     OUString aPreferredName;
                     aValue >>= aPreferredName;
                     aValue <<= PropertyHelper::addLineDashUniqueNameToTable(
-                        aLineDash, m_xNamedPropertyTableFactory, 
aPreferredName );
+                        aLineDash, m_xChartModel, aPreferredName );
 
                     GetPropertySet()->setPropertyValue( aPropName, aValue );
                     bChanged = true;
@@ -600,7 +597,7 @@ bool GraphicPropertyItemConverter::ApplySpecialItem(
                         OUString aPreferredName;
                         aValue >>= aPreferredName;
                         aValue <<= 
PropertyHelper::addGradientUniqueNameToTable(
-                            aGradient, m_xNamedPropertyTableFactory, 
aPreferredName );
+                            aGradient, m_xChartModel, aPreferredName );
 
                         GetPropertySet()->setPropertyValue( aPropName, aValue 
);
                         bChanged = true;
@@ -633,7 +630,7 @@ bool GraphicPropertyItemConverter::ApplySpecialItem(
                         OUString aPreferredName;
                         aValue >>= aPreferredName;
                         aValue <<= PropertyHelper::addHatchUniqueNameToTable(
-                            aHatch, m_xNamedPropertyTableFactory, 
aPreferredName );
+                            aHatch, m_xChartModel, aPreferredName );
 
                         GetPropertySet()->setPropertyValue( aPropName, aValue 
);
                         bChanged = true;
@@ -661,7 +658,7 @@ bool GraphicPropertyItemConverter::ApplySpecialItem(
                         OUString aPreferredName;
                         aValue >>= aPreferredName;
                         aValue <<= PropertyHelper::addBitmapUniqueNameToTable(
-                            aBitmap, m_xNamedPropertyTableFactory, 
aPreferredName );
+                            aBitmap, m_xChartModel, aPreferredName );
 
                         GetPropertySet()->setPropertyValue( 
u"FillBitmapName"_ustr , aValue );
                         bChanged = true;
diff --git a/chart2/source/controller/itemsetwrapper/LegendItemConverter.cxx 
b/chart2/source/controller/itemsetwrapper/LegendItemConverter.cxx
index a431efc34349..e58dbc544f37 100644
--- a/chart2/source/controller/itemsetwrapper/LegendItemConverter.cxx
+++ b/chart2/source/controller/itemsetwrapper/LegendItemConverter.cxx
@@ -21,6 +21,7 @@
 #include "SchWhichPairs.hxx"
 #include <GraphicPropertyItemConverter.hxx>
 #include <CharacterPropertyItemConverter.hxx>
+#include <ChartModel.hxx>
 #include <com/sun/star/chart2/LegendPosition.hpp>
 #include <com/sun/star/chart/ChartLegendExpansion.hpp>
 #include <com/sun/star/beans/XPropertySet.hpp>
@@ -40,12 +41,12 @@ LegendItemConverter::LegendItemConverter(
     const css::uno::Reference< css::beans::XPropertySet > & rPropertySet,
     SfxItemPool& rItemPool,
     SdrModel& rDrawModel,
-    const uno::Reference< lang::XMultiServiceFactory > & 
xNamedPropertyContainerFactory,
+    const rtl::Reference< ChartModel > & xChartModel,
     const std::optional<awt::Size>& pRefSize ) :
         ItemConverter( rPropertySet, rItemPool )
 {
     m_aConverters.emplace_back( new GraphicPropertyItemConverter(
-                                 rPropertySet, rItemPool, rDrawModel, 
xNamedPropertyContainerFactory,
+                                 rPropertySet, rItemPool, rDrawModel, 
xChartModel,
                                  GraphicObjectType::LineAndFillProperties ));
     m_aConverters.emplace_back( new CharacterPropertyItemConverter(
                                  rPropertySet, rItemPool, pRefSize,
diff --git 
a/chart2/source/controller/itemsetwrapper/MultipleChartConverters.cxx 
b/chart2/source/controller/itemsetwrapper/MultipleChartConverters.cxx
index 22ba346aaf14..a6dbf6d2faac 100644
--- a/chart2/source/controller/itemsetwrapper/MultipleChartConverters.cxx
+++ b/chart2/source/controller/itemsetwrapper/MultipleChartConverters.cxx
@@ -72,8 +72,7 @@ const WhichRangesContainer& 
AllAxisItemConverter::GetWhichPairs() const
 AllGridItemConverter::AllGridItemConverter(
     const rtl::Reference<::chart::ChartModel> & xChartModel,
     SfxItemPool& rItemPool,
-    SdrModel& rDrawModel,
-    const uno::Reference< lang::XMultiServiceFactory > & 
xNamedPropertyContainerFactory )
+    SdrModel& rDrawModel )
         : MultipleItemConverter( rItemPool )
 {
     rtl::Reference< Diagram > xDiagram( xChartModel->getFirstChartDiagram() );
@@ -81,7 +80,7 @@ AllGridItemConverter::AllGridItemConverter(
     for( rtl::Reference< GridProperties > const & xObjectProperties : 
aElementList )
     {
         m_aConverters.emplace_back( new 
::chart::wrapper::GraphicPropertyItemConverter(
-                                        xObjectProperties, rItemPool, 
rDrawModel, xNamedPropertyContainerFactory,
+                                        xObjectProperties, rItemPool, 
rDrawModel, xChartModel,
                                         
::chart::wrapper::GraphicObjectType::LineProperties ) );
     }
 }
@@ -99,8 +98,7 @@ const WhichRangesContainer& 
AllGridItemConverter::GetWhichPairs() const
 AllDataLabelItemConverter::AllDataLabelItemConverter(
     const rtl::Reference<::chart::ChartModel> & xChartModel,
     SfxItemPool& rItemPool,
-    SdrModel& rDrawModel,
-    const uno::Reference< lang::XMultiServiceFactory > & 
xNamedPropertyContainerFactory )
+    SdrModel& rDrawModel )
         : MultipleItemConverter( rItemPool )
 {
     std::vector< rtl::Reference< DataSeries > > aSeriesList =
@@ -117,7 +115,7 @@ AllDataLabelItemConverter::AllDataLabelItemConverter(
         m_aConverters.emplace_back(
             new ::chart::wrapper::DataPointItemConverter(
                 xChartModel, xContext, series, series, rItemPool, rDrawModel,
-                xNamedPropertyContainerFactory, 
GraphicObjectType::FilledDataPoint,
+                GraphicObjectType::FilledDataPoint,
                 std::nullopt, true, false, 0, true, nNumberFormat, 
nPercentNumberFormat));
     }
 }
@@ -135,8 +133,7 @@ const WhichRangesContainer& 
AllDataLabelItemConverter::GetWhichPairs() const
 AllTitleItemConverter::AllTitleItemConverter(
     const rtl::Reference<::chart::ChartModel> & xChartModel,
     SfxItemPool& rItemPool,
-    SdrModel& rDrawModel,
-    const uno::Reference< lang::XMultiServiceFactory > & 
xNamedPropertyContainerFactory )
+    SdrModel& rDrawModel )
         : MultipleItemConverter( rItemPool )
 {
     for(sal_Int32 nTitle = TitleHelper::TITLE_BEGIN; nTitle < 
TitleHelper::NORMAL_TITLE_END; nTitle++ )
@@ -147,7 +144,7 @@ AllTitleItemConverter::AllTitleItemConverter(
         m_aConverters.emplace_back(
             new ::chart::wrapper::TitleItemConverter(
                 uno::Reference< beans::XPropertySet >( xTitle ),
-                rItemPool, rDrawModel, xNamedPropertyContainerFactory, 
std::nullopt));
+                rItemPool, rDrawModel, xChartModel, std::nullopt));
     }
 }
 
diff --git 
a/chart2/source/controller/itemsetwrapper/RegressionCurveItemConverter.cxx 
b/chart2/source/controller/itemsetwrapper/RegressionCurveItemConverter.cxx
index 684c6fc1cdd5..721f1247f439 100644
--- a/chart2/source/controller/itemsetwrapper/RegressionCurveItemConverter.cxx
+++ b/chart2/source/controller/itemsetwrapper/RegressionCurveItemConverter.cxx
@@ -23,6 +23,7 @@
 #include "SchWhichPairs.hxx"
 #include <GraphicPropertyItemConverter.hxx>
 #include <DataSeries.hxx>
+#include <ChartModel.hxx>
 
 #include <com/sun/star/chart2/XRegressionCurve.hpp>
 #include <osl/diagnose.h>
@@ -91,11 +92,11 @@ RegressionCurveItemConverter::RegressionCurveItemConverter(
     rtl::Reference< DataSeries > xContainer,
     SfxItemPool& rItemPool,
     SdrModel& rDrawModel,
-    const uno::Reference< lang::XMultiServiceFactory > & 
xNamedPropertyContainerFactory ) :
+    const rtl::Reference< ChartModel > & xChartModel ) :
         ItemConverter( rPropertySet, rItemPool ),
         m_spGraphicConverter( std::make_shared<GraphicPropertyItemConverter>(
                                   rPropertySet, rItemPool, rDrawModel,
-                                  xNamedPropertyContainerFactory,
+                                  xChartModel,
                                   GraphicObjectType::LineProperties )),
         m_xCurveContainer(std::move( xContainer ))
 {}
diff --git 
a/chart2/source/controller/itemsetwrapper/RegressionEquationItemConverter.cxx 
b/chart2/source/controller/itemsetwrapper/RegressionEquationItemConverter.cxx
index 239bf2ecadf9..5487d8d2e757 100644
--- 
a/chart2/source/controller/itemsetwrapper/RegressionEquationItemConverter.cxx
+++ 
b/chart2/source/controller/itemsetwrapper/RegressionEquationItemConverter.cxx
@@ -22,6 +22,7 @@
 #include <ItemPropertyMap.hxx>
 #include <GraphicPropertyItemConverter.hxx>
 #include <CharacterPropertyItemConverter.hxx>
+#include <ChartModel.hxx>
 #include <unonames.hxx>
 #include <com/sun/star/beans/XPropertySet.hpp>
 
@@ -48,13 +49,13 @@ 
RegressionEquationItemConverter::RegressionEquationItemConverter(
     const css::uno::Reference< css::beans::XPropertySet > & rPropertySet,
     SfxItemPool& rItemPool,
     SdrModel& rDrawModel,
-    const uno::Reference< lang::XMultiServiceFactory > & 
xNamedPropertyContainerFactory,
+    const rtl::Reference< ChartModel > & xChartModel,
     const std::optional<awt::Size>& pRefSize ) :
         ItemConverter( rPropertySet, rItemPool )
 {
     m_aConverters.emplace_back( new GraphicPropertyItemConverter(
                                  rPropertySet, rItemPool, rDrawModel,
-                                 xNamedPropertyContainerFactory,
+                                 xChartModel,
                                  GraphicObjectType::LineAndFillProperties ));
 
     m_aConverters.emplace_back(
diff --git a/chart2/source/controller/itemsetwrapper/TitleItemConverter.cxx 
b/chart2/source/controller/itemsetwrapper/TitleItemConverter.cxx
index 004309868dbe..23341ac287bd 100644
--- a/chart2/source/controller/itemsetwrapper/TitleItemConverter.cxx
+++ b/chart2/source/controller/itemsetwrapper/TitleItemConverter.cxx
@@ -23,6 +23,7 @@
 #include <GraphicPropertyItemConverter.hxx>
 #include <CharacterPropertyItemConverter.hxx>
 #include <MultipleItemConverter.hxx>
+#include <ChartModel.hxx>
 #include <svx/sdangitm.hxx>
 #include <rtl/math.hxx>
 
@@ -91,13 +92,13 @@ TitleItemConverter::TitleItemConverter(
     const uno::Reference<beans::XPropertySet> & rPropertySet,
     SfxItemPool& rItemPool,
     SdrModel& rDrawModel,
-    const uno::Reference< lang::XMultiServiceFactory > & 
xNamedPropertyContainerFactory,
+    const rtl::Reference< ChartModel > & xChartModel,
     const std::optional<awt::Size>& pRefSize ) :
         ItemConverter( rPropertySet, rItemPool )
 {
     m_aConverters.emplace_back( new GraphicPropertyItemConverter(
                                  rPropertySet, rItemPool, rDrawModel,
-                                 xNamedPropertyContainerFactory,
+                                 xChartModel,
                                  GraphicObjectType::LineAndFillProperties ));
 
     // CharacterProperties are not at the title but at its contained 
XFormattedString objects
diff --git a/chart2/source/controller/main/ChartController_Insert.cxx 
b/chart2/source/controller/main/ChartController_Insert.cxx
index 7d12aa9097b9..5b930ee7190c 100644
--- a/chart2/source/controller/main/ChartController_Insert.cxx
+++ b/chart2/source/controller/main/ChartController_Insert.cxx
@@ -382,8 +382,7 @@ void 
ChartController::executeDispatch_InsertMenu_DataLabels()
         wrapper::AllDataLabelItemConverter aItemConverter(
             getChartModel(),
             m_pDrawModelWrapper->GetItemPool(),
-            m_pDrawModelWrapper->getSdrModel(),
-            getChartModel() );
+            m_pDrawModelWrapper->getSdrModel() );
         SfxItemSet aItemSet = aItemConverter.CreateEmptyItemSet();
         aItemConverter.FillItemSet( aItemSet );
 
@@ -543,8 +542,7 @@ void ChartController::executeDispatch_InsertErrorBars( bool 
bYError )
         // get an appropriate item converter
         auto aItemConverter = std::make_shared<wrapper::ErrorBarItemConverter> 
(
             getChartModel(), xErrorBarProp, 
m_pDrawModelWrapper->getSdrModel().GetItemPool(),
-            m_pDrawModelWrapper->getSdrModel(),
-            getChartModel() );
+            m_pDrawModelWrapper->getSdrModel() );
 
         // open dialog
         SfxItemSet aItemSet = aItemConverter->CreateEmptyItemSet();
diff --git a/chart2/source/controller/main/ChartController_Properties.cxx 
b/chart2/source/controller/main/ChartController_Properties.cxx
index 2225dcdbcbb6..762d730d0539 100644
--- a/chart2/source/controller/main/ChartController_Properties.cxx
+++ b/chart2/source/controller/main/ChartController_Properties.cxx
@@ -234,7 +234,6 @@ wrapper::ItemConverter* createItemConverter(
 
                 pItemConverter =  new wrapper::DataPointItemConverter( 
xChartModel, xContext,
                                         xObjectProperties, xSeries, 
rDrawModel.GetItemPool(), rDrawModel,
-                                        xChartModel,
                                         eMapTo, pRefSize, bDataSeries, 
bUseSpecialFillColor, nSpecialFillColor, true,
                                         nNumberFormat, nPercentNumberFormat, 
nPointIndex );
                 break;
@@ -253,7 +252,7 @@ wrapper::ItemConverter* createItemConverter(
             case OBJECTTYPE_DATA_ERRORS_Z:
                 pItemConverter =  new wrapper::ErrorBarItemConverter(
                     xChartModel, xObjectProperties, rDrawModel.GetItemPool(),
-                    rDrawModel, xChartModel);
+                    rDrawModel );
                 break;
 
             case OBJECTTYPE_DATA_CURVE:
@@ -295,7 +294,7 @@ wrapper::ItemConverter* createItemConverter(
         {
             case OBJECTTYPE_TITLE:
                 pItemConverter =  new wrapper::AllTitleItemConverter( 
xChartModel, rDrawModel.GetItemPool(),
-                                                                     
rDrawModel, xChartModel);
+                                                                     
rDrawModel );
                 break;
             case OBJECTTYPE_AXIS:
             {
@@ -311,7 +310,7 @@ wrapper::ItemConverter* createItemConverter(
             case OBJECTTYPE_GRID:
             case OBJECTTYPE_SUBGRID:
                 pItemConverter =  new wrapper::AllGridItemConverter( 
xChartModel, rDrawModel.GetItemPool(),
-                                                                     
rDrawModel, xChartModel);
+                                                                     
rDrawModel );
                 break;
             default: //for this type it is not supported to change all 
elements at once
                 break;
@@ -760,7 +759,6 @@ void 
ChartController::executeDlg_ObjectProperties_withUndoGuard(
                                         , xObjectProperties, 
ObjectIdentifier::getDataSeriesForCID( rObjectCID, xChartDoc )
                                         , 
m_pDrawModelWrapper->getSdrModel().GetItemPool()
                                         , m_pDrawModelWrapper->getSdrModel()
-                                        , xChartDoc
                                         , 
wrapper::GraphicObjectType::FilledDataPoint );
 
             SfxItemSet 
aSymbolShapeProperties(aSymbolItemConverter.CreateEmptyItemSet() );

Reply via email to