Reject 0 as register date on Bukkit based servers

Applied a patch to existing data

Affects issues:
- Fixed #4201

Author: AuroraLS3

Plan/bukkit/src/main/java/com/djrapitops/plan/gathering/domain/BukkitPlayerData.java

@@ -75,7 +75,8 @@ public Optional<String> getCurrentGameMode() {
 
     @Override
     public Optional<Long> getRegisterDate() {
-        return Optional.of(player.getFirstPlayed());
+        long firstPlayed = player.getFirstPlayed();
+        return firstPlayed > 0 ? Optional.of(firstPlayed) : Optional.empty();
     }
 
     @Override

Plan/common/src/main/java/com/djrapitops/plan/storage/database/transactions/patches/RegisterDateMinimizationPatch.java

@@ -55,12 +55,13 @@ private Query<Map<UUID, Long>> fetchSmallestServerRegisterDates() {
                 UserInfoTable.USER_ID + ',' +
                 "MIN(" + UserInfoTable.REGISTERED + ") as min_registered" +
                 FROM + UserInfoTable.TABLE_NAME +
+                WHERE + UserInfoTable.REGISTERED + "!=0" +
                 GROUP_BY + UserInfoTable.USER_ID;
 
         String sql = SELECT + UsersTable.USER_UUID + ",u1." + UsersTable.REGISTERED + ",min_registered" +
                 FROM + '(' + selectSmallestRegisterDates + ") u2" +
                 INNER_JOIN + UsersTable.TABLE_NAME + " u1 on u1." + UsersTable.ID + "=u2." + UserInfoTable.USER_ID +
-                WHERE + "u1." + UsersTable.REGISTERED + ">min_registered";
+                WHERE + "u1." + UsersTable.REGISTERED + ">min_registered OR u1." + UsersTable.REGISTERED + "=0";
 
         return new QueryAllStatement<>(sql, 500) {
             @Override
@@ -83,7 +84,7 @@ protected void applyPatch() {
 
         String sql = "UPDATE " + UsersTable.TABLE_NAME + " SET " + UsersTable.REGISTERED + "=?" +
                 WHERE + UsersTable.USER_UUID + "=?" +
-                AND + UsersTable.REGISTERED + ">?";
+                AND + UsersTable.REGISTERED + ">? OR " + UsersTable.REGISTERED + "=0";
 
         execute(new ExecBatchStatement(sql) {
             @Override