Browse Source

Use the right method when checking bundled plugins

Certain PluginManager methods will read plugin info from disk,
while others just use cached info. There's no obvious way to
tell which is which, and we ended up calling the wrong one.

Now we actually know about plugins that're installed, we can
extract the bundled ones. Yay.

Conflicts:
        src/com/dmdirc/ClientModule.java

Change-Id: I995f0a5bec8ef06db887127119d4f0f9209bf4bf
Fixes-Issue: CLIENT-498
Reviewed-on: http://gerrit.dmdirc.com/3374
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager
Reviewed-on: http://gerrit.dmdirc.com/3375
tags/0.8.1rc1^0
Chris Smith 10 years ago
parent
commit
aa9d0c8458
1 changed files with 6 additions and 7 deletions
  1. 6
    7
      src/com/dmdirc/ClientModule.java

+ 6
- 7
src/com/dmdirc/ClientModule.java View File

@@ -48,9 +48,9 @@ import com.dmdirc.logger.ErrorLevel;
48 48
 import com.dmdirc.logger.ErrorManager;
49 49
 import com.dmdirc.logger.Logger;
50 50
 import com.dmdirc.messages.MessageSinkManager;
51
-import com.dmdirc.plugins.PluginInfo;
52 51
 import com.dmdirc.plugins.PluginInjectorInitialiser;
53 52
 import com.dmdirc.plugins.PluginManager;
53
+import com.dmdirc.plugins.PluginMetaData;
54 54
 import com.dmdirc.plugins.ServiceManager;
55 55
 import com.dmdirc.ui.IconManager;
56 56
 import com.dmdirc.ui.WarningDialog;
@@ -558,15 +558,14 @@ public class ClientModule {
558 558
             final CorePluginExtractor corePluginExtractor,
559 559
             final PluginManager pm,
560 560
             final AggregateConfigProvider config) {
561
-        for (PluginInfo plugin : pm.getPluginInfos()) {
562
-            if (config.hasOptionString("bundledplugins_versions", plugin.getMetaData().getName())) {
561
+        for (PluginMetaData plugin : pm.getAllPlugins()) {
562
+            if (config.hasOptionString("bundledplugins_versions", plugin.getName())) {
563 563
                 final Version bundled = new Version(config.getOption("bundledplugins_versions",
564
-                        plugin.getMetaData().getName()));
565
-                final Version installed = plugin.getMetaData().getVersion();
564
+                        plugin.getName()));
565
+                final Version installed = plugin.getVersion();
566 566
 
567 567
                 if (installed.compareTo(bundled) < 0) {
568
-                    corePluginExtractor.extractCorePlugins(plugin.getMetaData().getName());
569
-                    pm.reloadPlugin(plugin.getFilename());
568
+                    corePluginExtractor.extractCorePlugins(plugin.getName());
570 569
                 }
571 570
             }
572 571
         }

Loading…
Cancel
Save