Proper RequestCancelPostAttachment InventoryUpdate logic.

Contributed by aviengier.

L2J_Mobius_01.0_Ertheia/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList(false);
-			player.sendItemList(false);
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_02.5_Underground/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList(false);
-			player.sendItemList(false);
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_03.0_Helios/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList(false);
-			player.sendItemList(false);
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_04.0_GrandCrusade/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList(false);
-			player.sendItemList(false);
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_05.0_Salvation/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_05.5_EtinasFate/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_06.0_Fafurion/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_07.0_PreludeOfWar/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_08.2_Homunculus/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_09.2_ReturnOfTheQueenAnt/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_10.0_MasterClass/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_CT_2.4_Epilogue/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -180,7 +180,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -202,10 +202,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendPacket(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendPacket(new ItemList(player, false));
-			player.sendPacket(new ItemList(player, false));
-		}
 		
 		// Update current load status on player
 		final StatusUpdate su = new StatusUpdate(player);

L2J_Mobius_CT_2.6_HighFive/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -180,7 +180,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -202,10 +202,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendPacket(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendPacket(new ItemList(player, false));
-			player.sendPacket(new ItemList(player, false));
-		}
 		
 		// Update current load status on player
 		final StatusUpdate su = new StatusUpdate(player);

L2J_Mobius_Classic_2.0_Saviors/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList(false);
-			player.sendItemList(false);
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_Classic_2.1_Zaken/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList(false);
-			player.sendItemList(false);
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_Classic_2.2_Antharas/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_Classic_2.3_SevenSigns/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_Classic_2.4_SecretOfEmpire/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_Classic_3.0_TheKamael/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_Classic_Interlude/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList(false);
-			player.sendItemList(false);
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_Essence_4.2_DwellingOfSpirits/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_Essence_5.2_FrostLord/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)

L2J_Mobius_Essence_6.0_BattleChronicle/java/org/l2jmobius/gameserver/network/clientpackets/RequestCancelPostAttachment.java

@@ -178,7 +178,7 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 			
 			if (playerIU != null)
 			{
-				if (newItem.getCount() > count)
+				if (newItem.isStackable() && (newItem.getCount() > count))
 				{
 					playerIU.addModifiedItem(newItem);
 				}
@@ -200,10 +200,8 @@ public class RequestCancelPostAttachment implements IClientIncomingPacket
 		{
 			player.sendInventoryUpdate(playerIU);
 		}
-		else
+		// Send full list to avoid duplicates.
-		{
+		player.sendItemList();
-			player.sendItemList();
-		}
 		
 		final Player receiver = World.getInstance().getPlayer(msg.getReceiverId());
 		if (receiver != null)