Parcourir la source

Remove some uses of parser LocalUser.

pull/241/head
Greg Holmes il y a 9 ans
Parent
révision
7af13d9fb1

+ 1
- 1
dcc/src/com/dmdirc/addons/dcc/DCCCommand.java Voir le fichier

@@ -116,7 +116,7 @@ public class DCCCommand extends Command implements IntelligentCommand {
116 116
             final String target = args.getArguments()[1];
117 117
             final Connection connection = ((ServerCommandContext) context).getConnection();
118 118
             final Parser parser = connection.getParser().get();
119
-            final String myNickname = parser.getLocalClient().getNickname();
119
+            final String myNickname = connection.getLocalUser().getNickname();
120 120
 
121 121
             if (parser.isValidChannelName(target)
122 122
                     || parser.getStringConverter().equalsIgnoreCase(target,

+ 2
- 2
identd/src/com/dmdirc/addons/identd/IdentClient.java Voir le fichier

@@ -184,9 +184,9 @@ public class IdentClient implements Runnable {
184 184
                 != null && !customName.isEmpty() && customName.length() < 513) {
185 185
             username = customName;
186 186
         } else if (connection != null && config.getOptionBool(domain, "general.useNickname")) {
187
-            username = connection.getParser().get().getLocalClient().getNickname();
187
+            username = connection.getLocalUser().getNickname();
188 188
         } else if (connection != null && config.getOptionBool(domain, "general.useUsername")) {
189
-            username = connection.getParser().get().getLocalClient().getUsername();
189
+            username = connection.getLocalUser().getUsername().orElse(System.getProperty("user.name"));
190 190
         } else {
191 191
             username = System.getProperty("user.name");
192 192
         }

+ 5
- 0
identd/test/com/dmdirc/addons/identd/IdentClientTest.java Voir le fichier

@@ -25,6 +25,7 @@ package com.dmdirc.addons.identd;
25 25
 import com.dmdirc.DMDircMBassador;
26 26
 import com.dmdirc.interfaces.Connection;
27 27
 import com.dmdirc.interfaces.ConnectionManager;
28
+import com.dmdirc.interfaces.User;
28 29
 import com.dmdirc.interfaces.config.AggregateConfigProvider;
29 30
 import com.dmdirc.parser.irc.IRCClientInfo;
30 31
 import com.dmdirc.parser.irc.IRCParser;
@@ -50,6 +51,7 @@ public class IdentClientTest {
50 51
     @Mock private Connection connection;
51 52
     @Mock private IRCParser parser;
52 53
     @Mock private IRCClientInfo client;
54
+    @Mock private User user;
53 55
     @Mock private AggregateConfigProvider config;
54 56
     @Mock private DMDircMBassador eventBus;
55 57
 
@@ -59,10 +61,13 @@ public class IdentClientTest {
59 61
 
60 62
         when(sm.getConnections()).thenReturn(servers);
61 63
         when(connection.getParser()).thenReturn(Optional.of(parser));
64
+        when(connection.getLocalUser()).thenReturn(user);
62 65
         when(parser.getLocalPort()).thenReturn(60);
63 66
         when(parser.getLocalClient()).thenReturn(client);
64 67
         when(client.getNickname()).thenReturn("nickname");
65 68
         when(client.getUsername()).thenReturn("username");
69
+        when(user.getNickname()).thenReturn("nickname");
70
+        when(user.getUsername()).thenReturn(Optional.of("username"));
66 71
 
67 72
         return new IdentClient(eventBus, null, null, sm, config, "plugin-Identd");
68 73
     }

Chargement…
Annuler
Enregistrer