Repo-Mirror/eco
9
0
Fork 0
Code Issues Releases 2 Activity

Finally removed Display#callDisplayModule

Browse Source
This commit is contained in:
Auxilor
2022-04-27 13:04:24 +01:00
parent 84f439976c
commit 0fa30a5f62
2 changed files with 13 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
eco-api/src/main/java/com/willfp/eco/core/display/Display.java
View File

@@ -126,39 +126,6 @@ public final class Display {
Display.handler = handler;
}
/**
* Extremely janky method - also internal, so don't use it. <b>This method is
* NOT part of the API and may be removed at any time!</b>
* <p>
* This calls a display module with the specified parameters, now
* you might ask why I need a static java method when the DisplayHandler
* implementation could just call it itself? Well, kotlin doesn't really
* like dealing with vararg ambiguity, and so while kotlin can't figure out
* what is and isn't a vararg when I call display with a player, java can.
* <p>
* Because of this, I need to have this part of the code in java.
*
* <b>Don't call this method as part of your plugins!</b>
* <p>
* No, seriously - don't. This skips a bunch of checks and you'll almost
* definitely break something.
*
* @param module The display module.
* @param itemStack The ItemStack.
* @param player The player.
* @param args The args.
*/
@ApiStatus.Internal
public static void callDisplayModule(@NotNull final DisplayModule module,
@NotNull final ItemStack itemStack,
@Nullable final Player player,
@NotNull final Object... args) {
module.display(itemStack, args);
if (player != null) {
module.display(itemStack, player, args);
}
}
/**
* Set the display handler.
* <p>