Meishin/LoginSystem
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix command mapping for /authme:unregister etc.

Browse Source
This commit is contained in:
ljacqu 2017-02-18 15:56:08 +01:00
parent e3426cd731
commit c9b66183de
2 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/main/java/fr/xephi/authme/command/CommandMapper.java
View File

@ -123,6 +123,9 @@ public class CommandMapper {
private CommandDescription getBaseCommand(String label) { private CommandDescription getBaseCommand(String label) {
String baseLabel = label.toLowerCase(); String baseLabel = label.toLowerCase();
if (baseLabel.startsWith("authme:")) {
baseLabel = baseLabel.substring("authme:".length());
}
for (CommandDescription command : baseCommands) { for (CommandDescription command : baseCommands) {
if (command.hasLabel(baseLabel)) { if (command.hasLabel(baseLabel)) {
return command; return command;

15
src/test/java/fr/xephi/authme/command/CommandMapperTest.java
View File

@ -290,6 +290,21 @@ public class CommandMapperTest {
assertThat(result.getArguments(), contains(parts.get(2))); assertThat(result.getArguments(), contains(parts.get(2)));
} }
@Test
public void shouldSupportAuthMePrefix() {
// given
List<String> parts = asList("authme:unregister", "Betty");
CommandSender sender = mock(CommandSender.class);
given(permissionsManager.hasPermission(eq(sender), any(PermissionNode.class))).willReturn(true);
// when
FoundCommandResult result = mapper.mapPartsToCommand(sender, parts);
// then
assertThat(result.getResultStatus(), equalTo(FoundResultStatus.SUCCESS));
assertThat(result.getCommandDescription(), equalTo(getCommandWithLabel(commands, "unregister")));
}
@SuppressWarnings("unchecked") @SuppressWarnings("unchecked")
@Test @Test
public void shouldReturnExecutableCommandClasses() { public void shouldReturnExecutableCommandClasses() {