sw/source/writerfilter/dmapper/DomainMapper.cxx      |    6 -
 sw/source/writerfilter/dmapper/DomainMapper.hxx      |    2 
 sw/source/writerfilter/dmapper/DomainMapper_Impl.cxx |   89 +++++++++----------
 sw/source/writerfilter/dmapper/DomainMapper_Impl.hxx |    5 -
 sw/source/writerfilter/dmapper/PropertyMap.cxx       |    5 -
 sw/source/writerfilter/dmapper/SdtHelper.cxx         |   10 +-
 sw/source/writerfilter/dmapper/StyleSheetTable.cxx   |    2 
 7 files changed, 56 insertions(+), 63 deletions(-)

New commits:
commit 556af85ae78d1d11f8b3c59953342b770e25620c
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Wed Apr 24 11:43:44 2024 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Fri Apr 26 13:19:35 2024 +0200

    use more concrete UNO in writerfilter
    
    elimate the TextFactory stuff, since it is the same object as the
    TextDocument
    
    Change-Id: I10577f2ac9ffa4d079b96052f7879a83e38f1cf6
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166693
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/sw/source/writerfilter/dmapper/DomainMapper.cxx 
b/sw/source/writerfilter/dmapper/DomainMapper.cxx
index 16106cf0e5d5..5a9f84aa3b0d 100644
--- a/sw/source/writerfilter/dmapper/DomainMapper.cxx
+++ b/sw/source/writerfilter/dmapper/DomainMapper.cxx
@@ -172,7 +172,7 @@ DomainMapper::DomainMapper( const uno::Reference< 
uno::XComponentContext >& xCon
         // the intended font to provide best layout match.
         try
         {
-            uno::Reference< beans::XPropertySet > 
xDefProps(GetTextFactory()->createInstance("com.sun.star.text.Defaults"),
+            uno::Reference< beans::XPropertySet > 
xDefProps(GetTextDocument()->createInstance("com.sun.star.text.Defaults"),
                 uno::UNO_QUERY_THROW);
             xDefProps->setPropertyValue(getPropertyName(PROP_CHAR_FONT_NAME), 
css::uno::Any(OUString("Calibri")));
             xDefProps->setPropertyValue(getPropertyName(PROP_CHAR_HEIGHT), 
css::uno::Any(double(11)));
@@ -4859,9 +4859,9 @@ bool DomainMapper::IsRTFImport() const
     return m_pImpl->IsRTFImport();
 }
 
-uno::Reference < lang::XMultiServiceFactory > const & 
DomainMapper::GetTextFactory() const
+rtl::Reference<SwXTextDocument> const & DomainMapper::GetTextDocument() const
 {
-    return m_pImpl->GetTextFactory();
+    return m_pImpl->GetTextDocument();
 }
 
 uno::Reference< text::XTextRange > DomainMapper::GetCurrentTextRange()
diff --git a/sw/source/writerfilter/dmapper/DomainMapper.hxx 
b/sw/source/writerfilter/dmapper/DomainMapper.hxx
index de43478bae7f..ecd5c4459273 100644
--- a/sw/source/writerfilter/dmapper/DomainMapper.hxx
+++ b/sw/source/writerfilter/dmapper/DomainMapper.hxx
@@ -103,7 +103,7 @@ public:
 
     bool IsOOXMLImport() const;
     bool IsRTFImport() const;
-    css::uno::Reference<css::lang::XMultiServiceFactory> const & 
GetTextFactory() const;
+    rtl::Reference<SwXTextDocument> const & GetTextDocument() const;
     css::uno::Reference<css::text::XTextRange> GetCurrentTextRange();
 
     OUString getOrCreateCharStyle( PropertyValueVector_t& rCharProperties, 
bool bAlwaysCreate );
diff --git a/sw/source/writerfilter/dmapper/DomainMapper_Impl.cxx 
b/sw/source/writerfilter/dmapper/DomainMapper_Impl.cxx
index 2a4f3052df81..aab27ef1a4bc 100644
--- a/sw/source/writerfilter/dmapper/DomainMapper_Impl.cxx
+++ b/sw/source/writerfilter/dmapper/DomainMapper_Impl.cxx
@@ -333,7 +333,6 @@ DomainMapper_Impl::DomainMapper_Impl(
         m_rDMapper( rDMapper ),
         m_pOOXMLDocument(nullptr),
         m_xTextDocument( xModel ),
-        m_xTextFactory( xModel ),
         m_xComponentContext(std::move( xContext )),
         
m_bForceGenericFields(officecfg::Office::Common::Filter::Microsoft::Import::ForceImportWWFieldsAsGenericFields::get()),
         m_bIsDecimalComma( false ),
@@ -526,9 +525,9 @@ uno::Reference< text::XText > const & 
DomainMapper_Impl::GetBodyText()
 
 uno::Reference< beans::XPropertySet > const & 
DomainMapper_Impl::GetDocumentSettings()
 {
-    if( !m_xDocumentSettings.is() && m_xTextFactory.is())
+    if( !m_xDocumentSettings.is() && m_xTextDocument.is())
     {
-        m_xDocumentSettings.set( 
m_xTextFactory->createInstance("com.sun.star.document.Settings"), 
uno::UNO_QUERY );
+        m_xDocumentSettings.set( 
m_xTextDocument->createInstance("com.sun.star.document.Settings"), 
uno::UNO_QUERY );
     }
     return m_xDocumentSettings;
 }
@@ -875,7 +874,7 @@ void DomainMapper_Impl::RemoveLastParagraph( )
                 {
                     // Yes, it was removed. Restore
                     uno::Reference<text::XTextContent> xBookmark(
-                        
m_xTextFactory->createInstance("com.sun.star.text.Bookmark"),
+                        
m_xTextDocument->createInstance("com.sun.star.text.Bookmark"),
                         uno::UNO_QUERY_THROW);
 
                     uno::Reference<container::XNamed> xBkmNamed(xBookmark,
@@ -1083,7 +1082,7 @@ void DomainMapper_Impl::PopSdt()
     }
 
     uno::Reference<text::XTextContent> xContentControl(
-        m_xTextFactory->createInstance("com.sun.star.text.ContentControl"), 
uno::UNO_QUERY);
+        m_xTextDocument->createInstance("com.sun.star.text.ContentControl"), 
uno::UNO_QUERY);
     uno::Reference<beans::XPropertySet> xContentControlProps(xContentControl, 
uno::UNO_QUERY);
     if (m_pSdtHelper->GetShowingPlcHdr())
     {
@@ -1581,7 +1580,7 @@ ListsManager::Pointer const & 
DomainMapper_Impl::GetListTable()
 {
     if(!m_pListTable)
         m_pListTable =
-            new ListsManager( m_rDMapper, m_xTextFactory );
+            new ListsManager( m_rDMapper, m_xTextDocument );
     return m_pListTable;
 }
 
@@ -3379,7 +3378,7 @@ void DomainMapper_Impl::appendOLE( const OUString& 
rStreamName, const std::share
 {
     try
     {
-        uno::Reference< text::XTextContent > xOLE( 
m_xTextFactory->createInstance("com.sun.star.text.TextEmbeddedObject"), 
uno::UNO_QUERY_THROW );
+        uno::Reference< text::XTextContent > xOLE( 
m_xTextDocument->createInstance("com.sun.star.text.TextEmbeddedObject"), 
uno::UNO_QUERY_THROW );
         uno::Reference< beans::XPropertySet > xOLEProperties(xOLE, 
uno::UNO_QUERY_THROW);
 
         OUString aCLSID = pOLEHandler->getCLSID();
@@ -3496,7 +3495,7 @@ void DomainMapper_Impl::appendStarMath( const Value& val )
 
     try
     {
-        uno::Reference< text::XTextContent > xStarMath( 
m_xTextFactory->createInstance("com.sun.star.text.TextEmbeddedObject"), 
uno::UNO_QUERY_THROW );
+        uno::Reference< text::XTextContent > xStarMath( 
m_xTextDocument->createInstance("com.sun.star.text.TextEmbeddedObject"), 
uno::UNO_QUERY_THROW );
         uno::Reference< beans::XPropertySet > xStarMathProperties(xStarMath, 
uno::UNO_QUERY_THROW);
 
         xStarMathProperties->setPropertyValue(getPropertyName( 
PROP_EMBEDDED_OBJECT ),
@@ -3675,7 +3674,7 @@ uno::Reference< beans::XPropertySet > 
DomainMapper_Impl::appendTextSectionAfter(
             // to the newly appended paragraph, which will be kept in the end.
             copyAllProps(xEndPara, xNewPara);
 
-            uno::Reference< text::XTextContent > xSection( 
m_xTextFactory->createInstance("com.sun.star.text.TextSection"), 
uno::UNO_QUERY_THROW );
+            uno::Reference< text::XTextContent > xSection( 
m_xTextDocument->createInstance("com.sun.star.text.TextSection"), 
uno::UNO_QUERY_THROW );
             xSection->attach(xCursor);
 
             // Remove the extra paragraph (last inside the section)
@@ -4002,8 +4001,8 @@ void DomainMapper_Impl::PushFootOrEndnote( bool 
bIsFootnote )
         pTopContext->Erase(PROP_CHAR_STYLE_NAME);
 
         uno::Reference< text::XText > xFootnoteText;
-        if (GetTextFactory().is())
-            xFootnoteText.set( GetTextFactory()->createInstance(
+        if (m_xTextDocument)
+            xFootnoteText.set( m_xTextDocument->createInstance(
             bIsFootnote ?
                 OUString( "com.sun.star.text.Footnote" ) : OUString( 
"com.sun.star.text.Endnote" )),
             uno::UNO_QUERY_THROW );
@@ -4249,9 +4248,9 @@ void DomainMapper_Impl::PushAnnotation()
     try
     {
         m_StreamStateStack.top().eSubstreamType = SubstreamType::Annotation;
-        if (!GetTextFactory().is())
+        if (!m_xTextDocument)
             return;
-        m_xAnnotationField.set( GetTextFactory()->createInstance( 
"com.sun.star.text.TextField.Annotation" ),
+        m_xAnnotationField.set( m_xTextDocument->createInstance( 
"com.sun.star.text.TextField.Annotation" ),
                                 uno::UNO_QUERY_THROW );
         uno::Reference< text::XText > xAnnotationText;
         m_xAnnotationField->getPropertyValue("TextRange") >>= xAnnotationText;
@@ -4719,7 +4718,7 @@ void DomainMapper_Impl::PushShapeContext( const 
uno::Reference< drawing::XShape
             m_aAnchoredStack.push(AnchoredContext(xTextContent));
             uno::Reference<beans::XPropertySet> xShapePropertySet(xShape, 
uno::UNO_QUERY);
 
-            
m_StreamStateStack.top().xEmbedded.set(m_xTextFactory->createInstance("com.sun.star.text.TextEmbeddedObject"),
 uno::UNO_QUERY_THROW);
+            
m_StreamStateStack.top().xEmbedded.set(m_xTextDocument->createInstance("com.sun.star.text.TextEmbeddedObject"),
 uno::UNO_QUERY_THROW);
             uno::Reference<beans::XPropertySet> 
xEmbeddedProperties(m_StreamStateStack.top().xEmbedded, uno::UNO_QUERY_THROW);
             
xEmbeddedProperties->setPropertyValue(getPropertyName(PROP_EMBEDDED_OBJECT), 
xShapePropertySet->getPropertyValue(getPropertyName(PROP_EMBEDDED_OBJECT)));
             
xEmbeddedProperties->setPropertyValue(getPropertyName(PROP_ANCHOR_TYPE), 
uno::Any(text::TextContentAnchorType_AS_CHARACTER));
@@ -5157,10 +5156,10 @@ void DomainMapper_Impl::HandleLineBreak(const 
PropertyMapPtr& pPropertyMap)
         return;
     }
 
-    if (GetTextFactory().is())
+    if (m_xTextDocument)
     {
         uno::Reference<text::XTextContent> xLineBreak(
-            GetTextFactory()->createInstance("com.sun.star.text.LineBreak"), 
uno::UNO_QUERY);
+            m_xTextDocument->createInstance("com.sun.star.text.LineBreak"), 
uno::UNO_QUERY);
         uno::Reference<beans::XPropertySet> xLineBreakProps(xLineBreak, 
uno::UNO_QUERY);
         xLineBreakProps->setPropertyValue("Clear", 
uno::Any(*m_StreamStateStack.top().oLineBreakClear));
         appendTextContent(xLineBreak, pPropertyMap->GetPropertyValues());
@@ -5770,7 +5769,7 @@ void DomainMapper_Impl::PushTextBoxContent()
     try
     {
         uno::Reference<text::XTextFrame> xTBoxFrame(
-            m_xTextFactory->createInstance("com.sun.star.text.TextFrame"), 
uno::UNO_QUERY_THROW);
+            m_xTextDocument->createInstance("com.sun.star.text.TextFrame"), 
uno::UNO_QUERY_THROW);
         uno::Reference<container::XNamed>(xTBoxFrame, uno::UNO_QUERY_THROW)
             ->setName("textbox" + 
OUString::number(m_xPendingTextBoxFrames.size() + 1));
         
uno::Reference<text::XTextAppendAndConvert>(m_aTextAppendStack.top().xTextAppend,
@@ -5936,10 +5935,10 @@ uno::Reference<beans::XPropertySet> 
DomainMapper_Impl::FindOrCreateFieldMaster(c
         //get the master
         xMaster.set(xFieldMasterAccess->getByName(sFieldMasterName), 
uno::UNO_QUERY_THROW);
     }
-    else if( m_xTextFactory.is() )
+    else if( m_xTextDocument )
     {
         //create the master
-        xMaster.set( m_xTextFactory->createInstance(sFieldMasterService), 
uno::UNO_QUERY_THROW);
+        xMaster.set( m_xTextDocument->createInstance(sFieldMasterService), 
uno::UNO_QUERY_THROW);
         if ( !bIsMergeField || sDatabaseDataSourceName.isEmpty() )
         {
             //set the master's name
@@ -6716,8 +6715,8 @@ void DomainMapper_Impl::handleAuthor
     {
         sServiceName += sFieldServiceName;
     }
-    if (m_xTextFactory.is())
-        xFieldInterface = m_xTextFactory->createInstance(sServiceName);
+    if (m_xTextDocument)
+        xFieldInterface = m_xTextDocument->createInstance(sServiceName);
     uno::Reference<beans::XPropertySet> xFieldProperties( xFieldInterface, 
uno::UNO_QUERY_THROW);
     if( bIsCustomField )
     {
@@ -7045,7 +7044,7 @@ void DomainMapper_Impl::handleToc
 
     uno::Reference<beans::XPropertySet> xTOC;
 
-    if (m_xTextFactory.is() && ! m_aTextAppendStack.empty())
+    if (m_xTextDocument && ! m_aTextAppendStack.empty())
     {
         const auto& xTextAppend = GetTopTextAppend();
         if (aTocTitle.isEmpty() || bTableOfFigures)
@@ -7275,7 +7274,7 @@ uno::Reference<beans::XPropertySet> 
DomainMapper_Impl::createSectionForRange(
             //the paragraph after this new section is already inserted
             if (stepLeft)
                 xCursor->goLeft(1, true);
-            uno::Reference< text::XTextContent > xSection( 
m_xTextFactory->createInstance(sObjectType), uno::UNO_QUERY_THROW );
+            uno::Reference< text::XTextContent > xSection( 
m_xTextDocument->createInstance(sObjectType), uno::UNO_QUERY_THROW );
             try
             {
                 xSection->attach( uno::Reference< text::XTextRange >( xCursor, 
uno::UNO_QUERY_THROW) );
@@ -7584,9 +7583,9 @@ void DomainMapper_Impl::CloseFieldCommand()
                 TagLogger::getInstance().endElement();
 #endif
 
-                if (m_xTextFactory.is())
+                if (m_xTextDocument)
                 {
-                    xFieldInterface = 
m_xTextFactory->createInstance(sServiceName);
+                    xFieldInterface = 
m_xTextDocument->createInstance(sServiceName);
                     xFieldProperties.set( xFieldInterface, 
uno::UNO_QUERY_THROW);
                 }
             }
@@ -7687,9 +7686,9 @@ void DomainMapper_Impl::CloseFieldCommand()
                     OUString aCommand = pContext->GetCommand().trim();
 
                     msfilter::util::EquationResult 
aResult(msfilter::util::ParseCombinedChars(aCommand));
-                    if (!aResult.sType.isEmpty() && m_xTextFactory.is())
+                    if (!aResult.sType.isEmpty() && m_xTextDocument)
                     {
-                        xFieldInterface = 
m_xTextFactory->createInstance("com.sun.star.text.TextField." + aResult.sType);
+                        xFieldInterface = 
m_xTextDocument->createInstance("com.sun.star.text.TextField." + aResult.sType);
                         xFieldProperties =
                             uno::Reference< beans::XPropertySet >( 
xFieldInterface,
                                 uno::UNO_QUERY_THROW);
@@ -8067,9 +8066,9 @@ void DomainMapper_Impl::CloseFieldCommand()
                         xFieldProperties->setPropertyValue(
                                 getPropertyName( PROP_REFERENCE_FIELD_PART ), 
uno::Any( nFieldPart ));
                     }
-                    else if( m_xTextFactory.is() )
+                    else if( m_xTextDocument )
                     {
-                        xFieldInterface = 
m_xTextFactory->createInstance("com.sun.star.text.TextField.GetExpression");
+                        xFieldInterface = 
m_xTextDocument->createInstance("com.sun.star.text.TextField.GetExpression");
                         xFieldProperties.set(xFieldInterface, uno::UNO_QUERY);
                         xFieldProperties->setPropertyValue(
                             getPropertyName(PROP_CONTENT),
@@ -8224,7 +8223,7 @@ void DomainMapper_Impl::CloseFieldCommand()
                 break;
                 case FIELD_XE:
                 {
-                    if( !m_xTextFactory.is() )
+                    if( !m_xTextDocument )
                         break;
 
                     // only UserIndexMark can handle user index types defined 
by 
@@ -8235,7 +8234,7 @@ void DomainMapper_Impl::CloseFieldCommand()
                             ? "com.sun.star.text.UserIndexMark"
                             : 
OUString::createFromAscii(aIt->second.cFieldServiceName);
                     uno::Reference< beans::XPropertySet > xTC(
-                            m_xTextFactory->createInstance(sFieldServiceName),
+                            m_xTextDocument->createInstance(sFieldServiceName),
                                     uno::UNO_QUERY_THROW);
 
                     if (!sFirstParam.isEmpty())
@@ -8268,10 +8267,10 @@ void DomainMapper_Impl::CloseFieldCommand()
                     break;
                 case FIELD_CITATION:
                 {
-                    if( !m_xTextFactory.is() )
+                    if( !m_xTextDocument )
                         break;
 
-                    xFieldInterface = m_xTextFactory->createInstance(
+                    xFieldInterface = m_xTextDocument->createInstance(
                               
OUString::createFromAscii(aIt->second.cFieldServiceName));
                     uno::Reference< beans::XPropertySet > xTC(xFieldInterface,
                               uno::UNO_QUERY_THROW);
@@ -8293,11 +8292,11 @@ void DomainMapper_Impl::CloseFieldCommand()
 
                 case FIELD_TC :
                 {
-                    if( !m_xTextFactory.is() )
+                    if( !m_xTextDocument )
                         break;
 
                     uno::Reference< beans::XPropertySet > xTC(
-                        m_xTextFactory->createInstance(
+                        m_xTextDocument->createInstance(
                             
OUString::createFromAscii(aIt->second.cFieldServiceName)),
                             uno::UNO_QUERY_THROW);
                     if (!sFirstParam.isEmpty())
@@ -8348,9 +8347,9 @@ void DomainMapper_Impl::CloseFieldCommand()
              */
             OUString aCode( pContext->GetCommand().trim() );
             // Don't waste resources on wrapping shapes inside a fieldmark.
-            if (sType != "SHAPE" && m_xTextFactory.is() && 
!m_aTextAppendStack.empty())
+            if (sType != "SHAPE" && m_xTextDocument && 
!m_aTextAppendStack.empty())
             {
-                xFieldInterface = 
m_xTextFactory->createInstance("com.sun.star.text.Fieldmark");
+                xFieldInterface = 
m_xTextDocument->createInstance("com.sun.star.text.Fieldmark");
 
                 uno::Reference<text::XFormField> const 
xFormField(xFieldInterface, uno::UNO_QUERY);
                 InsertFieldmark(m_aTextAppendStack, xFormField, 
pContext->GetStartRange(),
@@ -8911,9 +8910,9 @@ void DomainMapper_Impl::StartOrEndBookmark( const 
OUString& rId )
     {
         if( aBookmarkIter != m_aBookmarkMap.end() )
         {
-            if (m_xTextFactory.is())
+            if (m_xTextDocument)
             {
-                uno::Reference< text::XTextContent > xBookmark( 
m_xTextFactory->createInstance( "com.sun.star.text.Bookmark" ), 
uno::UNO_QUERY_THROW );
+                uno::Reference< text::XTextContent > xBookmark( 
m_xTextDocument->createInstance( "com.sun.star.text.Bookmark" ), 
uno::UNO_QUERY_THROW );
                 uno::Reference< text::XTextCursor > xCursor;
                 uno::Reference< text::XText > xText = 
aBookmarkIter->second.m_xTextRange->getText();
                 if( aBookmarkIter->second.m_bIsStartOfText && 
!bIsAfterDummyPara)
@@ -9061,7 +9060,7 @@ void 
DomainMapper_Impl::startOrEndPermissionRange(sal_Int32 permissinId)
         }
         else
         {
-            if (m_xTextFactory.is())
+            if (m_xTextDocument)
             {
                 uno::Reference< text::XTextCursor > xCursor;
                 uno::Reference< text::XText > xText = 
aPermIter->second.m_xTextRange->getText();
@@ -9087,7 +9086,7 @@ void 
DomainMapper_Impl::startOrEndPermissionRange(sal_Int32 permissinId)
                 }
 
                 // create a new bookmark using specific bookmark name pattern 
for permissions
-                uno::Reference< text::XTextContent > 
xPerm(m_xTextFactory->createInstance("com.sun.star.text.Bookmark"), 
uno::UNO_QUERY_THROW);
+                uno::Reference< text::XTextContent > 
xPerm(m_xTextDocument->createInstance("com.sun.star.text.Bookmark"), 
uno::UNO_QUERY_THROW);
                 uno::Reference< container::XNamed > xPermNamed(xPerm, 
uno::UNO_QUERY_THROW);
                 xPermNamed->setName(aPermIter->second.createBookmarkName());
 
@@ -9150,7 +9149,7 @@ GraphicImportPtr const & 
DomainMapper_Impl::GetGraphicImport()
 {
     if(!m_pGraphicImport)
     {
-        m_pGraphicImport = new GraphicImport(m_xComponentContext, 
m_xTextFactory, m_rDMapper, m_eGraphicImportType, m_aPositionOffsets, 
m_aAligns, m_aPositivePercentages);
+        m_pGraphicImport = new GraphicImport(m_xComponentContext, 
m_xTextDocument, m_rDMapper, m_eGraphicImportType, m_aPositionOffsets, 
m_aAligns, m_aPositivePercentages);
     }
     return m_pGraphicImport;
 }
@@ -9559,12 +9558,12 @@ void DomainMapper_Impl::RemoveTopRedline( )
 
 void DomainMapper_Impl::ApplySettingsTable()
 {
-    if (!(m_pSettingsTable && m_xTextFactory.is()))
+    if (!(m_pSettingsTable && m_xTextDocument))
         return;
 
     try
     {
-        uno::Reference< beans::XPropertySet > 
xTextDefaults(m_xTextFactory->createInstance("com.sun.star.text.Defaults"), 
uno::UNO_QUERY_THROW );
+        uno::Reference< beans::XPropertySet > 
xTextDefaults(m_xTextDocument->createInstance("com.sun.star.text.Defaults"), 
uno::UNO_QUERY_THROW );
         sal_Int32 nDefTab = m_pSettingsTable->GetDefaultTabStop();
         xTextDefaults->setPropertyValue( getPropertyName( 
PROP_TAB_STOP_DISTANCE ), uno::Any(nDefTab) );
         if (m_pSettingsTable->GetLinkStyles())
@@ -9593,7 +9592,7 @@ void DomainMapper_Impl::ApplySettingsTable()
             m_xTextDocument->setViewData(xBox);
         }
 
-        uno::Reference< beans::XPropertySet > 
xSettings(m_xTextFactory->createInstance("com.sun.star.document.Settings"), 
uno::UNO_QUERY);
+        uno::Reference< beans::XPropertySet > 
xSettings(m_xTextDocument->createInstance("com.sun.star.document.Settings"), 
uno::UNO_QUERY);
 
         if (m_pSettingsTable->GetDoNotExpandShiftReturn())
             xSettings->setPropertyValue( "DoNotJustifyLinesWithManualBreak", 
uno::Any(true) );
diff --git a/sw/source/writerfilter/dmapper/DomainMapper_Impl.hxx 
b/sw/source/writerfilter/dmapper/DomainMapper_Impl.hxx
index 79c4cc8e8f3c..76a861564d62 100644
--- a/sw/source/writerfilter/dmapper/DomainMapper_Impl.hxx
+++ b/sw/source/writerfilter/dmapper/DomainMapper_Impl.hxx
@@ -536,7 +536,6 @@ private:
     OUString m_aBaseUrl;
     rtl::Reference<SwXTextDocument> m_xTextDocument;
     css::uno::Reference<css::beans::XPropertySet> m_xDocumentSettings;
-    css::uno::Reference<css::lang::XMultiServiceFactory> m_xTextFactory;
     css::uno::Reference<css::uno::XComponentContext> m_xComponentContext;
     css::uno::Reference<css::container::XNameContainer> m_xPageStyles1;
     // cache next available number, expensive to repeatedly compute
@@ -714,10 +713,6 @@ public:
     css::uno::Reference<css::container::XNameContainer> const& 
GetParagraphStyles();
     OUString GetUnusedCharacterStyleName();
     css::uno::Reference<css::text::XText> const & GetBodyText();
-    const css::uno::Reference<css::lang::XMultiServiceFactory>& 
GetTextFactory() const
-    {
-        return m_xTextFactory;
-    }
     const rtl::Reference<SwXTextDocument>& GetTextDocument() const
     {
         return m_xTextDocument;
diff --git a/sw/source/writerfilter/dmapper/PropertyMap.cxx 
b/sw/source/writerfilter/dmapper/PropertyMap.cxx
index fd683b5d455b..b416ca400f6f 100644
--- a/sw/source/writerfilter/dmapper/PropertyMap.cxx
+++ b/sw/source/writerfilter/dmapper/PropertyMap.cxx
@@ -480,7 +480,6 @@ SectionPropertyMap::SectionPropertyMap( bool 
bIsFirstSection )
 uno::Reference<beans::XPropertySet> 
SectionPropertyMap::GetPageStyle(DomainMapper_Impl& rDM_Impl)
 {
     const uno::Reference< container::XNameContainer >& xPageStyles = 
rDM_Impl.GetPageStyles();
-    const uno::Reference < lang::XMultiServiceFactory >& xTextFactory = 
rDM_Impl.GetTextFactory();
     uno::Reference<beans::XPropertySet> xReturnPageStyle;
     try
     {
@@ -490,7 +489,7 @@ uno::Reference<beans::XPropertySet> 
SectionPropertyMap::GetPageStyle(DomainMappe
 
             m_sPageStyleName = rDM_Impl.GetUnusedPageStyleName();
 
-            
m_aPageStyle.set(xTextFactory->createInstance("com.sun.star.style.PageStyle"), 
uno::UNO_QUERY );
+            
m_aPageStyle.set(rDM_Impl.GetTextDocument()->createInstance("com.sun.star.style.PageStyle"),
 uno::UNO_QUERY );
             xPageStyles->insertByName(m_sPageStyleName, 
uno::Any(m_aPageStyle));
         }
         else if (!m_aPageStyle.is() && xPageStyles.is())
@@ -1449,7 +1448,7 @@ void 
SectionPropertyMap::CreateEvenOddPageStyleCopy(DomainMapper_Impl& rDM_Impl,
 {
     OUString evenOddStyleName = rDM_Impl.GetUnusedPageStyleName();
     uno::Reference<beans::XPropertySet> evenOddStyle(
-        
rDM_Impl.GetTextFactory()->createInstance("com.sun.star.style.PageStyle"),
+        
rDM_Impl.GetTextDocument()->createInstance("com.sun.star.style.PageStyle"),
         uno::UNO_QUERY);
     // Unfortunately using setParent() does not work for page styles, so make 
a deep copy of the page style.
     uno::Reference<beans::XPropertySet> pageProperties(m_aPageStyle);
diff --git a/sw/source/writerfilter/dmapper/SdtHelper.cxx 
b/sw/source/writerfilter/dmapper/SdtHelper.cxx
index 8d9ffb956f24..06f3262b767e 100644
--- a/sw/source/writerfilter/dmapper/SdtHelper.cxx
+++ b/sw/source/writerfilter/dmapper/SdtHelper.cxx
@@ -285,7 +285,7 @@ void SdtHelper::createDropDownControl()
     {
         // create field
         uno::Reference<css::text::XTextField> xControlModel(
-            
m_rDM_Impl.GetTextFactory()->createInstance("com.sun.star.text.TextField.DropDown"),
+            
m_rDM_Impl.GetTextDocument()->createInstance("com.sun.star.text.TextField.DropDown"),
             uno::UNO_QUERY);
 
         const auto it = std::find_if(
@@ -312,7 +312,7 @@ void SdtHelper::createDropDownControl()
     {
         // create control
         uno::Reference<awt::XControlModel> xControlModel(
-            
m_rDM_Impl.GetTextFactory()->createInstance("com.sun.star.form.component.ComboBox"),
+            
m_rDM_Impl.GetTextDocument()->createInstance("com.sun.star.form.component.ComboBox"),
             uno::UNO_QUERY);
 
         // set properties
@@ -374,7 +374,7 @@ void SdtHelper::createPlainTextControl()
         xCrsr->setString(*oData);
 
     uno::Reference<text::XTextContent> xContentControl(
-        
m_rDM_Impl.GetTextFactory()->createInstance("com.sun.star.text.ContentControl"),
+        
m_rDM_Impl.GetTextDocument()->createInstance("com.sun.star.text.ContentControl"),
         uno::UNO_QUERY);
     uno::Reference<beans::XPropertySet> xContentControlProps(xContentControl, 
uno::UNO_QUERY);
 
@@ -474,7 +474,7 @@ void SdtHelper::createDateContentControl()
     }
 
     uno::Reference<uno::XInterface> xFieldInterface
-        = 
m_rDM_Impl.GetTextFactory()->createInstance("com.sun.star.text.Fieldmark");
+        = 
m_rDM_Impl.GetTextDocument()->createInstance("com.sun.star.text.Fieldmark");
     uno::Reference<text::XFormField> xFormField(xFieldInterface, 
uno::UNO_QUERY);
     uno::Reference<text::XTextContent> xToInsert(xFormField, uno::UNO_QUERY);
     if (!(xFormField.is() && xToInsert.is()))
@@ -523,7 +523,7 @@ void SdtHelper::createControlShape(awt::Size aSize,
                                    const uno::Sequence<beans::PropertyValue>& 
rGrabBag)
 {
     uno::Reference<drawing::XControlShape> xControlShape(
-        
m_rDM_Impl.GetTextFactory()->createInstance("com.sun.star.drawing.ControlShape"),
+        
m_rDM_Impl.GetTextDocument()->createInstance("com.sun.star.drawing.ControlShape"),
         uno::UNO_QUERY);
     xControlShape->setSize(aSize);
     xControlShape->setControl(xControlModel);
diff --git a/sw/source/writerfilter/dmapper/StyleSheetTable.cxx 
b/sw/source/writerfilter/dmapper/StyleSheetTable.cxx
index dc678e065aa9..ff121c5bf227 100644
--- a/sw/source/writerfilter/dmapper/StyleSheetTable.cxx
+++ b/sw/source/writerfilter/dmapper/StyleSheetTable.cxx
@@ -1721,7 +1721,7 @@ void StyleSheetTable::applyDefaults(bool bParaProperties)
         if(!m_pImpl->m_xTextDefaults.is())
         {
             m_pImpl->m_xTextDefaults.set(
-                
m_pImpl->m_rDMapper.GetTextFactory()->createInstance("com.sun.star.text.Defaults"),
+                
m_pImpl->m_rDMapper.GetTextDocument()->createInstance("com.sun.star.text.Defaults"),
                 uno::UNO_QUERY_THROW );
         }
 

Reply via email to