-
Notifications
You must be signed in to change notification settings - Fork 58
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
Holograms not working on 1.16.4 #359
Comments
Update Managed to debug and reproduce the issue on my end. As it turns out, some economy plugins do not provide a meaningful implementation of That, in turn, causes the following code to throw on a null value: switch (placeholder) {
case BUY_PRICE:
replace = plugin.getEconomy().format(getBuyPrice()); // sets replace to null
break;
case SELL_PRICE:
replace = plugin.getEconomy().format(getSellPrice()); // also sets replace to null
break;
default:
replace = String.valueOf(placeholders.get(placeholder));
}
format = format.replace(placeholder.toString(), replace); // consumes the null and throws Despite the issue being caused by incomplete implementations of |
This change fixes interfacing with some economy-dependant plugins, for instance ShopChest. Related issue: EpicEricEE/ShopChest/issues/359
The issue
Created shops display no holograms whatsoever. Still able to buy/sell; player balance changes properly.
PaperMC version: paper-1.16.5-428
Plugin version: Build #99
Hologram format config:
Relevant logs:
on using a shop
The text was updated successfully, but these errors were encountered: