Fix #579

Also tweaks to deprecation for development purposes.

Bukkit/src/main/java/com/intellectualcrafters/plot/api/PlotAPI.java

@@ -228,13 +228,14 @@ public class PlotAPI {
      * Get the plot manager for a world. Most of these methods can be accessed
      * through the MainUtil.
      *
-     * @param world Which manager to get
+     * @param world the world to retrieve the manager from
      *
      * @return PlotManager
      *
      * @see PlotManager
      * @see PS#getPlotManager(Plot)
      */
+    @SuppressWarnings("deprecation")
     @Deprecated
     public PlotManager getPlotManager(World world) {
         if (world == null) {
@@ -292,6 +293,7 @@ public class PlotAPI {
      * @see #getPlotAreas(World)
      * @see PlotArea
      */
+    @SuppressWarnings("deprecation")
     @Deprecated
     public PlotArea getWorldSettings(World world) {
         if (world == null) {
@@ -395,6 +397,7 @@ public class PlotAPI {
      *
      * @see PlotArea#getPlot(PlotId)
      */
+    @SuppressWarnings("deprecation")
     @Deprecated
     public Plot getPlot(World world, int x, int z) {
         if (world == null) {
@@ -446,6 +449,7 @@ public class PlotAPI {
      *
      * @see #getPlots(World, Player, boolean)
      */
+    @SuppressWarnings("deprecation")
     @Deprecated
     public boolean hasPlot(World world, Player player) {
         return getPlots(world, player, true).length > 0;
@@ -500,6 +504,7 @@ public class PlotAPI {
      * @return World[] - array of plot worlds
      *
      */
+    @SuppressWarnings("deprecation")
     @Deprecated
     public String[] getPlotWorlds() {
         Set<String> plotWorldStrings = PS.get().getPlotWorldStrings();
@@ -531,6 +536,7 @@ public class PlotAPI {
      *
      * @see Plot
      */
+    @SuppressWarnings("deprecation")
     @Deprecated
     public Location[] getLocations(Plot plot) {
         Location bukkitBottom = BukkitUtil.getLocation(plot.getCorners()[0]);
@@ -563,9 +569,10 @@ public class PlotAPI {
      *
      * @see Plot
      */
+    @SuppressWarnings("deprecation")
     @Deprecated
     public Location getBottomLocation(Plot plot) {
-        return BukkitUtil.getLocation(plot.getBottom());
+        return BukkitUtil.getLocation(plot.getCorners()[0]);
     }
 
     /**
@@ -579,9 +586,10 @@ public class PlotAPI {
      *
      * @see Plot
      */
+    @SuppressWarnings("deprecation")
     @Deprecated
     public Location getTopLocation(Plot plot) {
-        return BukkitUtil.getLocation(plot.getTop());
+        return BukkitUtil.getLocation(plot.getCorners()[1]);
     }
 
     /**

Bukkit/src/main/java/com/plotsquared/bukkit/BukkitMain.java

@@ -79,6 +79,7 @@ import org.bukkit.OfflinePlayer;
 import org.bukkit.World;
 import org.bukkit.command.PluginCommand;
 import org.bukkit.entity.Entity;
+import org.bukkit.entity.LivingEntity;
 import org.bukkit.entity.Player;
 import org.bukkit.event.Listener;
 import org.bukkit.generator.ChunkGenerator;
@@ -313,12 +314,23 @@ public final class BukkitMain extends JavaPlugin implements Listener, IPlotMain
                                         }
                                         Location location = entity.getLocation();
                                         if (BukkitUtil.getLocation(location).isPlotRoad()) {
+                                            if (entity instanceof LivingEntity) {
+                                                LivingEntity livingEntity = (LivingEntity) entity;
+                                                if (!livingEntity.isLeashed() || !entity.hasMetadata("keep")) {
                                                     Entity passenger = entity.getPassenger();
                                                     if (!(passenger instanceof Player) && entity.getMetadata("keep").isEmpty()) {
                                                         iterator.remove();
                                                         entity.remove();
                                                     }
                                                 }
+                                            } else {
+                                                Entity passenger = entity.getPassenger();
+                                                if (!(passenger instanceof Player) && entity.getMetadata("keep").isEmpty()) {
+                                                    iterator.remove();
+                                                    entity.remove();
+                                                }
+                                            }
+                                        }
                                 }
                             }
                         } catch (Throwable e) {

Bukkit/src/main/java/com/plotsquared/bukkit/listeners/PlayerEvents_1_8.java

@@ -32,6 +32,7 @@ import java.util.UUID;
 
 public class PlayerEvents_1_8 extends PlotListener implements Listener {
     
+    @SuppressWarnings("deprecation")
     @EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true)
     public void onInventoryClick(InventoryClickEvent event) {
         if (!event.isLeftClick() || (event.getAction() != InventoryAction.PLACE_ALL) || event.isShiftClick()) {

Bukkit/src/main/java/com/plotsquared/bukkit/object/BukkitPlayer.java

@@ -9,6 +9,7 @@ import com.intellectualcrafters.plot.util.UUIDHandler;
 import com.plotsquared.bukkit.util.BukkitUtil;
 import org.bukkit.Effect;
 import org.bukkit.GameMode;
+import org.bukkit.Material;
 import org.bukkit.WeatherType;
 import org.bukkit.entity.Player;
 import org.bukkit.event.player.PlayerTeleportEvent.TeleportCause;
@@ -180,7 +181,7 @@ public class BukkitPlayer extends PlotPlayer {
     
     @Override
     public void playMusic(Location location, int id) {
-        this.player.playEffect(BukkitUtil.getLocation(location), Effect.RECORD_PLAY, id);
+        this.player.playEffect(BukkitUtil.getLocation(location), Effect.RECORD_PLAY, Material.getMaterial(id));
     }
     
     @Override