-
Notifications
You must be signed in to change notification settings - Fork 7
Open
Labels
enhancementNew feature or requestNew feature or request
Description
This if statement has no good purpose.
| if (p == null) return null; |
Check if the player instance is null lower when it would actually be required, in this condition:
LocalTime-Expansion/src/main/java/net/aboodyy/localtime/LocalTimeExpansion.java
Lines 102 to 103 in 6626942
| if (identifier.equalsIgnoreCase("time")) | |
| return dateManager.getDate(format, dateManager.getTimeZone(p)); |
Needlessly requiring the player object before it's actually needed prevents placeholders from this plugin from working at all when called without a player object.
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request