fdo#46808, use service constructor for i18n::Collator

Change-Id: I15a360723e335345aad09e73fcb0f6815ed9e0d4

2 files changed, 17 insertions, 14 deletions

diff --git a/unotools/source/i18n/collatorwrapper.cxx b/unotools/source/i18n/collatorwrapper.cxx
index 5611ab7b2f39..b64268ffe2cc 100644
--- a/unotools/source/i18n/collatorwrapper.cxx
+++ b/unotools/source/i18n/collatorwrapper.cxx
@@ -19,19 +19,13 @@
 
 #include <unotools/collatorwrapper.hxx>
 #include <tools/debug.hxx>
-
-#include "instance.hxx"
+#include <com/sun/star/i18n/Collator.hpp>
 
 using namespace ::com::sun::star;
 
-CollatorWrapper::CollatorWrapper (
-        const uno::Reference< lang::XMultiServiceFactory > &xServiceFactory)
-    : mxServiceFactory (xServiceFactory)
+CollatorWrapper::CollatorWrapper ( const uno::Reference< uno::XComponentContext > &rxContext )
 {
-    mxInternationalCollator = uno::Reference< i18n::XCollator > (
-        intl_createInstance( xServiceFactory, "com.sun.star.i18n.Collator",
-                             "CollatorWrapper" ), uno::UNO_QUERY );
-    DBG_ASSERT (mxInternationalCollator.is(), "CollatorWrapper: no i18n collator");
+    mxInternationalCollator = i18n::Collator::create( rxContext );
 }
 
 CollatorWrapper::~CollatorWrapper()
diff --git a/unotools/source/i18n/intlwrapper.cxx b/unotools/source/i18n/intlwrapper.cxx
index 075604e92b9c..059f08d9c22a 100644
--- a/unotools/source/i18n/intlwrapper.cxx
+++ b/unotools/source/i18n/intlwrapper.cxx
@@ -24,17 +24,27 @@
 #include <comphelper/processfactory.hxx>
 
 IntlWrapper::IntlWrapper(
-            const ::com::sun::star::uno::Reference< ::com::sun::star::lang::XMultiServiceFactory > & xSF,
+            const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext > & rxContext,
             const LanguageTag& rLanguageTag )
         :
         maLanguageTag( rLanguageTag ),
-        xSMgr( xSF ),
+        m_xContext( rxContext ),
         pLocaleData( NULL ),
         pCollator( NULL ),
         pCaseCollator( NULL )
 {
 }
 
+IntlWrapper::IntlWrapper(
+            const LanguageTag& rLanguageTag )
+        :
+        maLanguageTag( rLanguageTag ),
+        m_xContext( comphelper::getProcessComponentContext() ),
+        pLocaleData( NULL ),
+        pCollator( NULL ),
+        pCaseCollator( NULL )
+{
+}
 
 IntlWrapper::~IntlWrapper()
 {
@@ -46,14 +56,13 @@ IntlWrapper::~IntlWrapper()
 
 void IntlWrapper::ImplNewLocaleData() const
 {
-    ((IntlWrapper*)this)->pLocaleData = new LocaleDataWrapper(
-        comphelper::getComponentContext(xSMgr), maLanguageTag );
+    ((IntlWrapper*)this)->pLocaleData = new LocaleDataWrapper( m_xContext, maLanguageTag );
 }
 
 
 void IntlWrapper::ImplNewCollator( sal_Bool bCaseSensitive ) const
 {
-    CollatorWrapper* p = new CollatorWrapper( xSMgr );
+    CollatorWrapper* p = new CollatorWrapper( m_xContext );
     if ( bCaseSensitive )
     {
         p->loadDefaultCollator( maLanguageTag.getLocale(), 0 );