[Libreoffice-commits] .: reportdesign/source

Stephan Bergmann sbergmann at kemper.freedesktop.org
Tue Jul 24 08:36:25 PDT 2012


 reportdesign/source/ui/report/propbrw.cxx |    4 ----
 1 file changed, 4 deletions(-)

New commits:
commit c2617e960e1ff9c132af34b11f2e099f04fb46a3
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Tue Jul 24 17:29:20 2012 +0200

    fdo#52399: Do not needlessly set DefaultContext of global ServiceManager
    
    ...the new cppuhelper/source/defaultbootstrap.cxx now forbids this nonsense with
    a PropertyVetoException.  It looks like the code in propbrw.cxx had always been
    confused about what to do exactly (using xOwnContext as delegate in the call to
    createComponentContext should just be enough) and the useless re-setting of the
    global ServiceManager's DefaultContext happened to work by accident.
    
    Change-Id: I80a87fe9b0948d9aa6dc9e2e63497097044a4959

diff --git a/reportdesign/source/ui/report/propbrw.cxx b/reportdesign/source/ui/report/propbrw.cxx
index 45b4f1a..cb4bd10 100644
--- a/reportdesign/source/ui/report/propbrw.cxx
+++ b/reportdesign/source/ui/report/propbrw.cxx
@@ -173,7 +173,6 @@ PropBrw::PropBrw(const Reference< XMultiServiceFactory >&   _xORB,Window* pParen
                     uno::Reference< uno::XInterface > xDefaultHelpProvider( inspection::DefaultHelpProvider::create( m_xInspectorContext, xInspectorUI ) );
                 }
             }
-            xFactoryProperties->setPropertyValue( ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "DefaultContext" ) ) ,makeAny(xOwnContext));
         }
         catch (Exception&)
         {
@@ -182,9 +181,6 @@ PropBrw::PropBrw(const Reference< XMultiServiceFactory >&   _xORB,Window* pParen
             {
                 ::comphelper::disposeComponent(m_xBrowserController);
                 ::comphelper::disposeComponent(m_xBrowserComponentWindow);
-                Reference< XPropertySet > xFactoryProperties( m_xORB, UNO_QUERY_THROW );
-                if ( xOwnContext.is() )
-                    xFactoryProperties->setPropertyValue( ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "DefaultContext" ) ) ,makeAny(xOwnContext));
             }
             catch(Exception&) { }
             m_xBrowserController.clear();


More information about the Libreoffice-commits mailing list