fix npe when new pet target is null

pom.xml

@@ -6,7 +6,7 @@
 
     <groupId>me.youhavetrouble</groupId>
     <artifactId>PreventStabby</artifactId>
-    <version>1.4</version>
+    <version>1.5.1</version>
     <packaging>jar</packaging>
 
     <name>PreventStabby</name>
@@ -80,6 +80,10 @@
             <id>sk89q-repo</id>
             <url>https://maven.enginehub.org/repo/</url>
         </repository>
+        <repository>
+            <id>placeholderapi</id>
+            <url>https://repo.extendedclip.com/content/repositories/placeholderapi/</url>
+        </repository>
     </repositories>
 
     <dependencies>
@@ -117,5 +121,11 @@
             <version>7.0.4-SNAPSHOT</version>
             <scope>provided</scope>
         </dependency>
+        <dependency>
+            <groupId>me.clip</groupId>
+            <artifactId>placeholderapi</artifactId>
+            <version>2.11.1</version>
+            <scope>provided</scope>
+        </dependency>
     </dependencies>
 </project>

src/main/java/me/youhavetrouble/preventstabby/PreventStabby.java

@@ -2,6 +2,7 @@ package me.youhavetrouble.preventstabby;
 
 import me.youhavetrouble.preventstabby.commands.MainCommand;
 import me.youhavetrouble.preventstabby.config.ConfigCache;
+import me.youhavetrouble.preventstabby.hooks.PlacoholderApiHook;
 import me.youhavetrouble.preventstabby.hooks.WorldGuardHook;
 import me.youhavetrouble.preventstabby.players.PlayerManager;
 import me.youhavetrouble.preventstabby.players.SmartCache;
@@ -77,6 +78,10 @@ public final class PreventStabby extends JavaPlugin {
             worldGuardHook = false;
         }
 
+        if (getServer().getPluginManager().getPlugin("PlaceholderAPI") != null) {
+            new PlacoholderApiHook(this).register();
+        }
+
         Metrics metrics = new Metrics(this, 14074);
     }
 

src/main/java/me/youhavetrouble/preventstabby/api/PreventStabbyAPI.java

@@ -65,6 +65,26 @@ public class PreventStabbyAPI {
         return PreventStabby.getPlugin().getPlayerManager().canDamage(attacker.getUniqueId(), victim.getUniqueId(), sendDenyMessage);
     }
 
