[Libreoffice-commits] core.git: basic/source offapi/com

Stephan Bergmann sbergman at redhat.com
Wed May 29 08:37:56 PDT 2013


 basic/source/classes/eventatt.cxx          |   44 ++++++++++++++---------------
 offapi/com/sun/star/awt/DialogProvider.idl |    5 +--
 2 files changed, 25 insertions(+), 24 deletions(-)

New commits:
commit 69e697deefc9bebefa2daac4ae4b8cde5ae742f1
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Wed May 29 17:35:06 2013 +0200

    Improve DialogProvider service ctor
    
    ...recently introduced with c75a46fbd0ba4daf857fcd7d70badeed5aae8e28 "fdo#46808,
    use DialogProvider service constructor."  The general theme of this additional
    ctor appears to be more about "Scripting" than "Listener," and the DialogLib
    argument is actually expected to be of XNameContainer type.
    
    Change-Id: Iea7d906d9b2ffc3b3ee5b2cbfaf7c58191037c9b

diff --git a/basic/source/classes/eventatt.cxx b/basic/source/classes/eventatt.cxx
index f74bb95..68ba831 100644
--- a/basic/source/classes/eventatt.cxx
+++ b/basic/source/classes/eventatt.cxx
@@ -330,9 +330,9 @@ void BasicScriptListener_Impl::firing_impl( const ScriptEvent& aScriptEvent, Any
     }
 }
 
-Any implFindDialogLibForDialog( const Any& rDlgAny, SbxObject* pBasic )
+css::uno::Reference< css::container::XNameContainer > implFindDialogLibForDialog( const Any& rDlgAny, SbxObject* pBasic )
 {
-    Any aRetDlgLibAny;
+    css::uno::Reference< css::container::XNameContainer > aRetDlgLib;
 
     SbxVariable* pDlgLibContVar = pBasic->Find(OUString("DialogLibraries"), SbxCLASS_OBJECT);
     if( pDlgLibContVar && pDlgLibContVar->ISA(SbUnoObject) )
@@ -356,20 +356,20 @@ Any implFindDialogLibForDialog( const Any& rDlgAny, SbxObject* pBasic )
 
                 Any aDlgLibAny = xDlgLibContNameAccess->getByName( pLibNames[ iLib ] );
 
-                Reference< XNameAccess > xDlgLibNameAccess( aDlgLibAny, UNO_QUERY );
-                OSL_ENSURE( xDlgLibNameAccess.is(), "implFindDialogLibForDialog: invalid dialog lib!" );
-                if( xDlgLibNameAccess.is() )
+                Reference< XNameContainer > xDlgLibNameCont( aDlgLibAny, UNO_QUERY );
+                OSL_ENSURE( xDlgLibNameCont.is(), "implFindDialogLibForDialog: invalid dialog lib!" );
+                if( xDlgLibNameCont.is() )
                 {
-                    Sequence< OUString > aDlgNames = xDlgLibNameAccess->getElementNames();
+                    Sequence< OUString > aDlgNames = xDlgLibNameCont->getElementNames();
                     const OUString* pDlgNames = aDlgNames.getConstArray();
                     sal_Int32 nDlgNameCount = aDlgNames.getLength();
 
                     for( sal_Int32 iDlg = 0 ; iDlg < nDlgNameCount ; iDlg++ )
                     {
-                        Any aDlgAny = xDlgLibNameAccess->getByName( pDlgNames[ iDlg ] );
+                        Any aDlgAny = xDlgLibNameCont->getByName( pDlgNames[ iDlg ] );
                         if( aDlgAny == rDlgAny )
                         {
-                            aRetDlgLibAny = aDlgLibAny;
+                            aRetDlgLib = xDlgLibNameCont;
                             break;
                         }
                     }
@@ -378,12 +378,12 @@ Any implFindDialogLibForDialog( const Any& rDlgAny, SbxObject* pBasic )
         }
     }
 
-    return aRetDlgLibAny;
+    return aRetDlgLib;
 }
 
-Any implFindDialogLibForDialogBasic( const Any& aAnyISP, SbxObject* pBasic, StarBASIC*& pFoundBasic )
+css::uno::Reference< css::container::XNameContainer > implFindDialogLibForDialogBasic( const Any& aAnyISP, SbxObject* pBasic, StarBASIC*& pFoundBasic )
 {
-    Any aDlgLibAny;
+    css::uno::Reference< css::container::XNameContainer > aDlgLib;
     // Find dialog library for dialog, direct access is not possible here
     StarBASIC* pStartedBasic = (StarBASIC*)pBasic;
     SbxObject* pParentBasic = pStartedBasic ? pStartedBasic->GetParent() : NULL;
@@ -403,19 +403,19 @@ Any implFindDialogLibForDialogBasic( const Any& aAnyISP, SbxObject* pBasic, Star
     }
     if( pSearchBasic1 )
     {
-        aDlgLibAny = implFindDialogLibForDialog( aAnyISP, pSearchBasic1 );
+        aDlgLib = implFindDialogLibForDialog( aAnyISP, pSearchBasic1 );
 
-        if ( aDlgLibAny.hasValue() )
+        if ( aDlgLib.is() )
             pFoundBasic = (StarBASIC*)pSearchBasic1;
 
         else if( pSearchBasic2 )
         {
-            aDlgLibAny = implFindDialogLibForDialog( aAnyISP, pSearchBasic2 );
-            if ( aDlgLibAny.hasValue() )
+            aDlgLib = implFindDialogLibForDialog( aAnyISP, pSearchBasic2 );
+            if ( aDlgLib.is() )
                 pFoundBasic = (StarBASIC*)pSearchBasic2;
         }
     }
-    return aDlgLibAny;
+    return aDlgLib;
 }
 
 void RTL_Impl_CreateUnoDialog( StarBASIC* pBasic, SbxArray& rPar, sal_Bool bWrite )
