Преглед на файлове

Tidy PluginInfo setting.

pull/282/head
Chris Smith преди 9 години
родител
ревизия
b5cc39089e

+ 3
- 6
nickcolours/src/com/dmdirc/addons/nickcolours/NickColourManager.java Целия файл

@@ -78,8 +78,7 @@ public class NickColourManager {
78 78
     /** Event bus to subscribe to events on . */
79 79
     private final DMDircMBassador eventBus;
80 80
     /** The plugin's info. */
81
-    // TODO: Should probably make this final and introduce a factory to create NickColourManagers.
82
-    private PluginInfo pluginInfo;
81
+    private final PluginInfo pluginInfo;
83 82
     /** "Random" colours to use to colour nicknames. */
84 83
     private String[] randColours = {
85 84
         "E90E7F", "8E55E9", "B30E0E", "18B33C", "58ADB3", "9E54B3", "B39875", "3176B3",};
@@ -94,12 +93,14 @@ public class NickColourManager {
94 93
             @GlobalConfig final AggregateConfigProvider globalConfig,
95 94
             final IconManager iconManager,
96 95
             @MainWindow final Provider<Window> mainWindowProvider,
96
+            final PluginInfo pluginInfo,
97 97
             final DMDircMBassador eventBus) {
98 98
         this.domain = domain;
99 99
         this.globalConfig = globalConfig;
100 100
         this.colourManager = colourManager;
101 101
         this.iconManager = iconManager;
102 102
         this.mainWindowProvider = mainWindowProvider;
103
+        this.pluginInfo = pluginInfo;
103 104
         this.eventBus = eventBus;
104 105
         configBinder = globalConfig.getBinder().withDefaultDomain(domain);
105 106
     }
@@ -277,10 +278,6 @@ public class NickColourManager {
277 278
         randColours = value.toArray(new String[value.size()]);
278 279
     }
279 280
 
280
-    public void setPluginInfo(final PluginInfo pluginInfo) {
281
-        this.pluginInfo = pluginInfo;
282
-    }
283
-
284 281
     @Handler
285 282
     public void handlePrefsOpened(final ClientPrefsOpenedEvent event) {
286 283
         final PreferencesCategory general = new PluginPreferencesCategory(

+ 10
- 1
nickcolours/src/com/dmdirc/addons/nickcolours/NickColourModule.java Целия файл

@@ -24,6 +24,7 @@ package com.dmdirc.addons.nickcolours;
24 24
 
25 25
 import com.dmdirc.addons.ui_swing.injection.SwingModule;
26 26
 import com.dmdirc.plugins.PluginDomain;
27
+import com.dmdirc.plugins.PluginInfo;
27 28
 
28 29
 import dagger.Module;
29 30
 import dagger.Provides;
@@ -34,10 +35,13 @@ import dagger.Provides;
34 35
 @Module(injects = NickColourManager.class, addsTo = SwingModule.class)
35 36
 public class NickColourModule {
36 37
 
38
+    /** This plugin's plugin info. */
39
+    private final PluginInfo pluginInfo;
37 40
     /** The domain for plugin settings. */
38 41
     private final String domain;
39 42
 
40
-    public NickColourModule(final String domain) {
43
+    public NickColourModule(final PluginInfo pluginInfo, final String domain) {
44
+        this.pluginInfo = pluginInfo;
41 45
         this.domain = domain;
42 46
     }
43 47
 
@@ -52,4 +56,9 @@ public class NickColourModule {
52 56
         return domain;
53 57
     }
54 58
 
59
+    @Provides
60
+    public PluginInfo getPluginInfo() {
61
+        return pluginInfo;
62
+    }
63
+
55 64
 }

+ 1
- 2
nickcolours/src/com/dmdirc/addons/nickcolours/NickColourPlugin.java Целия файл

@@ -39,9 +39,8 @@ public class NickColourPlugin extends BasePlugin {
39 39
     public void load(final PluginInfo pluginInfo, final ObjectGraph graph) {
40 40
         super.load(pluginInfo, graph);
41 41
 
42
-        setObjectGraph(graph.plus(new NickColourModule(pluginInfo.getDomain())));
42
+        setObjectGraph(graph.plus(new NickColourModule(pluginInfo, pluginInfo.getDomain())));
43 43
         nickColourManager = getObjectGraph().get(NickColourManager.class);
44
-        nickColourManager.setPluginInfo(pluginInfo);
45 44
     }
46 45
 
47 46
     @Override

Loading…
Отказ
Запис