SpectateLogin implementation

src/main/java/fr/xephi/authme/AuthMe.java

@@ -24,6 +24,7 @@
 import fr.xephi.authme.service.BackupService;
 import fr.xephi.authme.service.BukkitService;
 import fr.xephi.authme.service.MigrationService;
+import fr.xephi.authme.service.SpectateLoginService;
 import fr.xephi.authme.service.bungeecord.BungeeReceiver;
 import fr.xephi.authme.service.yaml.YamlParseException;
 import fr.xephi.authme.settings.Settings;
@@ -69,6 +70,7 @@ public class AuthMe extends JavaPlugin {
     private Injector injector;
     private BackupService backupService;
     private ConsoleLogger logger;
+    private SpectateLoginService spectateLoginService;
 
     /**
      * Constructor.
@@ -246,6 +248,7 @@ void instantiateServices(Injector injector) {
         bukkitService = injector.getSingleton(BukkitService.class);
         commandHandler = injector.getSingleton(CommandHandler.class);
         backupService = injector.getSingleton(BackupService.class);
+        spectateLoginService = injector.getSingleton(SpectateLoginService.class);
 
         // Trigger instantiation (class not used elsewhere)
         injector.getSingleton(BungeeReceiver.class);
@@ -316,6 +319,8 @@ public void onDisable() {
         // Wait for tasks and close data source
         new TaskCloser(this, database).run();
 
+        injector.createIfHasDependencies(SpectateLoginService.class).removeArmorstands();
+
         // Disabled correctly
         Consumer<String> infoLogMethod = logger == null ? getLogger()::info : logger::info;
         infoLogMethod.accept("AuthMe " + this.getDescription().getVersion() + " disabled!");

src/main/java/fr/xephi/authme/data/limbo/LimboPlayer.java

@@ -1,6 +1,7 @@
 package fr.xephi.authme.data.limbo;
 
 import fr.xephi.authme.task.MessageTask;
+import org.bukkit.GameMode;
 import org.bukkit.Location;
 import org.bukkit.scheduler.BukkitTask;
 
@@ -22,18 +23,20 @@ public class LimboPlayer {
     private final Location loc;
     private final float walkSpeed;
     private final float flySpeed;
+    private final GameMode gameMode;
     private BukkitTask timeoutTask = null;
     private MessageTask messageTask = null;
     private LimboPlayerState state = LimboPlayerState.PASSWORD_REQUIRED;
 
     public LimboPlayer(Location loc, boolean operator, Collection<UserGroup> groups, boolean fly, float walkSpeed,
-                       float flySpeed) {
+                       float flySpeed, GameMode gameMode) {
         this.loc = loc;
         this.operator = operator;
         this.groups = new ArrayList<>(groups); // prevent bug #2413
         this.canFly = fly;
         this.walkSpeed = walkSpeed;
         this.flySpeed = flySpeed;
+        this.gameMode = gameMode;
     }
 
     /**
@@ -45,6 +48,15 @@ public Location getLocation() {
         return loc;
     }
 
+    /**
+     * Return the player's original gamemode.
+     *
+     * @return The player's gamemode
+     */
+    public GameMode getGameMode() {
+        return gameMode;
+    }
+
     /**
      * Return whether the player is an operator or not (i.e. whether he is an OP).
      *

src/main/java/fr/xephi/authme/data/limbo/LimboService.java

@@ -119,6 +119,7 @@ public void restoreData(Player player) {
             logger.debug("No LimboPlayer found for `{0}` - cannot restore", lowerName);
         } else {
             player.setOp(limbo.isOperator());
+            player.setGameMode(limbo.getGameMode());
             settings.getProperty(RESTORE_ALLOW_FLIGHT).restoreAllowFlight(player, limbo);
             settings.getProperty(RESTORE_FLY_SPEED).restoreFlySpeed(player, limbo);
             settings.getProperty(RESTORE_WALK_SPEED).restoreWalkSpeed(player, limbo);

src/main/java/fr/xephi/authme/data/limbo/LimboServiceHelper.java

@@ -6,6 +6,7 @@
 import fr.xephi.authme.settings.Settings;
 import fr.xephi.authme.settings.properties.LimboSettings;
 import fr.xephi.authme.settings.properties.RestrictionSettings;
+import org.bukkit.GameMode;
 import org.bukkit.Location;
 import org.bukkit.entity.Player;
 
@@ -44,6 +45,7 @@ LimboPlayer createLimboPlayer(Player player, boolean isRegistered, Location loca
         boolean flyEnabled = player.getAllowFlight();
         float walkSpeed = player.getWalkSpeed();
         float flySpeed = player.getFlySpeed();
+        GameMode gameMode = player.getGameMode();
         Collection<UserGroup> playerGroups = permissionsManager.hasGroupSupport()
             ? permissionsManager.getGroups(player) : Collections.emptyList();
 
@@ -52,7 +54,7 @@ LimboPlayer createLimboPlayer(Player player, boolean isRegistered, Location loca
             .collect(toList());
 
         logger.debug("Player `{0}` has groups `{1}`", player.getName(), String.join(", ", groupNames));
-        return new LimboPlayer(location, isOperator, playerGroups, flyEnabled, walkSpeed, flySpeed);
+        return new LimboPlayer(location, isOperator, playerGroups, flyEnabled, walkSpeed, flySpeed, gameMode);
     }
 
     /**
@@ -97,10 +99,11 @@ LimboPlayer merge(LimboPlayer newLimbo, LimboPlayer oldLimbo) {
         boolean canFly = newLimbo.isCanFly() || oldLimbo.isCanFly();
         float flySpeed = Math.max(newLimbo.getFlySpeed(), oldLimbo.getFlySpeed());
         float walkSpeed = Math.max(newLimbo.getWalkSpeed(), oldLimbo.getWalkSpeed());
+        GameMode gameMode = oldLimbo.getGameMode();
         Collection<UserGroup> groups = getLimboGroups(oldLimbo.getGroups(), newLimbo.getGroups());
         Location location = firstNotNull(oldLimbo.getLocation(), newLimbo.getLocation());
 
-        return new LimboPlayer(location, isOperator, groups, canFly, walkSpeed, flySpeed);
+        return new LimboPlayer(location, isOperator, groups, canFly, walkSpeed, flySpeed, gameMode);
     }
 
     private static Location firstNotNull(Location first, Location second) {

src/main/java/fr/xephi/authme/listener/PlayerListener.java

@@ -11,6 +11,7 @@
 import fr.xephi.authme.service.AntiBotService;
 import fr.xephi.authme.service.BukkitService;
 import fr.xephi.authme.service.JoinMessageService;
+import fr.xephi.authme.service.SpectateLoginService;
 import fr.xephi.authme.service.TeleportationService;
 import fr.xephi.authme.service.ValidationService;
 import fr.xephi.authme.settings.Settings;
@@ -19,6 +20,7 @@
 import fr.xephi.authme.settings.properties.RegistrationSettings;
 import fr.xephi.authme.settings.properties.RestrictionSettings;
 import org.bukkit.ChatColor;
+import org.bukkit.GameMode;
 import org.bukkit.Location;
 import org.bukkit.entity.HumanEntity;
 import org.bukkit.entity.Player;
@@ -49,6 +51,8 @@
 import org.bukkit.event.player.PlayerQuitEvent;
 import org.bukkit.event.player.PlayerRespawnEvent;
 import org.bukkit.event.player.PlayerShearEntityEvent;
+import org.bukkit.event.player.PlayerTeleportEvent;
+import org.bukkit.event.player.PlayerToggleSneakEvent;
 import org.bukkit.inventory.InventoryView;
 
 import javax.inject.Inject;
@@ -91,6 +95,8 @@ public class PlayerListener implements Listener {
     private PermissionsManager permissionsManager;
     @Inject
     private QuickCommandsProtectionManager quickCommandsProtectionManager;
+    @Inject
+    private SpectateLoginService spectateLoginService;
 
     // Lowest priority to apply fast protection checks
     @EventHandler(priority = EventPriority.LOWEST)
@@ -376,6 +382,31 @@ public void onPlayerRespawn(PlayerRespawnEvent event) {
         if (spawn != null && spawn.getWorld() != null) {
             event.setRespawnLocation(spawn);
         }
+
+        if (settings.getProperty(RestrictionSettings.SPECTATE_STAND_LOGIN)) {
+            bukkitService.runTaskLater(() -> spectateLoginService.createStand(event.getPlayer()), 1L);
+        }
+    }
+
+    @EventHandler(ignoreCancelled = true, priority = EventPriority.LOWEST)
+    public void onToggleSneak(PlayerToggleSneakEvent event) {
+        if (listenerService.shouldCancelEvent(event.getPlayer())
+            && (settings.getProperty(RestrictionSettings.SPECTATE_STAND_LOGIN)
+            || spectateLoginService.hasStand(event.getPlayer()))) {
+            event.setCancelled(true);
+        }
+    }
+
+    @EventHandler(ignoreCancelled = true, priority = EventPriority.HIGHEST)
+    public void onTeleport(PlayerTeleportEvent event) {
+        if (listenerService.shouldCancelEvent(event.getPlayer())
+            && event.getCause() == PlayerTeleportEvent.TeleportCause.SPECTATE
+            && event.getPlayer().getGameMode() == GameMode.SPECTATOR
+            && (settings.getProperty(RestrictionSettings.SPECTATE_STAND_LOGIN)
+            || spectateLoginService.hasStand(event.getPlayer()))) {
+            spectateLoginService.updateTarget(event.getPlayer());
+            event.setCancelled(true);
+        }
     }
 
     /*

src/main/java/fr/xephi/authme/process/join/AsynchronousJoin.java

@@ -14,6 +14,7 @@
 import fr.xephi.authme.service.CommonService;
 import fr.xephi.authme.service.PluginHookService;
 import fr.xephi.authme.service.SessionService;
+import fr.xephi.authme.service.SpectateLoginService;
 import fr.xephi.authme.service.ValidationService;
 import fr.xephi.authme.service.bungeecord.BungeeSender;
 import fr.xephi.authme.service.bungeecord.MessageType;
@@ -83,6 +84,9 @@ public class AsynchronousJoin implements AsynchronousProcess {
     @Inject
     private ProxySessionManager proxySessionManager;
 
+    @Inject
+    private SpectateLoginService spectateLoginService;
+
     AsynchronousJoin() {
     }
 
@@ -199,6 +203,13 @@ private void processJoinSync(Player player, boolean isAuthAvailable) {
                 int blindTimeOut = (registrationTimeout <= 0) ? 99999 : registrationTimeout;
                 player.addPotionEffect(new PotionEffect(PotionEffectType.BLINDNESS, blindTimeOut, 2));
             }
+
+            if (service.getProperty(RestrictionSettings.SPECTATE_STAND_LOGIN)) {
+                // The delay is necessary in order to make sure that the player is teleported to spawn
+                // and after authorization appears in the same place
+                bukkitService.runTaskLater(() -> spectateLoginService.createStand(player), 1);
+            }
+
             commandManager.runCommandsOnJoin(player);
         });
     }

src/main/java/fr/xephi/authme/process/login/ProcessSyncPlayerLogin.java

@@ -12,11 +12,13 @@
 import fr.xephi.authme.service.BukkitService;
 import fr.xephi.authme.service.CommonService;
 import fr.xephi.authme.service.JoinMessageService;
+import fr.xephi.authme.service.SpectateLoginService;
 import fr.xephi.authme.service.TeleportationService;
 import fr.xephi.authme.service.bungeecord.BungeeSender;
 import fr.xephi.authme.settings.WelcomeMessageConfiguration;
 import fr.xephi.authme.settings.commandconfig.CommandManager;
 import fr.xephi.authme.settings.properties.RegistrationSettings;
+import fr.xephi.authme.settings.properties.RestrictionSettings;
 import org.bukkit.entity.Player;
 import org.bukkit.potion.PotionEffectType;
 
@@ -58,6 +60,9 @@ public class ProcessSyncPlayerLogin implements SynchronousProcess {
     @Inject
     private PermissionsManager permissionsManager;
 
+    @Inject
+    private SpectateLoginService spectateLoginService;
+
     ProcessSyncPlayerLogin() {
     }
 
@@ -99,6 +104,11 @@ public void processPlayerLogin(Player player, boolean isFirstLogin, List<String>
             player.removePotionEffect(PotionEffectType.BLINDNESS);
         }
 
+        if (commonService.getProperty(RestrictionSettings.SPECTATE_STAND_LOGIN)
+            || spectateLoginService.hasStand(player)) {
+            spectateLoginService.removeStand(player);
+        }
+
         // The Login event now fires (as intended) after everything is processed
         bukkitService.callEvent(new LoginEvent(player));
 

src/main/java/fr/xephi/authme/process/logout/ProcessSyncPlayerLogout.java

@@ -9,6 +9,7 @@
 import fr.xephi.authme.process.SynchronousProcess;
 import fr.xephi.authme.service.BukkitService;
 import fr.xephi.authme.service.CommonService;
+import fr.xephi.authme.service.SpectateLoginService;
 import fr.xephi.authme.service.TeleportationService;
 import fr.xephi.authme.settings.commandconfig.CommandManager;
 import fr.xephi.authme.settings.properties.RegistrationSettings;
@@ -44,6 +45,9 @@ public class ProcessSyncPlayerLogout implements SynchronousProcess {
     @Inject
     private CommandManager commandManager;
 
+    @Inject
+    private SpectateLoginService spectateLoginService;
+
     ProcessSyncPlayerLogout() {
     }
 
@@ -65,6 +69,10 @@ public void processSyncLogout(Player player) {
 
         service.send(player, MessageKey.LOGOUT_SUCCESS);
         logger.info(player.getName() + " logged out");
+
+        if (service.getProperty(RestrictionSettings.SPECTATE_STAND_LOGIN)) {
+            spectateLoginService.createStand(player);
+        }
     }
 
     private void applyLogoutEffect(Player player) {

src/main/java/fr/xephi/authme/process/quit/ProcessSyncPlayerQuit.java

@@ -2,14 +2,23 @@
 
 import fr.xephi.authme.data.limbo.LimboService;
 import fr.xephi.authme.process.SynchronousProcess;
+import fr.xephi.authme.service.CommonService;
+import fr.xephi.authme.service.SpectateLoginService;
 import fr.xephi.authme.settings.commandconfig.CommandManager;
+import fr.xephi.authme.settings.properties.RestrictionSettings;
 import org.bukkit.entity.Player;
 
 import javax.inject.Inject;
 
 
 public class ProcessSyncPlayerQuit implements SynchronousProcess {
 
+    @Inject
+    private CommonService service;
+
+    @Inject
+    private SpectateLoginService spectateLoginService;
+
     @Inject
     private LimboService limboService;
 
@@ -26,6 +35,11 @@ public void processSyncQuit(Player player, boolean wasLoggedIn) {
         if (wasLoggedIn) {
             commandManager.runCommandsOnLogout(player);
         } else {
+            if (service.getProperty(RestrictionSettings.SPECTATE_STAND_LOGIN)
+                || spectateLoginService.hasStand(player)) {
+                spectateLoginService.removeStand(player);
+            }
+
             limboService.restoreData(player);
             player.saveData(); // #1238: Speed is sometimes not restored properly
         }