Update players when potion effects are moved from them. Fixes BUKKIT-789

src/main/java/net/minecraft/server/EntityLiving.java

@@ -67,7 +67,7 @@ public abstract class EntityLiving extends Entity {
     public int aI = 0;
     public int aJ = 0;
     public HashMap effects = new HashMap(); // CraftBukkit - protected -> public
-    private boolean e = true;
+    public boolean e = true; // CraftBukkit - private -> public
     private int f;
     private ControllerLook lookController;
     private ControllerMove moveController;

src/main/java/org/bukkit/craftbukkit/entity/CraftLivingEntity.java

@@ -17,6 +17,7 @@ import net.minecraft.server.EntitySnowball;
 import net.minecraft.server.EntityPlayer;
 import net.minecraft.server.MobEffect;
 import net.minecraft.server.MobEffectList;
+import net.minecraft.server.Packet42RemoveMobEffect;
 
 import org.bukkit.Location;
 import org.bukkit.block.Block;
@@ -237,6 +238,10 @@ public class CraftLivingEntity extends CraftEntity implements LivingEntity {
 
     public void removePotionEffect(PotionEffectType type) {
         getHandle().effects.remove(type.getId());
+        getHandle().e = true; // Should be called updateEffects
+        if (getHandle() instanceof EntityPlayer) {
+            ((EntityPlayer)getHandle()).netServerHandler.sendPacket(new Packet42RemoveMobEffect(getHandle().id, new MobEffect(type.getId(), 0, 0)));
+        }
     }
 
     public Collection<PotionEffect> getActivePotionEffects() {