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:
|
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.
This
ifstatement has no good purpose.LocalTime-Expansion/src/main/java/net/aboodyy/localtime/LocalTimeExpansion.java
Line 76 in 6626942
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
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.