@@ -489,12 +489,12 @@ void RTL_Impl_CreateUnoDialog( StarBASIC* pBasic, SbxArray& rPar, sal_Bool bWrit
         {}
     }
 
-    Any aDlgLibAny;
+    css::uno::Reference< css::container::XNameContainer > aDlgLib;
     bool bDocDialog = false;
     StarBASIC* pFoundBasic = NULL;
     OSL_TRACE("About to try get a hold of ThisComponent");
     Reference< frame::XModel > xModel = StarBASIC::GetModelFromBasic( GetSbData()->pInst->GetBasic() ) ;
-    aDlgLibAny = implFindDialogLibForDialogBasic( aAnyISP, GetSbData()->pInst->GetBasic(), pFoundBasic );
+    aDlgLib = implFindDialogLibForDialogBasic( aAnyISP, GetSbData()->pInst->GetBasic(), pFoundBasic );
     // If we found the dialog then it belongs to the Search basic
     if ( !pFoundBasic )
     {
@@ -515,9 +515,9 @@ void RTL_Impl_CreateUnoDialog( StarBASIC* pBasic, SbxArray& rPar, sal_Bool bWrit
                     BasicManager* pMgr = basic::BasicManagerRepository::getDocumentBasicManager( xNextModel );
                     if ( pMgr )
                     {
-                        aDlgLibAny = implFindDialogLibForDialogBasic( aAnyISP, pMgr->GetLib(0), pFoundBasic );
+                        aDlgLib = implFindDialogLibForDialogBasic( aAnyISP, pMgr->GetLib(0), pFoundBasic );
                     }
-                    if ( aDlgLibAny.hasValue() )
+                    if ( aDlgLib.is() )
                     {
                         bDocDialog = true;
                         xModel = xNextModel;
@@ -539,9 +539,9 @@ void RTL_Impl_CreateUnoDialog( StarBASIC* pBasic, SbxArray& rPar, sal_Bool bWrit
     {
        Reference< XDialogProvider >  xDlgProv;;
        if( bDocDialog )
-           xDlgProv = css::awt::DialogProvider::createWithModelAndListener( xContext, xModel, xInput, aDlgLibAny, xScriptListener );
+           xDlgProv = css::awt::DialogProvider::createWithModelAndScripting( xContext, xModel, xInput, aDlgLib, xScriptListener );
        else
-           xDlgProv = css::awt::DialogProvider::createWithModelAndListener( xContext, uno::Reference< frame::XModel >(), xInput, aDlgLibAny, xScriptListener );
+           xDlgProv = css::awt::DialogProvider::createWithModelAndScripting( xContext, uno::Reference< frame::XModel >(), xInput, aDlgLib, xScriptListener );
 
        xCntrl.set( xDlgProv->createDialog(OUString() ), UNO_QUERY_THROW );
        // Add dialog model to dispose vector
diff --git a/offapi/com/sun/star/awt/DialogProvider.idl b/offapi/com/sun/star/awt/DialogProvider.idl
index 7288685..e64a4e8 100644
--- a/offapi/com/sun/star/awt/DialogProvider.idl
+++ b/offapi/com/sun/star/awt/DialogProvider.idl
@@ -21,6 +21,7 @@
 #define __com_sun_star_awt_DialogProvider_idl__
 
 #include <com/sun/star/awt/XDialogProvider.idl>
+#include <com/sun/star/container/XNameContainer.idl>
 #include <com/sun/star/frame/XModel.idl>
 #include <com/sun/star/io/XInputStream.idl>
 #include <com/sun/star/script/XScriptListener.idl>
@@ -38,9 +39,9 @@ service DialogProvider : com::sun::star::awt::XDialogProvider
 
     createWithModel([in] com::sun::star::frame::XModel Model);
 
-    createWithModelAndListener([in] com::sun::star::frame::XModel Model,
+    createWithModelAndScripting([in] com::sun::star::frame::XModel Model,
                      [in] com::sun::star::io::XInputStream InStream,
-                     [in] any DialogLib,
+                     [in] com::sun::star::container::XNameContainer DialogLib,
                      [in] com::sun::star::script::XScriptListener ScriptListener);
 };
 


More information about the Libreoffice-commits mailing list