PluginManager: remove loadPlugin()

loadPlugins() is now the preferred option, since it does all the proper checks.
In addition, the server now acknowledges that loading a single plugin may cause multiple plugins to be loaded, so returning only a single Plugin is not representative of what's actually happening.
This commit is contained in:
Dylan K. Taylor 2021-10-20 21:52:19 +01:00
parent 03fcd844eb
commit 76b4b23d98
No known key found for this signature in database
GPG Key ID: 8927471A91CAFD3D

View File

@ -175,22 +175,6 @@ class PluginManager{
return null;
}
/**
* @param PluginLoader[] $loaders
*/
public function loadPlugin(string $path, ?array $loaders = null) : ?Plugin{
foreach($loaders ?? $this->fileAssociations as $loader){
if($loader->canLoadPlugin($path)){
$description = $loader->getPluginDescription($path);
if($description instanceof PluginDescription){
$this->internalLoadPlugin($path, $loader, $description);
}
}
}
return null;
}
private function internalLoadPlugin(string $path, PluginLoader $loader, PluginDescription $description) : ?Plugin{
$language = $this->server->getLanguage();
$this->server->getLogger()->info($this->server->getLanguage()->translate(KnownTranslationFactory::pocketmine_plugin_load($description->getFullName())));