[Libreoffice-commits] .: ucb/source

Cédric Bosdonnat cbosdo at kemper.freedesktop.org
Wed Jun 27 01:16:33 PDT 2012


 ucb/source/ucp/cmis/cmis_content.cxx  |    7 ++++---
 ucb/source/ucp/cmis/cmis_provider.cxx |    6 ------
 2 files changed, 4 insertions(+), 9 deletions(-)

New commits:
commit b3c48febcd7d9f4574beba99b66e0974bf3a3c0e
Author: Cédric Bosdonnat <cedric.bosdonnat at free.fr>
Date:   Tue Jun 26 16:28:21 2012 +0200

    CMIS UCP: Use the URL host as ID for cached sessions
    
    Using the binding URL as ID for the cached sessions is bad as there may
    be several repositories on the same binding URL... which means different
    libcmis::Session objects.
    
    Change-Id: Id3605ee4fdfbfc32b72615fb573220e94c8748af

diff --git a/ucb/source/ucp/cmis/cmis_content.cxx b/ucb/source/ucp/cmis/cmis_content.cxx
index bb6e1a0..59febd9 100644
--- a/ucb/source/ucp/cmis/cmis_content.cxx
+++ b/ucb/source/ucp/cmis/cmis_content.cxx
@@ -162,13 +162,14 @@ namespace cmis
         m_sURL = m_xIdentifier->getContentIdentifier( );
         cmis::URL url( m_sURL );
 
-        // Look for a cached session
-        m_pSession = pProvider->getSession( url.getBindingUrl( ) );
+        // Look for a cached session, key is binding url + repo id
+        INetURLObject aUrlObj( m_sURL );
+        m_pSession = pProvider->getSession( aUrlObj.GetHost( ) );
         if ( NULL == m_pSession )
         {
             // Initiate a CMIS session and register it as we found nothing
             m_pSession = libcmis::SessionFactory::createSession( url.getSessionParams( ) );
-            pProvider->registerSession( url.getBindingUrl( ), m_pSession );
+            pProvider->registerSession( aUrlObj.GetHost( ), m_pSession );
         }
 
         m_sObjectPath = url.getObjectPath( );
diff --git a/ucb/source/ucp/cmis/cmis_provider.cxx b/ucb/source/ucp/cmis/cmis_provider.cxx
index a13e9f7..f65a1ca 100644
--- a/ucb/source/ucp/cmis/cmis_provider.cxx
+++ b/ucb/source/ucp/cmis/cmis_provider.cxx
@@ -88,16 +88,10 @@ ContentProvider::ContentProvider(
     const uno::Reference< lang::XMultiServiceFactory >& rSMgr )
 : ::ucbhelper::ContentProviderImplHelper( rSMgr )
 {
-#if OSL_DEBUG_LEVEL > 1
-    fprintf(stderr, "ContentProvider::ContentProvider( )\n" );
-#endif
 }
 
 ContentProvider::~ContentProvider()
 {
-#if OSL_DEBUG_LEVEL > 1
-    fprintf(stderr, "ContentProvider::~ContentProvider( )\n" );
-#endif
 }
 
 XINTERFACE_IMPL_3( ContentProvider,


More information about the Libreoffice-commits mailing list