Due to the way Optional#orElse works, the part inside orElse was run
even when the UUID could be parsed from the given String, leading to
unnecessary work as well as errors in the player UUID query
(That uses names and doesn't look at uuids like the server query)
Fixed by splitting the two execution branches
Affects issues:
- Fixed#1200 (Manually tested)
Favorite server is determined by playtime and if player has no sessions
the player's favorite server will be null, leading to NPE while parsing
JSON.
Added Optional to the source of the null value
Error was reported on Discord that exporting on join sometimes
caused a BadRequestException
A check was added to prevent exporting non-registered players.
- BindException printed a stacktrace on console.
- Bungee threw EnableException with Webserver disabled
- Now writes a easy to read error about bind error
- No longer mandates webserver to start on Bungee (Export possible).
Affects issues:
- Invalidate / Close#1184
A replacer could replace same String multiple times and produce
../../../../server/ instead of intended ../server/
Changed method call from replace to replaceOnce
A single instance of server/ exists in the file.
Affected issues:
- Possibly fixed#1195
Three different SQL queries were causing errors on a H2 database.
Found causes:
- [42001-199] Extra parentheses around subquery: '... FROM (subquery)'
- [HY004 not found] H2 performs query type checks at compile time and
thus parametized variables inside SELECT need to be implicitly cast
to a type
Fixes:
- Removed extra parentheses around 2 queries
- Added a cast to CHAR to a query
Affects issues:
- Fixed#1183
MySQL that restricts float being queried with getLong caused an
SQLException
All usages of getLong with AVG function replaced with
getDouble cast to long.
Affects issues:
- Fixed#1181
MySQL that restricts float being queried with getInt caused an
SQLException
All usages of getInt with AVG function replaced with
getDouble cast to int.
Affects issues:
- Fixed#1181
Improper sanitation was causing issues when server name contained
a single quote symbol.
.server-name replacement was not in use so the offending line of
JS code could be removed.
Affects issues:
- Fixed#1191
Reported StringIndexOutOfBoundsException was likely caused by
AdvancedBans giving something else than UUID with getUuid() method.
Fixed by rejecting Strings smaller than 32 characters
(UUID with dashes is 36)
Affects issues:
- Fixed#1192
Reported NoSuchMethodError did not seem to make sense.
Compiled the extension with newest api version to get it fixed.
Affects issues:
- Possibly fixed#1196
