pom.xml

@@ -4,7 +4,7 @@
 	<groupId>us.camin.regions</groupId>
 	<artifactId>Regions</artifactId>
 	<packaging>jar</packaging>
-	<version>0.4.0</version>
+	<version>0.3.1</version>
 	<name>regions</name>
 	<url>http://maven.apache.org</url>
 	<properties>
@@ -46,12 +46,6 @@
       <version>2.0</version>
       <scope>provided</scope>
 		</dependency>
-		<dependency>
-				<groupId>com.github.angeschossen</groupId>
-				<artifactId>LandsAPI</artifactId>
-				<version>6.15.0</version>
-				<scope>provided</scope>
-		</dependency>
     <dependency>
         <groupId>commons-codec</groupId>
         <artifactId>commons-codec</artifactId>
@@ -177,10 +171,7 @@
 			<id>papermc</id>
 			<url>http://papermc.io/repo/repository/maven-public/</url>
 		</repository>
-    <repository>
+    <repository><id>pl3x-repo</id><url>http://repo.pl3x.net/</url></repository>
-        <id>jitpack.io</id>
-        <url>https://jitpack.io</url>
-    </repository>
 		<repository><id>dynmap-repo</id><url>http://repo.mikeprimm.com/</url></repository>
 		<repository><id>imagej</id><url>http://maven.imagej.net/content/repositories/public/</url></repository>
 	</repositories>

src/main/java/us/camin/regions/Plugin.java

