Switched from Item.passenger mode to PacketGenerator mode.

src/config.yml

@@ -28,4 +28,5 @@ max-chair-width: 3
 sign-check: false
 distance: 2
 sitting-height: 0.7
-permissions: true
+permissions: true
+notify-player: true

src/net/spoothie/chairs/ChairWatcher.java

@@ -0,0 +1,30 @@
+/*
+ * To change this template, choose Tools | Templates
+ * and open the template in the editor.
+ */
+package net.spoothie.chairs;
+
+import java.util.ArrayList;
+import net.minecraft.server.DataWatcher;
+import net.minecraft.server.WatchableObject;
+
+/**
+ *
+ * @author cnaude
+ */
+public class ChairWatcher extends DataWatcher {
+
+    private byte metadata;
+
+    public ChairWatcher(byte i) {
+        this.metadata = i;
+    }
+
+    @Override
+    public ArrayList<WatchableObject> b() {
+        ArrayList<WatchableObject> list = new ArrayList<WatchableObject>();
+        WatchableObject wo = new WatchableObject(0, 0, Byte.valueOf(this.metadata));
+        list.add(wo);
+        return list;
+    }
+}

src/net/spoothie/chairs/Chairs.java

@@ -2,7 +2,9 @@ package net.spoothie.chairs;
 
 import java.io.File;
 import java.util.ArrayList;
+import java.util.HashMap;
 import java.util.List;
+import org.bukkit.Location;
 import org.bukkit.Material;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandSender;
