fix anchor update npe and update mc version range

Author: Z0rdak

common/src/main/java/de/z0rdak/yawp/commands/RegionCommands.java

@@ -58,6 +58,7 @@
 import static de.z0rdak.yawp.constants.Constants.MAX_BUILD_LIMIT;
 import static de.z0rdak.yawp.constants.Constants.MIN_BUILD_LIMIT;
 import static de.z0rdak.yawp.util.ChatComponentBuilder.shortBlockPos;
+import static de.z0rdak.yawp.util.ChatComponentBuilder.shortBlockPosBracketed;
 import static de.z0rdak.yawp.util.ChatLinkBuilder.*;
 
 
@@ -649,10 +650,14 @@ private static int updateTeleportAnchor(CommandContext<CommandSourceStack> ctx,
             return -1;
         }
         if (!region.getArea().contains(pos)) {
-            sendCmdFeedback(ctx.getSource(), Component.translatableWithFallback("cli.msg.info.region.tp-anchor.fail-msg.not-contained", "Teleport Anchor pos must be inside the region.", name, buildRegionInfoLink(region)));
+            sendCmdFeedback(ctx.getSource(), Component.translatableWithFallback("cli.msg.info.region.tp-anchor.fail-msg.not-contained", "Region %s doesn't contain %s - invalid position for teleport anchor supplied.", buildRegionInfoLink(region), shortBlockPosBracketed(pos), name));
             return -1;
         }
-        if (tpAnchors.getTpAnchor(name).getPos().equals(pos)) {
+        if (tpAnchors.hasAnchorWithPos(pos)) {
+            sendCmdFeedback(ctx.getSource(), Component.translatableWithFallback("cli.msg.info.region.tp-anchor.fail-msg.duplicate-pos", "Region %s already contains a Teleport Anchor with the same pos (%s).", buildRegionInfoLink(region), shortBlockPosBracketed(pos)));
+            return -1;
+        }
+        if (tpAnchors.hasAnchor(name, pos)) {
             // they are the same
             return 0;
         }

common/src/main/java/de/z0rdak/yawp/core/area/RegionAnchors.java

@@ -1,17 +1,11 @@
 package de.z0rdak.yawp.core.area;
 
-import com.mojang.serialization.Codec;
-import com.mojang.serialization.Lifecycle;
-import com.mojang.serialization.MapCodec;
-import com.mojang.serialization.codecs.RecordCodecBuilder;
-import de.z0rdak.yawp.core.group.PlayerContainer;
-import de.z0rdak.yawp.core.region.IMarkableRegion;
 import net.minecraft.core.BlockPos;
-import net.minecraft.core.UUIDUtil;
-import net.minecraft.nbt.CompoundTag;
-import net.minecraft.resources.Identifier;
 
-import java.util.*;
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
 
 import static de.z0rdak.yawp.util.ChatComponentBuilder.tinyBlockPos;
 
@@ -44,6 +38,15 @@ public boolean hasAnchor(String name) {
         return this.tpAnchors.containsKey(name);
     }
 
+    public boolean hasAnchor(String name, BlockPos pos) {
+        return this.tpAnchors.containsKey(name)
+                && this.tpAnchors.get(name).getPos().equals(pos);
+    }
+
+    public boolean hasAnchorWithPos(BlockPos pos) {
+        return this.tpAnchors.values().stream()
+                .anyMatch(tpAnchor -> tpAnchor.getPos().equals(pos));
+    }
 
     public TeleportAnchor addTpAnchor(BlockPos pos, String name) {
         var anchor = new TeleportAnchor(pos, name);

gradle.properties

@@ -18,7 +18,7 @@ description=Yet Another World Protector (YAWP) is a server-side mod to protect y
 resource_pack_desc=Internationalisation keys for Yet Another World Protector
 credits=Thanks to all contributors (see CREDITS.txt)!
 
-minecraft_version_range=1.21.10
+minecraft_version_range=1.21.11
 ## This is the version of minecraft that the 'common' project uses, you can find a list of all versions here
 ## https://projects.neoforged.net/neoforged/neoform
 neo_form_version=1.21.11-20251209.172050