- QgsPluginRegistry - metadata stored in QMap

- address plugins by their unique name:
  - library's basename for c++ plugins
  - module name for python plugins
(before plugins were addressed by their name, causing ambiguity)

- general cleanups in plugin handling



git-svn-id: http://svn.osgeo.org/qgis/trunk/qgis@9526 c8812cc2-4d05-0410-92ff-de0c093fc19c

Author: wonder

src/app/qgisapp.cpp

@@ -119,6 +119,7 @@
 #include "qgspastetransformations.h"
 #include "qgspluginitem.h"
 #include "qgspluginmanager.h"
+#include "qgspluginmetadata.h"
 #include "qgspluginregistry.h"
 #include "qgspoint.h"
 #include "qgsproject.h"
@@ -1846,7 +1847,7 @@ void QgisApp::restoreSessionPlugins( QString thePluginDirString )
   {
     QString myFullPath = thePluginDirString + "/" + myPluginDir[i];
 
-
+    QString baseName = QFileInfo(myFullPath).baseName();
     QLibrary *myLib = new QLibrary( myFullPath );
     bool loaded = myLib->load();
     if ( loaded )
@@ -1858,15 +1859,15 @@ void QgisApp::restoreSessionPlugins( QString thePluginDirString )
       if ( myName && myDescription  && myVersion )
       {
         //check if the plugin was active on last session
-        QString myEntryName = myName();
+        
         // Windows stores a "true" value as a 1 in the registry so we
         // have to use readBoolEntry in this function
 
-        if ( mySettings.value( "/Plugins/" + myEntryName ).toBool() )
+        if ( mySettings.value( "/Plugins/" + baseName ).toBool() )
         {
           //QgsDebugMsg("Loading plugin: " + myEntryName);
 
-          loadPlugin( myName(), myDescription(), myFullPath );
+          loadPlugin( myFullPath, myName() );
         }
         else
         {
@@ -4142,7 +4143,7 @@ void QgisApp::showPluginManager()
       }
       else
       {
-        loadPlugin( plugin.name(), plugin.description(), plugin.fullPath() );
+        loadPlugin( plugin.fullPath(), plugin.name() );
       }
       it++;
     }
@@ -4209,17 +4210,17 @@ void QgisApp::loadPythonPlugin( QString packageName, QString pluginName )
 
 
   QgsPluginRegistry *pRegistry = QgsPluginRegistry::instance();
-
+  
   // is loaded already?
-  if ( pRegistry->library( pluginName ).isEmpty() )
+  if ( ! pRegistry->isLoaded(packageName) )
   {
     mPythonUtils->loadPlugin( packageName );
     mPythonUtils->startPlugin( packageName );
 
     // TODO: test success
 
     // add to plugin registry
-    pRegistry->addPythonPlugin( packageName, pluginName );
+    pRegistry->addPlugin( packageName, QgsPluginMetadata( packageName, pluginName, NULL, true) );
 
     // add to settings
     QSettings settings;
@@ -4230,13 +4231,15 @@ void QgisApp::loadPythonPlugin( QString packageName, QString pluginName )
   }
 }
 