+    /**
+     * Checks if player can be damaged by another. Providing UUID of entity other than player may result in exceptions
+     * @param attackerUuid Attacker's UUID
+     * @param victimUuid Victim's UUID
+     * @return True if victim can be attacked by attacker, false if not
+     */
+    public static boolean canDamage(UUID attackerUuid, UUID victimUuid) {
+        return  PreventStabby.getPlugin().getPlayerManager().canDamage(attackerUuid, victimUuid, false);
+    }
+
+    /**
+     * Checks if player can be damaged by another.
+     * @param attacker Attacker
+     * @param victim Victim
+     * @return True if victim can be attacked by attacker, false if not
+     */
+    public static boolean canDamage(Player attacker, Player victim) {
+        return PreventStabby.getPlugin().getPlayerManager().canDamage(attacker.getUniqueId(), victim.getUniqueId(), false);
+    }
+
     /**
      * Checks if player has login protection.
      * @param uuid UUID of player to check

src/main/java/me/youhavetrouble/preventstabby/config/ConfigCache.java

@@ -20,7 +20,8 @@ public class ConfigCache {
     @Getter private final String pvp_enabled, pvp_disabled, cannot_attack_victim, cannot_attack_attacker,
             cannot_attack_pets_victim, cannot_attack_pets_attacker, no_permission, no_such_command, pvp_enabled_other,
             pvp_disabled_other, punish_for_combat_logout_message, entering_combat, leaving_combat,
-            cant_do_that_during_combat, cannot_attack_mounts_attacker, cannot_attack_mounts_victim;
+            cant_do_that_during_combat, cannot_attack_mounts_attacker, cannot_attack_mounts_victim,
+            placeholder_combat_time, placeholder_not_in_combat;
     @Getter private final double lava_and_fire_stopper_radius;
     @Getter private final long cache_time, combat_time, login_protection_time, teleport_protection_time;
     @Getter private final Set<String> combatBlockedCommands = new HashSet<>();
@@ -82,6 +83,8 @@ public class ConfigCache {
                 addDefault("messages.entering_combat", "&cEntering combat");
                 addDefault("messages.leaving_combat", "&cLeaving combat");
                 addDefault("messages.cant_do_that_during_combat", "&cYou can't do that while in combat!");
+                addDefault("placeholder.placeholder_combat_time", "Combat time: %time%");
+                addDefault("placeholder.not_in_combat", "Not in combat");
             }
         };
 
@@ -112,7 +115,7 @@ public class ConfigCache {
         if (block_commands_in_combat) {
             this.combatBlockedCommands.addAll(config.getStringList("settings.block_in_combat.block_commands.commands"));
         }
-        this.block_teleports_in_combat = config.getBoolean("settings.block_in_combat.block_teleports", true);
+        this.block_teleports_in_combat = config.getBoolean("settings.block_in_combat.block_teleports", false);
 
         this.cache_time = config.getLong("settings.cache_time", 30L);
 
@@ -135,6 +138,9 @@ public class ConfigCache {
         this.entering_combat = config.getString("messages.entering_combat", "&cEntering combat");
         this.leaving_combat = config.getString("messages.leaving_combat", "&cLeaving combat");
         this.cant_do_that_during_combat = config.getString("messages.cant_do_that_during_combat", "&cYou can't do that while in combat!");
+
+        this.placeholder_combat_time = config.getString("placeholder.placeholder_combat_time", "Combat time: %time%");
+        this.placeholder_not_in_combat = config.getString("placeholder.not_in_combat", "Not in combat");
     }
 
 

src/main/java/me/youhavetrouble/preventstabby/hooks/PlacoholderApiHook.java

@@ -0,0 +1,56 @@
+package me.youhavetrouble.preventstabby.hooks;
+
+import me.clip.placeholderapi.expansion.PlaceholderExpansion;
+import me.youhavetrouble.preventstabby.PreventStabby;
+import me.youhavetrouble.preventstabby.util.PluginMessages;
+import org.bukkit.OfflinePlayer;
+import org.jetbrains.annotations.NotNull;
+
+import java.time.Instant;
+import java.util.UUID;
+
+public class PlacoholderApiHook extends PlaceholderExpansion {
+
+    private final PreventStabby plugin;
+
+    public PlacoholderApiHook(PreventStabby preventStabby) {
+        plugin = preventStabby;
+    }
+
+    @Override
+    public @NotNull String getIdentifier() {
+        return "preventstabby";
+    }
+
+    @Override
+    public @NotNull String getAuthor() {
+        return plugin.getDescription().getAuthors().get(0);
+    }
+
+    @Override
+    public @NotNull String getVersion() {
+        return plugin.getDescription().getVersion();
+    }
+
+    @Override
+    public String onRequest(OfflinePlayer player, String params) {
+        if (params.equalsIgnoreCase("combat_time")) {
+            return getCombatTimePlaceholder(player.getUniqueId());
+        }
+        if (params.equalsIgnoreCase("in_combat")) {
+            return String.valueOf(plugin.getPlayerManager().getPlayer(player.getUniqueId()).isInCombat());
+        }
+        return null;
+    }
+
+    private String getCombatTimePlaceholder(UUID uuid) {
+        long seconds = plugin.getPlayerManager().getPlayer(uuid).getCombattime() - Instant.now().getEpochSecond();
+        if (seconds > 0) {
+            String msg = plugin.getConfigCache().getPlaceholder_combat_time();
+            msg = msg.replaceAll("%time%", String.valueOf(seconds));
+            return PluginMessages.parseMessage(msg);
+        }
+        return PluginMessages.parseMessage(plugin.getConfigCache().getPlaceholder_not_in_combat());
+    }
+
+}

src/main/java/me/youhavetrouble/preventstabby/listeners/pets/PetTargettingPlayerListener.java

@@ -20,9 +20,9 @@ public class PetTargettingPlayerListener implements Listener {
     public void onPetTargetPlayer(org.bukkit.event.entity.EntityTargetEvent event) {
         if (!(event.getEntity() instanceof Tameable)) return;
         Tameable entity = (Tameable) event.getEntity();
-        if (entity.getOwner() == null) return;
+        if (!(entity.getOwner() instanceof Player)) return;
+        if (!(event.getTarget() instanceof Player)) return;
 
-        if (event.getTarget() instanceof Player) {
         UUID damager = entity.getOwner().getUniqueId();
         UUID victim = event.getTarget().getUniqueId();
 
@@ -30,6 +30,6 @@ public class PetTargettingPlayerListener implements Listener {
             CombatTimer.refreshPlayersCombatTime(damager, victim);
         else
             event.setCancelled(true);
-        }
+
     }
 }

src/main/java/me/youhavetrouble/preventstabby/players/PlayerManager.java

@@ -12,14 +12,13 @@ import org.bukkit.entity.Player;
 import org.bukkit.scheduler.BukkitTask;
 
 import java.time.Instant;
-import java.util.HashMap;
-import java.util.Iterator;
 import java.util.UUID;
+import java.util.concurrent.ConcurrentHashMap;
 
 public class PlayerManager {
 
     @Getter
-    HashMap<UUID, PlayerData> playerList = new HashMap<>();
+    ConcurrentHashMap<UUID, PlayerData> playerList = new ConcurrentHashMap<>();
 
     public final BukkitTask combatTrackerTask;
 
@@ -30,9 +29,7 @@ public class PlayerManager {
         }
 
         combatTrackerTask = Bukkit.getScheduler().runTaskTimerAsynchronously(PreventStabby.getPlugin(), () -> {
-            Iterator<PlayerData> iterator = playerList.values().iterator();
+            for (PlayerData playerData : playerList.values()) {
-            while (iterator.hasNext()) {
-                PlayerData playerData = iterator.next();
                 UUID uuid = playerData.getPlayerUuid();
                 if (!CombatTimer.isInCombat(uuid)) {
                     if (playerData.getLastCombatCheck()) {
@@ -119,7 +116,8 @@ public class PlayerManager {
 
         if (!smartCache.getPlayerData(attacker).isPvpEnabled()) {
             Player attackerPlayer = Bukkit.getPlayer(attacker);
-            if (PreventStabby.worldGuardHookEnabled() && WorldGuardHook.isPlayerForcedToPvp(attackerPlayer)) return true;
+            if (PreventStabby.worldGuardHookEnabled() && attackerPlayer != null && WorldGuardHook.isPlayerForcedToPvp(attackerPlayer))
+                return true;
 
             if (sendDenyMessage) {
                 ConfigCache config = PreventStabby.getPlugin().getConfigCache();
@@ -129,7 +127,8 @@ public class PlayerManager {
         }
         if (!smartCache.getPlayerData(victim).isPvpEnabled()) {
             Player victimPlayer = Bukkit.getPlayer(victim);
-            if (PreventStabby.worldGuardHookEnabled() && WorldGuardHook.isPlayerForcedToPvp(victimPlayer)) return true;
+            if (PreventStabby.worldGuardHookEnabled() && victimPlayer != null && WorldGuardHook.isPlayerForcedToPvp(victimPlayer))
+                return true;
 
             if (sendDenyMessage) {
                 ConfigCache config = PreventStabby.getPlugin().getConfigCache();

src/main/java/me/youhavetrouble/preventstabby/util/PluginMessages.java

@@ -1,5 +1,7 @@
 package me.youhavetrouble.preventstabby.util;
 
+import me.clip.placeholderapi.PlaceholderAPI;
+import me.youhavetrouble.preventstabby.PreventStabby;
 import net.md_5.bungee.api.ChatMessageType;
 import net.md_5.bungee.api.chat.BaseComponent;
 import net.md_5.bungee.api.chat.TextComponent;
@@ -9,30 +11,33 @@ import org.bukkit.entity.Player;
 
 import java.util.UUID;
 
-
 public class PluginMessages {
 
     public static String parseMessage(String message) {
-        //TODO PAPI support
         return ChatColor.translateAlternateColorCodes('&', message);
     }
 
     public static void sendMessage(Player player, String message) {
-        String parsedMessage = ChatColor.translateAlternateColorCodes('&', message);
+        if (PreventStabby.getPlugin().getServer().getPluginManager().getPlugin("PlaceholderAPI") != null) {
-        player.sendMessage(parsedMessage);
+            message = PlaceholderAPI.setPlaceholders(player, message);
+        }
+        message = ChatColor.translateAlternateColorCodes('&', message);
+        player.sendMessage(message);
     }
 
     public static void sendActionBar(Player player, String message) {
         // TODO use adventure
+        if (PreventStabby.getPlugin().getServer().getPluginManager().getPlugin("PlaceholderAPI") != null) {
+            message = PlaceholderAPI.setPlaceholders(player, message);
+        }
         BaseComponent[] component = TextComponent.fromLegacyText(parseMessage(message));
         player.spigot().sendMessage(ChatMessageType.ACTION_BAR, component);
     }
 
     public static void sendActionBar(UUID uuid, String message) {
-        try {
         Player player = Bukkit.getPlayer(uuid);
+        if (player == null) return;
         sendActionBar(player, message);
-        } catch (NullPointerException ignored) {}
     }
 
     public static String parsePlayerName(Player player, String message) {
@@ -42,6 +47,9 @@ public class PluginMessages {
 
     public static void broadcastMessage(Player player, String message) {
         message = parsePlayerName(player, message);
+        if (PreventStabby.getPlugin().getServer().getPluginManager().getPlugin("PlaceholderAPI") != null) {
+            message = PlaceholderAPI.setPlaceholders(player, message);
+        }
         message = parseMessage(message);
         BaseComponent[] component = TextComponent.fromLegacyText(parseMessage(message));
         Bukkit.spigot().broadcast(component);

src/main/resources/plugin.yml

@@ -6,6 +6,7 @@ api-version: 1.13
 description: Stop people from getting stabbed!
 soft-depend:
   - WorldGuard
+  - PlaceholderAPI
 commands:
   preventstabby:
     permission: preventstabby.command