-
Notifications
You must be signed in to change notification settings - Fork 9
Retrieving ResultSets correctly #311
base: RELEASE-2023.03
Are you sure you want to change the base?
Conversation
| { | ||
| tryAddAdmin(ResultSetProvider.fromRow(adminSet)); | ||
| row.set(row.get() + 1); | ||
| } catch (Throwable e) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allman's style
Moved command loading into a runnable that will execute on the first server tick. This way, we can ensure that all the plugins are loaded before we load our commands, so TFD4J and Shop can both have their respective commands loaded in.
| return; | ||
| } | ||
|
|
||
| if (result != null && !result.wasSuccessful()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allman's code style
| plugin.dc.syncRoles(admin, plugin.pl.getData(admin.getName()).getDiscordID()); | ||
| } | ||
| Hierarchy.getHierarchy() | ||
| .promoteUser(Hierarchy.getHierarchy().admin(), player) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allman's code style
| { | ||
| FLog.severe("Failed to promote " | ||
| + player.getName() | ||
| + " to the next rank"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allman's code style
| }, () -> msgNew("<red>Failed to promote <player> to the next rank.", player(player))); | ||
| } | ||
| }); | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allman's code style
| @@ -173,78 +183,86 @@ | |||
|
|
|||
| return true; | |||
| } | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allman's code style
|
|
||
| private Map<String, Command> getFallbackCommands() { | ||
| final Map<String, Command> fallbackCommands = new HashMap<>(); | ||
| for (final Map.Entry<String, Command> entry : Bukkit.getCommandMap().getKnownCommands().entrySet()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allman's code style
| private Map<String, Command> getFallbackCommands() { | ||
| final Map<String, Command> fallbackCommands = new HashMap<>(); | ||
| for (final Map.Entry<String, Command> entry : Bukkit.getCommandMap().getKnownCommands().entrySet()) { | ||
| if (!(entry.getValue() instanceof PluginIdentifiableCommand)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allman's code style
| { | ||
| if (!manager.isLoaded(name) || !trackSet.contains(manager.getTrack(name))) | ||
| { | ||
| return manager.createAndLoadTrack(name).thenApplyAsync(a -> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allman's code style
No description provided.