-void QgisApp::loadPlugin( QString name, QString description, QString theFullPathName )
+void QgisApp::loadPlugin( QString theFullPathName, QString name )
 {
   QSettings settings;
   // first check to see if its already loaded
   QgsPluginRegistry *pRegistry = QgsPluginRegistry::instance();
-  QString lib = pRegistry->library( name );
-  if ( lib.length() > 0 )
+  
+  QString baseName = QFileInfo(theFullPathName).baseName();
+  
+  if ( pRegistry->isLoaded(baseName) )
   {
     // plugin is loaded
     // QMessageBox::warning(this, "Already Loaded", description + " is already loaded");
@@ -4269,9 +4272,9 @@ void QgisApp::loadPlugin( QString name, QString description, QString theFullPath
             {
               pl->initGui();
               // add it to the plugin registry
-              pRegistry->addPlugin( myLib->fileName(), name, pl );
+              pRegistry->addPlugin(baseName, QgsPluginMetadata(myLib->fileName(), name, pl) );
               //add it to the qsettings file [ts]
-              settings.setValue( "/Plugins/" + name, true );
+              settings.setValue( "/Plugins/" + baseName, true );
             }
             else
             {
@@ -4280,7 +4283,7 @@ void QgisApp::loadPlugin( QString name, QString description, QString theFullPath
                                     "The following diagnostic information may help the QGIS developers resolve the issue:\n%1." ).arg
                                     ( myError ) );
               //disable it to the qsettings file [ts]
-              settings.setValue( "/Plugins/" + name, false );
+              settings.setValue( "/Plugins/" + baseName, false );
             }
           }
           else
@@ -4290,6 +4293,8 @@ void QgisApp::loadPlugin( QString name, QString description, QString theFullPath
 
         }
         break;
+/*
+        // TODO: to be removed completely
         case QgisPlugin::MAPLAYER:
         {
           // Map layer - requires interaction with the canvas
@@ -4320,6 +4325,7 @@ void QgisApp::loadPlugin( QString name, QString description, QString theFullPath
           }
         }
         break;
+*/
         default:
           // type is unknown
           QgsDebugMsg( "Plugin " + theFullPathName + " did not return a valid type and cannot be loaded" );

src/app/qgisapp.h

@@ -385,7 +385,7 @@ class QgisApp : public QMainWindow
     //! load python support if possible
     void loadPythonSupport();
     //! plugin loader
-    void loadPlugin( QString name, QString description, QString mFullPath );
+    void loadPlugin( QString mFullPath, QString name );
     //! python plugin loader
     void loadPythonPlugin( QString packageName, QString pluginName );
     //! Find the QMenu with the given name (ie the user visible text on the menu item)

src/app/qgspluginitem.cpp

@@ -17,8 +17,8 @@
 /* $Id$ */
 #include "qgspluginitem.h"
 
-QgsPluginItem::QgsPluginItem( QString _name, QString _description, QString _fullPath, QString _type, bool _python ):
-    m_name( _name ), m_description( _description ), m_fullPath( _fullPath ), m_type( _type ), m_python( _python )
+QgsPluginItem::QgsPluginItem( QString _name, QString _fullPath, QString _type, bool _python ):
+    m_name( _name ), m_fullPath( _fullPath ), m_type( _type ), m_python( _python )
 {
 
 }

src/app/qgspluginitem.h

@@ -26,7 +26,7 @@ Class to contain information about a loadable plugin, including its name, descri
 class QgsPluginItem
 {
   public:
-    QgsPluginItem( QString name = 0, QString description = 0, QString fullPath = 0, QString type = 0, bool python = false );
+    QgsPluginItem( QString name = 0, QString fullPath = 0, QString type = 0, bool python = false );
     QString name();
     QString description();
     QString fullPath();

src/app/qgspluginmanager.cpp

@@ -55,7 +55,7 @@
 
 const int PLUGIN_DATA_ROLE = Qt::UserRole;
 const int PLUGIN_LIBRARY_ROLE = Qt::UserRole + 1;
-const int PLUGIN_LIBRARY_NAME_ROLE = Qt::UserRole + 2;
+const int PLUGIN_BASE_NAME_ROLE = Qt::UserRole + 2;
 
 QgsPluginManager::QgsPluginManager( QgsPythonUtils* pythonUtils, QWidget * parent, Qt::WFlags fl )
     : QDialog( parent, fl )
@@ -151,9 +151,9 @@ void QgsPluginManager::getPythonPluginDescriptions()
     // filtering will be done on the display role so give it name and desription
     // user wont see this text since we are using a custome delegate
     QStandardItem * mypDetailItem = new QStandardItem( pluginName + " - " + description );
-    QString myLibraryName = "python:" + packageName;;
+    QString myLibraryName = "python:" + packageName;
     mypDetailItem->setData( myLibraryName, PLUGIN_LIBRARY_ROLE ); //for loading libs later
-    mypDetailItem->setData( pluginName, PLUGIN_LIBRARY_NAME_ROLE ); //for matching in registry later
+    mypDetailItem->setData( packageName, PLUGIN_BASE_NAME_ROLE ); //for matching in registry later
     mypDetailItem->setCheckable( false );
     mypDetailItem->setEditable( false );
     // setData in the delegate with a variantised QgsDetailedItemData
@@ -167,19 +167,15 @@ void QgsPluginManager::getPythonPluginDescriptions()
 
     // check to see if the plugin is loaded and set the checkbox accordingly
     QgsPluginRegistry *pRegistry = QgsPluginRegistry::instance();
-    QString libName = pRegistry->library( pluginName );
-    if ( libName.length() == 0 || !pRegistry->isPythonPlugin( pluginName ) )
+    if ( pRegistry->isLoaded(packageName) && pRegistry->isPythonPlugin(packageName) )
     {
-      QgsDebugMsg( "Couldn't find library name in the registry" );
+      QgsDebugMsg( "Found plugin in the registry" );
+      // set the checkbox
+      myData.setChecked( true );
     }
     else
     {
-      QgsDebugMsg( "Found library name in the registry" );
-      if ( libName == packageName )
-      {
-        // set the checkbox
-        myData.setChecked( true );
-      }
+      QgsDebugMsg( "Couldn't find plugin in the registry: " + packageName );
     }
     QVariant myVariant = qVariantFromValue( myData );
     mypDetailItem->setData( myVariant, PLUGIN_DATA_ROLE );
@@ -297,40 +293,40 @@ void QgsPluginManager::getPluginDescriptions()
       delete myLib;
       continue;
     }
+    
+    QString pluginName = pName();
+    QString pluginDesc = pDesc();
+    QString pluginVersion = pVersion();
+    QString baseName = QFileInfo(lib).baseName();
 
     QString myLibraryName = pluginDir[i];
     // filtering will be done on the display role so give it name and desription
     // user wont see this text since we are using a custome delegate
-    QStandardItem * mypDetailItem = new QStandardItem( pName() + " - " + pDesc() );
+    QStandardItem * mypDetailItem = new QStandardItem( pluginName + " - " + pluginDesc );
     mypDetailItem->setData( myLibraryName, PLUGIN_LIBRARY_ROLE );
-    mypDetailItem->setData( pName(), PLUGIN_LIBRARY_NAME_ROLE ); //for matching in registry later
+    mypDetailItem->setData( baseName, PLUGIN_BASE_NAME_ROLE ); //for matching in registry later
     QgsDetailedItemData myData;
-    myData.setTitle( pName() );
-    myData.setDetail( pDesc() );
+    myData.setTitle( pluginName );
+    myData.setDetail( pluginDesc );
     myData.setRenderAsWidget( false );
     myData.setCheckable( true );
     myData.setChecked( false ); //start unchecked - we will check it later if needed
 
-    //round trip test - delete this...no need to uncomment
-    //QgsDetailedItemData myData2 = qVariantValue<QgsDetailedItemData>(myVariant);
-    //Q_ASSERT(myData.title() == myData2.title());
-    //round trip test ends
-
     QgsDebugMsg( "Getting an instance of the QgsPluginRegistry" );
 
     // check to see if the plugin is loaded and set the checkbox accordingly
     QgsPluginRegistry *pRegistry = QgsPluginRegistry::instance();
-    QString libName = pRegistry->library( pName() );
 
     // get the library using the plugin description
-    if ( libName.length() == 0 )
+    if ( !pRegistry->isLoaded(baseName) )
     {
-      QgsDebugMsg( "Couldn't find library name in the registry" );
+      QgsDebugMsg( "Couldn't find plugin in the registry" );
     }
     else
     {
-      QgsDebugMsg( "Found library name in the registry" );
-      if ( libName == myLib->fileName() )
+      QgsDebugMsg( "Found plugin in the registry" );
+      // TODO: this check shouldn't be necessary, plugin base names must be unique
+      if ( pRegistry->library(baseName) == myLib->fileName() )
       {
         // set the checkbox
         myData.setChecked( true );
@@ -367,32 +363,36 @@ void QgsPluginManager::unload()
       myIndex = mModelPlugins->index( row, 0 );
       // its off -- see if it is loaded and if so, unload it
       QgsPluginRegistry *pRegistry = QgsPluginRegistry::instance();
-      QgsDebugMsg( QString( "Checking to see if %1 is loaded" ).arg( mModelPlugins->data( myIndex, PLUGIN_LIBRARY_NAME_ROLE ).toString() ) );
-      QString pluginName = mModelPlugins->data( myIndex, PLUGIN_LIBRARY_NAME_ROLE ).toString();
-      if ( pRegistry->isPythonPlugin( pluginName ) )
+      
+      // is loaded?
+      QString baseName = mModelPlugins->data( myIndex, PLUGIN_BASE_NAME_ROLE ).toString();
+      if ( ! pRegistry->isLoaded( baseName ) )
+        continue;
+      
+      if ( pRegistry->isPythonPlugin( baseName ) )
       {
         if ( mPythonUtils && mPythonUtils->isEnabled() )
         {
-          QString packageName = pRegistry->library( pluginName );
-          mPythonUtils->unloadPlugin( packageName );
+          mPythonUtils->unloadPlugin( baseName );
           //disable it to the qsettings file
-          settings.setValue( "/PythonPlugins/" + packageName, false );
+          settings.setValue( "/PythonPlugins/" + baseName, false );
         }
       }
       else // C++ plugin
       {
-        QgisPlugin *plugin = pRegistry->plugin( pluginName );
+        QgisPlugin *plugin = pRegistry->plugin( baseName );
         if ( plugin )
         {
           plugin->unload();
         }
         //disable it to the qsettings file [ts]
-        settings.setValue( "/Plugins/" + pluginName, false );
+        settings.setValue( "/Plugins/" + baseName, false );
       }
       // remove the plugin from the registry
-      pRegistry->removePlugin( pluginName );
+      pRegistry->removePlugin( baseName );
     }
   }
+  QgsPluginRegistry::instance()->dump();
 }
 
 std::vector < QgsPluginItem > QgsPluginManager::getSelectedPlugins()
