Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
- 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@9526 c8812cc2-4d05-0410-92ff-de0c093fc19c
  • Loading branch information
wonder committed Oct 23, 2008
1 parent 85122d1 commit 4ff166a
Show file tree
Hide file tree
Showing 7 changed files with 146 additions and 111 deletions.
34 changes: 20 additions & 14 deletions src/app/qgisapp.cpp
Expand Up @@ -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"
Expand Down Expand Up @@ -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 )
Expand All @@ -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
{
Expand Down Expand Up @@ -4142,7 +4143,7 @@ void QgisApp::showPluginManager()
}
else
{
loadPlugin( plugin.name(), plugin.description(), plugin.fullPath() );
loadPlugin( plugin.fullPath(), plugin.name() );
}
it++;
}
Expand Down Expand Up @@ -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;
Expand All @@ -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");
Expand Down Expand Up @@ -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
{
Expand All @@ -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
Expand All @@ -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
Expand Down Expand Up @@ -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" );
Expand Down
2 changes: 1 addition & 1 deletion src/app/qgisapp.h
Expand Up @@ -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)
Expand Down
4 changes: 2 additions & 2 deletions src/app/qgspluginitem.cpp
Expand Up @@ -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 )
{

}
Expand Down
2 changes: 1 addition & 1 deletion src/app/qgspluginitem.h
Expand Up @@ -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();
Expand Down
84 changes: 42 additions & 42 deletions src/app/qgspluginmanager.cpp
Expand Up @@ -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 )
Expand Down Expand Up @@ -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
Expand All @@ -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 );
Expand Down Expand Up @@ -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 );
Expand Down Expand Up @@ -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()
Expand All @@ -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();
Expand All @@ -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 ) );
}

}
Expand Down Expand Up @@ -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() )
{
Expand Down

0 comments on commit 4ff166a

Please sign in to comment.