Merge branch 'develop' into 447_magic_block_flag

Author: tastybento

pom.xml

@@ -64,7 +64,7 @@
         <!-- Do not change unless you want different name for local builds. -->
         <build.number>-LOCAL</build.number>
         <!-- This allows to change between versions. -->
-        <build.version>1.20.0</build.version>
+        <build.version>1.20.1</build.version>
         <!-- SonarCloud -->
         <sonar.projectKey>BentoBoxWorld_AOneBlock</sonar.projectKey>
         <sonar.organization>bentobox-world</sonar.organization>
@@ -281,7 +281,7 @@
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-compiler-plugin</artifactId>
-                <version>3.8.1</version>
+                <version>3.13.0</version>
                 <configuration>
                     <release>${java.version}</release>
                 </configuration>

src/main/java/world/bentobox/aoneblock/listeners/BlockListener.java

@@ -36,6 +36,7 @@
 import org.bukkit.event.block.BlockFromToEvent;
 import org.bukkit.event.entity.EntityInteractEvent;
 import org.bukkit.event.entity.EntitySpawnEvent;
+import org.bukkit.event.entity.ItemSpawnEvent;
 import org.bukkit.event.player.PlayerBucketFillEvent;
 import org.bukkit.event.player.PlayerInteractEvent;
 import org.bukkit.inventory.EquipmentSlot;
@@ -260,38 +261,61 @@ public void onBlockBreak(final PlayerBucketFillEvent e) {
          }
      }
 
+     
      /**
-      * Drop items at the top of the block.
-      * 
-      * @param event EntitySpawnEvent object.
+      * This handler listens for items spawning.
+      * If an item spawns exactly at an island's center block,
+      * it cancels the spawn and re-drops the item 1 block higher
+      * (at the center of that block) to stack it neatly.
       */
      @EventHandler(priority = EventPriority.NORMAL, ignoreCancelled = true)
-     public void onItemStackSpawn(EntitySpawnEvent event) {
+     public void onItemSpawn(ItemSpawnEvent event) {
+         // --- Guard Clauses: Exit early if conditions aren't met ---
+
+         // 1. Check if the "drop on top" feature is enabled.
          if (!this.addon.getSettings().isDropOnTop()) {
-             // Do nothing as item spawning is not interested in this case.
+             // Feature is disabled, so we don't need to do anything.
              return;
          }
 
-         if (!EntityType.ITEM.equals(event.getEntityType())) {
-             // We are interested only in dropped item entities.
-             return;
-         }
+         // Get the spawn location once.
+         Location spawnLocation = event.getLocation();
 
-         if (!this.addon.inWorld(event.getLocation().getWorld())) {
-             // Not correct world
+         // 2. Check if the spawn is happening in a world managed by the addon.
+         if (!this.addon.inWorld(spawnLocation.getWorld())) {
+             // Not a relevant world, ignore this event.
              return;
          }
 
-         Entity entity = event.getEntity();
-         Location location = event.getLocation();
-
-         Optional<Island> optionalIsland = this.addon.getIslands().getIslandAt(location)
-                 .filter(island -> location.getBlock().getLocation().equals(island.getCenter()));
+         // Find an island at the spawn location.
+         Optional<Island> optionalIsland = this.addon.getIslands().getIslandAt(spawnLocation)
+                 // Chained to the Optional: Filter the island.
+                 // Only keep it if the block the item spawned in
+                 // is *exactly* the island's center.
+                 .filter(island -> {
+                     // .getBlock().getLocation() converts a precise location 
+                     // (e.g., 10.2, 64.5, 12.8) to its block's location (10.0, 64.0, 12.0).
+                     Location blockLocation = spawnLocation.getBlock().getLocation();
+                     return blockLocation.equals(island.getCenter());
+                 });
 
+         // If we found an island AND it passed the filter (spawned at center)...
          if (optionalIsland.isPresent()) {
-             // Teleport entity to the top of magic block.
-             entity.teleport(optionalIsland.get().getCenter().add(0.5, 1, 0.5));
-             entity.setVelocity(new Vector(0, 0, 0));
+             // 1. Cancel the original item spawn.
+             event.setCancelled(true);
+
+             // 2. Get the island and the item stack that was supposed to spawn.
+             Island island = optionalIsland.get();
+             // We use event.getEntity() which is guaranteed to be an Item.
+             ItemStack itemStack = event.getEntity().getItemStack();
+
+             // 3. Calculate the new, clean drop location.
+             // .add(0.5, 1, 0.5) moves it to the center of the block (0.5)
+             // and one block up (1.0) so it sits on top.
+             Location newDropLocation = island.getCenter().add(0.5, 1, 0.5);
+
+             // 4. Drop the item stack at the new location.
+             spawnLocation.getWorld().dropItem(newDropLocation, itemStack);
          }
      }