Log tax messages to the shop logger too · ChestShop-authors/ChestShop-3@2388da6

Qodana Scan: src/main/java/com/Acrobot/ChestShop/Listeners/Economy/Plugins/ReserveListener.java#L37

Method invocation `name` may produce `NullPointerException`

Qodana Scan: src/main/java/com/Acrobot/ChestShop/Containers/AdminInventory.java#L229

`null` is returned by the method declared as @NotNull

Qodana Scan: src/main/java/com/Acrobot/ChestShop/Dependencies.java#L113

Method invocation `getDescription` may produce `NullPointerException`

Qodana Scan: src/main/java/com/Acrobot/ChestShop/UUIDs/NameManager.java#L171

Deprecated member 'getAccountFromShortName' is still used

Qodana Scan: src/main/java/com/Acrobot/ChestShop/UUIDs/NameManager.java#L42

Field `accounts` is accessed in both synchronized and unsynchronized contexts

Qodana Scan: src/main/java/com/Acrobot/ChestShop/UUIDs/NameManager.java#L44

Field `usernameToAccount` is accessed in both synchronized and unsynchronized contexts

Qodana Scan: src/main/java/com/Acrobot/ChestShop/UUIDs/NameManager.java#L46

Field `shortToAccount` is accessed in both synchronized and unsynchronized contexts

Qodana Scan: src/main/java/com/Acrobot/ChestShop/ChestShop.java#L121

Non-constant logger field `shopLogger`

Qodana Scan: src/main/java/com/Acrobot/ChestShop/Updater/Updater.java#L57

Variable `id` initializer `-1` is redundant

Qodana Scan: src/main/java/com/Acrobot/Breeze/Utils/Encoding/Base64.java#L1329

Variable `encodedData` initializer `null` is redundant

Provide feedback