@@ -13,11 +15,13 @@ public class Chairs extends JavaPlugin {
 
     public List<Material> allowedBlocks = new ArrayList<Material>();
     public Material item;
-    public boolean sneaking, autorotate, signcheck, permissions;
+    public boolean sneaking, autorotate, signcheck, permissions, notifyplayer;
     public double sittingheight, distance;
     public int maxchairwidth;
     private File pluginFolder;
     private File configFile;
+    public byte metadata;
+    public HashMap<String, Location> sit = new HashMap<String, Location>();
 
     @Override
     public void onEnable() {
@@ -62,6 +66,7 @@ public class Chairs extends JavaPlugin {
         distance = getConfig().getDouble("distance");
         maxchairwidth = getConfig().getInt("max-chair-width");
         permissions = getConfig().getBoolean("permissions");
+        notifyplayer = getConfig().getBoolean("notify-player");
 
         for (String type : getConfig().getStringList("allowed-blocks")) {
             allowedBlocks.add(Material.getMaterial(type));

src/net/spoothie/chairs/EventListener.java

@@ -2,10 +2,14 @@ package net.spoothie.chairs;
 
 import java.util.ArrayList;
 import java.util.List;
+import net.minecraft.server.Packet40EntityMetadata;
+import org.bukkit.Bukkit;
+import org.bukkit.ChatColor;
 import org.bukkit.Location;
 import org.bukkit.Material;
 import org.bukkit.block.Block;
 import org.bukkit.block.BlockFace;
+import org.bukkit.craftbukkit.entity.CraftPlayer;
 import org.bukkit.entity.Entity;
 import org.bukkit.entity.Item;
 import org.bukkit.entity.Player;
@@ -15,6 +19,7 @@ import org.bukkit.event.Listener;
 import org.bukkit.event.block.Action;
 import org.bukkit.event.block.BlockBreakEvent;
 import org.bukkit.event.player.PlayerInteractEvent;
+import org.bukkit.event.player.PlayerMoveEvent;
 import org.bukkit.event.player.PlayerQuitEvent;
 import org.bukkit.inventory.ItemStack;
 import org.bukkit.material.Stairs;
@@ -28,16 +33,37 @@ public class EventListener implements Listener {
         this.plugin = plugin;
     }
 
+    @EventHandler
+    public void onPlayerMove (PlayerMoveEvent event) {
+        Player player = event.getPlayer();        
+        String pname = player.getName();
+        if (plugin.sit.containsKey(player.getName())) {            
+            if (player.getLocation().distance(plugin.sit.get(pname)) > 1) {                
+                if (plugin.notifyplayer) {
+                    player.sendMessage(ChatColor.GRAY + "You are no longer sitting.");
+                }
+                plugin.sit.remove(player.getName());
+                Packet40EntityMetadata packet = new Packet40EntityMetadata(player.getPlayer().getEntityId(), new ChairWatcher((byte) 0), false);
+                for (Player play : Bukkit.getOnlinePlayers()) {
+                    ((CraftPlayer) play).getHandle().netServerHandler.sendPacket(packet);
+                }
+            } else {
+                Packet40EntityMetadata packet = new Packet40EntityMetadata(player.getPlayer().getEntityId(), new ChairWatcher((byte) 4), false);
+                for (Player play : Bukkit.getOnlinePlayers()) {
+                    ((CraftPlayer) play).getHandle().netServerHandler.sendPacket(packet);
+                }
+            }
+        }
+    }
+    
     @EventHandler
     public void onPlayerInteract(PlayerInteractEvent event) {
         if (event.hasBlock() && event.getAction() == Action.RIGHT_CLICK_BLOCK) {
-            Block block = event.getClickedBlock();
+            Block block = event.getClickedBlock();            
-            //event.getPlayer().sendMessage("Block: " + block.getType());
             if (plugin.allowedBlocks.contains(block.getType())) {
                 Player player = event.getPlayer();
-                Stairs stairs = (Stairs) block.getState().getData();
                 int chairwidth = 1;
-                        
+
                 // Check if block beneath chair is solid.
                 if (block.getRelative(BlockFace.DOWN).getType() == Material.AIR) {
                     return;
@@ -48,7 +74,7 @@ public class EventListener implements Listener {
                 if (block.getRelative(BlockFace.DOWN).getType() == Material.LAVA) {
                     return;
                 }
-                if (!net.minecraft.server.Block.byId[block.getTypeId()].material.isSolid()) {                        
+                if (!net.minecraft.server.Block.byId[block.getTypeId()].material.isSolid()) {
                     return;
                 }
 
@@ -60,9 +86,14 @@ public class EventListener implements Listener {
                 }
 
                 // Check if player is sitting.
-                if (!player.isSneaking() && player.getVehicle() != null) {
+                
-                    player.getVehicle().remove();
+                if (plugin.sit.containsKey(event.getPlayer().getName())) {
-                    return;
+                    plugin.sit.remove(player.getName());
+                    event.setCancelled(true);
+                    if (plugin.notifyplayer) {
+                        player.sendMessage(ChatColor.GRAY + "You are no longer sitting.");
+                    }
+                    return;                    
                 }
 
                 // Check for distance distance between player and chair.
@@ -71,7 +102,8 @@ public class EventListener implements Listener {
                 }
 
                 // Check for signs.
-                if (plugin.signcheck == true) {
+                if (plugin.signcheck == true && block instanceof Stairs) {
+                    Stairs stairs = (Stairs) block.getState().getData();
                     boolean sign1 = false;
                     boolean sign2 = false;
 
@@ -89,7 +121,8 @@ public class EventListener implements Listener {
                 }
 
                 // Check for maximal chair width.
-                if (plugin.maxchairwidth > 0) {
+                if (plugin.maxchairwidth > 0 && block instanceof Stairs) {
+                    Stairs stairs = (Stairs) block.getState().getData();
                     if (stairs.getDescendingDirection() == BlockFace.NORTH || stairs.getDescendingDirection() == BlockFace.SOUTH) {
                         chairwidth += getChairWidth(block, BlockFace.EAST);
                         chairwidth += getChairWidth(block, BlockFace.WEST);
@@ -109,18 +142,16 @@ public class EventListener implements Listener {
                         player.getVehicle().remove();
                     }
 
-                    Item drop = dropSeat(block);
-                    List<Item> drops = checkChair(drop);
-
-                    if (drops != null) {
-                        drop.remove();
-                        return;
-                    }
-
                     // Rotate the player's view to the descending side of the block.
-                    if (plugin.autorotate == true) {
+                    
-                        Location plocation = player.getLocation();
+                    Packet40EntityMetadata packet = new Packet40EntityMetadata(player.getPlayer().getEntityId(), new ChairWatcher((byte) 4), false);
-
+                    for (Player play : Bukkit.getOnlinePlayers()) {
+                        ((CraftPlayer) play).getHandle().netServerHandler.sendPacket(packet);
+                    }
+                                        
+                    if (plugin.autorotate == true && block instanceof Stairs) {
+                        Location plocation = block.getLocation();
+                        Stairs stairs = (Stairs) block.getState().getData();
                         switch (stairs.getDescendingDirection()) {
                             case NORTH:
                                 plocation.setYaw(90);
@@ -136,14 +167,16 @@ public class EventListener implements Listener {
                         }
 
                         player.teleport(plocation);
+
+                    } else {
+                        player.teleport(block.getLocation().add(0.5D, 0.0D, 0.5D));
                     }
-
+                    player.setSneaking(true);
-                    // Changing the drop material is only necessary for the item merge feature of CB++
+                    if (plugin.notifyplayer) {
-                    // The client won't update the material, though.
+                        player.sendMessage(ChatColor.GRAY + "You are now sitting.");
-                    drop.setItemStack(new ItemStack(Material.PUMPKIN_STEM));
+                    }
-                    drop.setPassenger(player);
+                    plugin.sit.put(player.getName(), block.getLocation());
-
+                    
-                    // Cancel BlockPlaceEvent Result, if player is rightclicking with a block in his hand.
                     event.setUseInteractedBlock(Result.DENY);
                 }
             }
@@ -177,30 +210,14 @@ public class EventListener implements Listener {
 
     private Item dropSeat(Block chair) {
         Location location = chair.getLocation().add(0.5, (plugin.sittingheight - 0.5), 0.5);
-        Item drop = location.getWorld().dropItemNaturally(location, new ItemStack(plugin.item));
+
+        Item drop = location.getWorld().dropItem(location, new ItemStack(plugin.item));
         drop.setPickupDelay(Integer.MAX_VALUE);
         drop.teleport(location);
         drop.setVelocity(new Vector(0, 0, 0));
         return drop;
     }
 
-    private List<Item> checkChair(Item drop) {
-        List<Item> drops = new ArrayList<Item>();
-
-        // Check for already existing chair items.
-        for (Entity e : drop.getNearbyEntities(0.2, 0.2, 0.2)) {
-            if (e != null && e instanceof Item && e.getPassenger() != null) {
-                drops.add(drop);
-            }
-        }
-
-        if (drops.isEmpty() == false) {
-            return drops;
-        }
-
-        return null;
-    }
-
     private int getChairWidth(Block block, BlockFace face) {
         int width = 0;
 

src/plugin.yml

@@ -1,7 +1,9 @@
 name: Chairs
-version: 1.5b
+version: 1.7
 description: Let players sit on stairs and slabs.
-authors: spoothie, cnaude
+authors: 
+ - spoothie
+ - cnaude
 main: net.spoothie.chairs.Chairs
 commands:
     chairs: