-
-
Notifications
You must be signed in to change notification settings - Fork 103
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Placeholders are missing for when you dont have an island #863
Comments
Bit of extra info, we call this method here: @Override
public List<Placeholder> getPlaceholders(Optional<Island> optional) {
return optional.isPresent() ? getPlaceholders(optional.get()) : defaultPlaceholders;
} Which then calls two different methods, one with an Island, and the other for if there is no island present. Which means we have two lists to maintain, and these two lists have differences in them now which is the issue. So if you have a PAPI plugin (e.g. a scoreboard) a placeholder e.g. |
(Also, make sure there are no duplicated Placeholders in this list, e.g. island_visitors, also extra bonus brownie points for how we can prevent this, again maybe unit tests?) |
i want to add the command "/is go" to my server, if it type the command it should go to the island, if the island doesn't exist then the island should be created Can you help me |
|
The Placeholders between when you have an island, and dont have an island is different
https://github.com/Iridium-Development/IridiumSkyblock/blob/master/src/main/java/com/iridium/iridiumskyblock/placeholders/IslandPlaceholderBuilder.java
Extra bonus points if we can think about how to stop this in the future, add some unit tests? recode it somehow?
The text was updated successfully, but these errors were encountered: