Fixed: #1193 Plugin system checks dependency before loading plugins.
This commit is contained in:
parent
5f8789e98f
commit
6d572275a1
1 changed files with 5 additions and 5 deletions
|
@ -214,9 +214,6 @@ void PluginManager::setPluginState(PluginSpec *spec, int destState)
|
|||
|
||||
switch (destState)
|
||||
{
|
||||
case State::Loaded:
|
||||
spec->loadLibrary();
|
||||
return;
|
||||
case State::Resolved:
|
||||
spec->resolveDependencies(m_pluginSpecs);
|
||||
return;
|
||||
|
@ -234,13 +231,16 @@ void PluginManager::setPluginState(PluginSpec *spec, int destState)
|
|||
if (depSpec->state() != destState)
|
||||
{
|
||||
spec->m_hasError = true;
|
||||
spec->m_errorString = tr("Cannot initializing plugin because dependency failed to load: %1\nReason: %2")
|
||||
.arg(depSpec->name()).arg(depSpec->errorString());
|
||||
spec->m_errorString = tr("Cannot load plugin because dependency failed to load: %1")
|
||||
.arg(depSpec->name());
|
||||
return;
|
||||
}
|
||||
}
|
||||
switch (destState)
|
||||
{
|
||||
case State::Loaded:
|
||||
spec->loadLibrary();
|
||||
return;
|
||||
case State::Initialized:
|
||||
spec->initializePlugin();
|
||||
break;
|
||||
|
|
Loading…
Reference in a new issue