@@ -406,7 +406,8 @@ std::vector < QgsPluginItem > QgsPluginManager::getSelectedPlugins()
 
     if ( myData.isChecked() )
     {
-      QString pluginName = mModelPlugins->item( row, 0 )->data( PLUGIN_LIBRARY_NAME_ROLE ).toString();
+      //QString baseName = mModelPlugins->item( row, 0 )->data( PLUGIN_BASE_NAME_ROLE ).toString();
+      QString pluginName = mModelPlugins->item( row, 0 )->data( Qt::DisplayRole ).toString();
       bool pythonic = false;
 
       QString library = mModelPlugins->item( row, 0 )->data( PLUGIN_LIBRARY_ROLE ).toString();
@@ -425,9 +426,7 @@ std::vector < QgsPluginItem > QgsPluginManager::getSelectedPlugins()
       //              QString fullPath=0,
       //              QString type=0,
       //              bool python=false);
-      pis.push_back( QgsPluginItem( pluginName,
-                                    mModelPlugins->item( row, 0 )->data( Qt::DisplayRole ).toString(), //display role
-                                    library, 0, pythonic ) );
+      pis.push_back( QgsPluginItem( pluginName, library, 0, pythonic ) );
     }
 
   }
@@ -472,9 +471,10 @@ void QgsPluginManager::on_vwPlugins_clicked( const QModelIndex &theIndex )
     // the index row in the underlying model so we need to jump through this
     // little hoop to get the correct item
     //
-    QStandardItem * mypItem =
-      mModelPlugins->findItems( theIndex.data( Qt ::DisplayRole ).toString() ).first();
-    QgsDetailedItemData myData =
+    
+    QModelIndex realIndex = mModelProxy->mapToSource(theIndex);
+    QStandardItem* mypItem = mModelPlugins->itemFromIndex(realIndex);
+    QgsDetailedItemData myData = 
       qVariantValue<QgsDetailedItemData>( mypItem->data( PLUGIN_DATA_ROLE ) );
     if ( myData.isChecked() )
     {