@@ -111,11 +111,7 @@ public class Plugin extends JavaPlugin {
         this.getDataFolder().mkdir();
         File regionConfigFile = new File(this.getDataFolder(), "regions.yml");
         Configuration regionConf = YamlConfiguration.loadConfiguration(regionConfigFile);
-        try {
+        m_regions.loadRegions(regionConf);
-          m_regions.loadRegions(regionConf);
-        } catch (Exception e) {
-          log.log(Level.SEVERE, "Could not load regions config! You risk overwriting and losing data!", e);
-        }
     }
 
     public void saveRegions() {

src/main/java/us/camin/regions/RegionManager.java

@@ -37,72 +37,13 @@ import us.camin.regions.events.RegionRemoveEvent;
 import us.camin.regions.geometry.RegionSet;
 import java.util.logging.Level;
 
-import me.angeschossen.lands.api.land.Land;
-import me.angeschossen.lands.api.land.Area;
-import me.angeschossen.lands.api.flags.Flags;
-import me.angeschossen.lands.api.integration.LandsIntegration;
-
 public class RegionManager {
 
-    private final LandsIntegration m_lands;
-
-    public enum ValidationResult {
-        VALID,
-        UNKNOWN,
-        NO_PERMISSION,
-        TOO_CLOSE,
-        BAD_NAME,
-    }
-
-    public String validateRegionName(String name, Location location) {
-        Land thisLand = m_lands.getLand(location);
-        if (thisLand == null) {
-          return name;
-        } else {
-          return thisLand.getName();
-        }
-    }
-
-    public ValidationResult validateAnchorPoint(Player player, Location location, String proposedName) {
-      // Require basic permission
-      if (!player.hasPermission("regions.create")) {
-          return ValidationResult.NO_PERMISSION;
-      }
-
-      // Allow admins to create regions in unclaimed land
-      Area thisArea = m_lands.getAreaByLoc(location);
-      if (thisArea == null) {
-        if (proposedName == null) {
-          return ValidationResult.BAD_NAME;
-        } else if (player.hasPermission("regions.create.bypass")) {
-          return ValidationResult.VALID;
-        } else {
-          return ValidationResult.NO_PERMISSION;
-        }
-      }
-
-      // For claimed land, require claiming perms to place post
-      if (!thisArea.hasFlag(player, Flags.LAND_CLAIM, false)) {
-        return ValidationResult.NO_PERMISSION;
-      }
-
-      // Ensure this is the only region post for this region
-      for (Region region : regionsForWorld(location.getWorld())) {
-        Land thatLand = m_lands.getLand(region.interactLocation());
-        if (thatLand != null && thatLand.getId() == thisArea.getLand().getId()) {
-          return ValidationResult.TOO_CLOSE;
-        }
-      }
-
-      return ValidationResult.VALID;
-    }
-
     Logger log = Logger.getLogger("Regions.RegionManager");
     private Map<String, RegionSet> m_regions;
     private Server m_server;
     public RegionManager(Plugin plugin, Server server) {
         m_server = server;
-        m_lands = new LandsIntegration(plugin);
         log.setLevel(Level.ALL);
         clear();
     }

src/main/java/us/camin/regions/RegionPostItemWatcher.java

@@ -60,11 +60,13 @@ public class RegionPostItemWatcher implements Listener {
       ShapedRecipe anchorRecipe = new ShapedRecipe(anchorKey, m_theAnchor);
       anchorRecipe.shape("DDD", "DGD", "DDD");
       anchorRecipe.setIngredient('D', new RecipeChoice.ExactChoice(m_theChargeItem));
-      anchorRecipe.setIngredient('G', Material.SOUL_LANTERN);
+      anchorRecipe.setIngredient('G', Material.LANTERN);
 
       NamespacedKey compassKey = new NamespacedKey(m_plugin, "region_post_compass");
       ShapedRecipe compassRecipe = new ShapedRecipe(compassKey, m_theCompass);
       // Uses four fewer charges, slightly cheaper.
+      // TODO: Maybe we just want this to be glowstone instead of effectively 4
+      // ghast tears?
       compassRecipe.shape(" D ", " G ", "   ");
       compassRecipe.setIngredient('D', new RecipeChoice.ExactChoice(m_theChargeItem));
       compassRecipe.setIngredient('G', Material.COMPASS);
@@ -94,7 +96,6 @@ public class RegionPostItemWatcher implements Listener {
       meta.setLore(lore);
       meta.addEnchant(Enchantment.SOUL_SPEED, 1, true);
       meta.addItemFlags(ItemFlag.HIDE_ENCHANTS);
-      meta.setCustomModelData(93199);
       stack.setItemMeta(meta);
       return stack;
     }
@@ -108,7 +109,6 @@ public class RegionPostItemWatcher implements Listener {
       meta.addItemFlags(ItemFlag.HIDE_ENCHANTS);
       meta.addEnchant(Enchantment.SOUL_SPEED, 1, true);
       meta.setDisplayName("Region Post Charge");
-      meta.setCustomModelData(93197);
       stack.setItemMeta(meta);
       return stack;
     }
@@ -122,7 +122,6 @@ public class RegionPostItemWatcher implements Listener {
       meta.addEnchant(Enchantment.SOUL_SPEED, 1, true);
       meta.setLore(lore);
       meta.setDisplayName("Region Post Anchor");
-      meta.setCustomModelData(93198);
       stack.setItemMeta(meta);
       return stack;
     }
@@ -186,32 +185,28 @@ public class RegionPostItemWatcher implements Listener {
           ItemStack compassItem = createCompass(nearest);
           player.setItemInHand(compassItem);
           player.sendMessage("Now tracking " + nearest.name());
-        } else if (!event.isCancelled() && isRegionCreateItem(handStack, player) && event.getAction() == Action.RIGHT_CLICK_BLOCK && !event.getClickedBlock().getType().isInteractable()) {
+        } else if (!event.isCancelled() && isRegionCreateItem(handStack, player) && event.getAction() == Action.RIGHT_CLICK_BLOCK && !event.getClickedBlock().getType().isInteractable() && player.hasPermission("regions.create")) {
           event.setUseItemInHand(Event.Result.DENY);
           event.setCancelled(true);
-          String postName = meta.getDisplayName();
+          if (meta.getDisplayName().equals("") || meta.getDisplayName().equals("Region Post Anchor")) {
-          if (postName.equals("") || postName.equals("Region Post Anchor")) {
+            player.sendMessage("You must first give this item a name!");
-            postName = null;
-          }
-          postName = m_manager.validateRegionName(postName, event.getClickedBlock().getLocation());
-          RegionManager.ValidationResult result = m_manager.validateAnchorPoint(player, event.getClickedBlock().getLocation(), postName);
-          if (result == RegionManager.ValidationResult.VALID) {
-            Region r = new Region(postName, event.getClickedBlock().getRelative(event.getBlockFace()).getLocation());
-            m_plugin.getServer().getScheduler().runTask(m_plugin, () -> {
-              RegionPostBuilder builder = new RegionPostBuilder(r, m_plugin);
-              builder.build();
-            });
-            handStack.setAmount(handStack.getAmount()-1);
-            player.setItemInHand(handStack);
-            m_plugin.regionManager().addRegion(r);
-            m_plugin.saveRegions();
-            player.sendMessage("You established the region "+r.coloredName());
-          } else if (result == RegionManager.ValidationResult.TOO_CLOSE) {
-            player.sendMessage("You are too close to another region post!");
-          } else if (result == RegionManager.ValidationResult.BAD_NAME) {
-            player.sendMessage("You must first give this item a better name.");
           } else {
-            player.sendMessage("You aren't allowed to create a region here.");
+            Region nearest = m_manager.nearestRegion(event.getClickedBlock().getLocation());
+            if (nearest != null && event.getClickedBlock().getLocation().distance(nearest.interactLocation()) < 500) {
+              int distance = 500 - (int)event.getClickedBlock().getLocation().distance(nearest.interactLocation());
+              player.sendMessage("You are " + distance + " blocks too close to the region post for " + nearest.name() + ".");
+            } else {
+              Region r = new Region(meta.getDisplayName(), event.getClickedBlock().getRelative(event.getBlockFace()).getLocation());
+              m_plugin.getServer().getScheduler().runTask(m_plugin, () -> {
+                RegionPostBuilder builder = new RegionPostBuilder(r, m_plugin);
+                builder.build();
+              });
+              handStack.setAmount(handStack.getAmount()-1);
+              player.setItemInHand(handStack);
+              m_plugin.regionManager().addRegion(r);
+              m_plugin.saveRegions();
+              player.sendMessage("You established the region "+r.coloredName());
+            }
           }
         }
     }

src/main/java/us/camin/regions/geometry/BorderMesh.java

@@ -131,7 +131,7 @@ public class BorderMesh {
         DelaunayTriangulator mesh = new DelaunayTriangulator(regionPoints);
         try {
           mesh.triangulate();
-          log.fine("Mesh triangulated!");
+          log.info("Mesh triangulated!");
         } catch (NullPointerException e) {
           log.warning("Got a null pointer when triangulating, skipping world.");
           return false;
@@ -173,7 +173,7 @@ public class BorderMesh {
         for(Region region : m_regions) {
           ArrayList<Vector2D> points = new ArrayList<Vector2D>();
           HashSet<Region> allNeighbors = new HashSet<Region>();
-          log.fine("Executing voronoi transform...");
+          log.info("Executing voronoi transform...");
           for(Triangle tri : triangleSoup) {
               if (tri.region == region) {
                   for (Region neighbor : m_regions) {

src/main/java/us/camin/regions/ui/PlayerInventoryTeleporter.java

@@ -92,19 +92,13 @@ public class PlayerInventoryTeleporter implements Listener {
             if (event.getClickedInventory() == neighborInv) {
                 ItemMeta meta = event.getCurrentItem().getItemMeta();
                 Material mat = event.getCurrentItem().getType();
-                final String selectedName = meta.getDisplayName();
+                if (mat == Material.BEDROCK || mat == Material.COMPASS || mat == Material.LANTERN) {
-                if (mat == Material.BEDROCK || mat == Material.COMPASS || mat == Material.LANTERN || mat == Material.SOUL_LANTERN) {
                     return;
                 }
-
+                m_plugin.getServer().getScheduler().runTask(m_plugin, () -> event.getView().close());
                 Region nearest = m_manager.nearestRegion(player.getLocation());
 
-                if (selectedName == nearest.name()) {
+                final String selectedName = meta.getDisplayName();
-                  return;
-                }
-
-                m_plugin.getServer().getScheduler().runTask(m_plugin, () -> event.getView().close());
-
                 final Optional<Region> destination = m_plugin.regionManager().regionsForWorld(player.getLocation().getWorld())
                   .stream()
                   .filter((r) -> r.name().equals(selectedName))
@@ -219,7 +213,7 @@ public class PlayerInventoryTeleporter implements Listener {
         }
         ItemStack chargesItem = new ItemStack(event.region.charges() == 0 ? Material.SOUL_LANTERN : Material.LANTERN);
         ItemMeta meta = chargesItem.getItemMeta();
-        meta.setDisplayName(ChatColor.BOLD + "Region Post Charges");
+        meta.setDisplayName(ChatColor.BOLD + "Region Post Configuration");
         ArrayList<String> lore = new ArrayList<String>();
         lore.add(ChatColor.WHITE + "Name: "+event.region.coloredName());
         lore.add(ChatColor.WHITE + "Visits: " + ChatColor.YELLOW + event.region.visits());
@@ -229,18 +223,7 @@ public class PlayerInventoryTeleporter implements Listener {
         meta.addEnchant(Enchantment.SOUL_SPEED, 1, true);
         chargesItem.setItemMeta(meta);
         // 22 Is the middle of the bottom row
-        neighborInventory.setItem(23, chargesItem);
+        neighborInventory.setItem(22, chargesItem);
-
-        ItemStack iconItem = event.region.icon();
-        meta = iconItem.getItemMeta();
-        meta.setDisplayName(event.region.name());
-        lore = new ArrayList<String>();
-        lore.add(ChatColor.BOLD + "This is the region's current banner icon.");
-        lore.add(ChatColor.ITALIC + "Right-click the region post with a banner to change it");
-        meta.setLore(lore);
-        meta.addItemFlags(ItemFlag.HIDE_ENCHANTS);
-        iconItem.setItemMeta(meta);
-        neighborInventory.setItem(21, iconItem);
 
         event.player.openInventory(neighborInventory);
     }

src/main/resources/plugin.yml

@@ -3,8 +3,8 @@ main: us.camin.regions.Plugin
 author: Torrie Fischer <tdfischer@hackerbots.net>
 website: http://hackerbots.net/
 version: ${version}
-api-version: 1.19
+api-version: 1.16
-softdepend: [dynmap, ProtocolLib, Lands]
+softdepend: [dynmap, ProtocolLib, Pl3xMap]
 commands:
     regions:
         description: "List available regions."
@@ -21,7 +21,6 @@ permissions:
         description: Allows use of all regions permissions
         children:
             regions.create: true
-            regions.create.bypass: true
             regions.commands.*: true
             regions.regen.*: true
             regions.bypass.*: true
@@ -38,9 +37,6 @@ permissions:
     regions.create:
         default: true
         description: Create a region with a region item
-    regions.create.bypass:
-        default: op
-        description: Bypass anything preventing creation of a new region
     regions.setbanner:
         default: true
         description: Allows setting a region post banner