Removal of RequestAlchemyConversion inventory update.

Contributed by teris1994.

L2J_Mobius_01.0_Ertheia/java/org/l2jmobius/gameserver/network/clientpackets/alchemy/RequestAlchemyConversion.java

@@ -32,7 +32,6 @@ import org.l2jmobius.gameserver.network.GameClient;
 import org.l2jmobius.gameserver.network.PacketLogger;
 import org.l2jmobius.gameserver.network.SystemMessageId;
 import org.l2jmobius.gameserver.network.clientpackets.IClientIncomingPacket;
-import org.l2jmobius.gameserver.network.serverpackets.InventoryUpdate;
 import org.l2jmobius.gameserver.network.serverpackets.alchemy.ExAlchemyConversion;
 import org.l2jmobius.gameserver.taskmanager.AttackStanceTaskManager;
 
@@ -182,29 +181,23 @@ public class RequestAlchemyConversion implements IClientIncomingPacket
 			return;
 		}
 		
-		final InventoryUpdate ui = new InventoryUpdate();
-		
 		// Destroy ingredients.
 		for (ItemHolder ingredient : data.getIngredients())
 		{
 			final Item item = player.getInventory().getItemByItemId(ingredient.getId());
-			ui.addItem(item);
 			player.getInventory().destroyItem("Alchemy", item, ingredient.getCount() * _craftTimes, player, null);
 		}
 		// Add success items.
 		if (successCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
-			ui.addItem(item);
 		}
 		// Add failed items.
 		if (failureCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
-			ui.addItem(item);
 		}
 		
 		player.sendPacket(new ExAlchemyConversion(successCount, failureCount));
-		player.sendInventoryUpdate(ui);
 	}
 }

L2J_Mobius_02.5_Underground/java/org/l2jmobius/gameserver/network/clientpackets/alchemy/RequestAlchemyConversion.java

@@ -32,7 +32,6 @@ import org.l2jmobius.gameserver.network.GameClient;
 import org.l2jmobius.gameserver.network.PacketLogger;
 import org.l2jmobius.gameserver.network.SystemMessageId;
 import org.l2jmobius.gameserver.network.clientpackets.IClientIncomingPacket;
-import org.l2jmobius.gameserver.network.serverpackets.InventoryUpdate;
 import org.l2jmobius.gameserver.network.serverpackets.alchemy.ExAlchemyConversion;
 import org.l2jmobius.gameserver.taskmanager.AttackStanceTaskManager;
 
@@ -182,29 +181,23 @@ public class RequestAlchemyConversion implements IClientIncomingPacket
 			return;
 		}
 		
-		final InventoryUpdate ui = new InventoryUpdate();
-		
 		// Destroy ingredients.
 		for (ItemHolder ingredient : data.getIngredients())
 		{
 			final Item item = player.getInventory().getItemByItemId(ingredient.getId());
-			ui.addItem(item);
 			player.getInventory().destroyItem("Alchemy", item, ingredient.getCount() * _craftTimes, player, null);
 		}
 		// Add success items.
 		if (successCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
-			ui.addItem(item);
 		}
 		// Add failed items.
 		if (failureCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
-			ui.addItem(item);
 		}
 		
 		player.sendPacket(new ExAlchemyConversion(successCount, failureCount));
-		player.sendInventoryUpdate(ui);
 	}
 }

L2J_Mobius_03.0_Helios/java/org/l2jmobius/gameserver/network/clientpackets/alchemy/RequestAlchemyConversion.java

@@ -32,7 +32,6 @@ import org.l2jmobius.gameserver.network.GameClient;
 import org.l2jmobius.gameserver.network.PacketLogger;
 import org.l2jmobius.gameserver.network.SystemMessageId;
 import org.l2jmobius.gameserver.network.clientpackets.IClientIncomingPacket;
-import org.l2jmobius.gameserver.network.serverpackets.InventoryUpdate;
 import org.l2jmobius.gameserver.network.serverpackets.alchemy.ExAlchemyConversion;
 import org.l2jmobius.gameserver.taskmanager.AttackStanceTaskManager;
 
@@ -182,29 +181,23 @@ public class RequestAlchemyConversion implements IClientIncomingPacket
 			return;
 		}
 		
-		final InventoryUpdate ui = new InventoryUpdate();
-		
 		// Destroy ingredients.
 		for (ItemHolder ingredient : data.getIngredients())
 		{
 			final Item item = player.getInventory().getItemByItemId(ingredient.getId());
-			ui.addItem(item);
 			player.getInventory().destroyItem("Alchemy", item, ingredient.getCount() * _craftTimes, player, null);
 		}
 		// Add success items.
 		if (successCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
-			ui.addItem(item);
 		}
 		// Add failed items.
 		if (failureCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
-			ui.addItem(item);
 		}
 		
 		player.sendPacket(new ExAlchemyConversion(successCount, failureCount));
-		player.sendInventoryUpdate(ui);
 	}
 }

L2J_Mobius_04.0_GrandCrusade/java/org/l2jmobius/gameserver/network/clientpackets/alchemy/RequestAlchemyConversion.java

@@ -32,7 +32,6 @@ import org.l2jmobius.gameserver.network.GameClient;
 import org.l2jmobius.gameserver.network.PacketLogger;
 import org.l2jmobius.gameserver.network.SystemMessageId;
 import org.l2jmobius.gameserver.network.clientpackets.IClientIncomingPacket;
-import org.l2jmobius.gameserver.network.serverpackets.InventoryUpdate;
 import org.l2jmobius.gameserver.network.serverpackets.alchemy.ExAlchemyConversion;
 import org.l2jmobius.gameserver.taskmanager.AttackStanceTaskManager;
 
@@ -182,29 +181,23 @@ public class RequestAlchemyConversion implements IClientIncomingPacket
 			return;
 		}
 		
-		final InventoryUpdate ui = new InventoryUpdate();
-		
 		// Destroy ingredients.
 		for (ItemHolder ingredient : data.getIngredients())
 		{
 			final Item item = player.getInventory().getItemByItemId(ingredient.getId());
-			ui.addItem(item);
 			player.getInventory().destroyItem("Alchemy", item, ingredient.getCount() * _craftTimes, player, null);
 		}
 		// Add success items.
 		if (successCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
-			ui.addItem(item);
 		}
 		// Add failed items.
 		if (failureCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
-			ui.addItem(item);
 		}
 		
 		player.sendPacket(new ExAlchemyConversion(successCount, failureCount));
-		player.sendInventoryUpdate(ui);
 	}
 }

L2J_Mobius_05.0_Salvation/java/org/l2jmobius/gameserver/network/clientpackets/alchemy/RequestAlchemyConversion.java

@@ -32,7 +32,6 @@ import org.l2jmobius.gameserver.network.GameClient;
 import org.l2jmobius.gameserver.network.PacketLogger;
 import org.l2jmobius.gameserver.network.SystemMessageId;
 import org.l2jmobius.gameserver.network.clientpackets.IClientIncomingPacket;
-import org.l2jmobius.gameserver.network.serverpackets.InventoryUpdate;
 import org.l2jmobius.gameserver.network.serverpackets.alchemy.ExAlchemyConversion;
 import org.l2jmobius.gameserver.taskmanager.AttackStanceTaskManager;
 
@@ -182,29 +181,23 @@ public class RequestAlchemyConversion implements IClientIncomingPacket
 			return;
 		}
 		
-		final InventoryUpdate ui = new InventoryUpdate();
-		
 		// Destroy ingredients.
 		for (ItemHolder ingredient : data.getIngredients())
 		{
 			final Item item = player.getInventory().getItemByItemId(ingredient.getId());
-			ui.addItem(item);
 			player.getInventory().destroyItem("Alchemy", item, ingredient.getCount() * _craftTimes, player, null);
 		}
 		// Add success items.
 		if (successCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
-			ui.addItem(item);
 		}
 		// Add failed items.
 		if (failureCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
-			ui.addItem(item);
 		}
 		
 		player.sendPacket(new ExAlchemyConversion(successCount, failureCount));
-		player.sendInventoryUpdate(ui);
 	}
 }

L2J_Mobius_05.5_EtinasFate/java/org/l2jmobius/gameserver/network/clientpackets/alchemy/RequestAlchemyConversion.java

@@ -32,7 +32,6 @@ import org.l2jmobius.gameserver.network.GameClient;
 import org.l2jmobius.gameserver.network.PacketLogger;
 import org.l2jmobius.gameserver.network.SystemMessageId;
 import org.l2jmobius.gameserver.network.clientpackets.IClientIncomingPacket;
-import org.l2jmobius.gameserver.network.serverpackets.InventoryUpdate;
 import org.l2jmobius.gameserver.network.serverpackets.alchemy.ExAlchemyConversion;
 import org.l2jmobius.gameserver.taskmanager.AttackStanceTaskManager;
 
@@ -182,29 +181,23 @@ public class RequestAlchemyConversion implements IClientIncomingPacket
 			return;
 		}
 		
-		final InventoryUpdate ui = new InventoryUpdate();
-		
 		// Destroy ingredients.
 		for (ItemHolder ingredient : data.getIngredients())
 		{
 			final Item item = player.getInventory().getItemByItemId(ingredient.getId());
-			ui.addItem(item);
 			player.getInventory().destroyItem("Alchemy", item, ingredient.getCount() * _craftTimes, player, null);
 		}
 		// Add success items.
 		if (successCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
-			ui.addItem(item);
 		}
 		// Add failed items.
 		if (failureCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
-			ui.addItem(item);
 		}
 		
 		player.sendPacket(new ExAlchemyConversion(successCount, failureCount));
-		player.sendInventoryUpdate(ui);
 	}
 }

L2J_Mobius_06.0_Fafurion/java/org/l2jmobius/gameserver/network/clientpackets/alchemy/RequestAlchemyConversion.java

@@ -32,7 +32,6 @@ import org.l2jmobius.gameserver.network.GameClient;
 import org.l2jmobius.gameserver.network.PacketLogger;
 import org.l2jmobius.gameserver.network.SystemMessageId;
 import org.l2jmobius.gameserver.network.clientpackets.IClientIncomingPacket;
-import org.l2jmobius.gameserver.network.serverpackets.InventoryUpdate;
 import org.l2jmobius.gameserver.network.serverpackets.alchemy.ExAlchemyConversion;
 import org.l2jmobius.gameserver.taskmanager.AttackStanceTaskManager;
 
@@ -182,29 +181,23 @@ public class RequestAlchemyConversion implements IClientIncomingPacket
 			return;
 		}
 		
-		final InventoryUpdate ui = new InventoryUpdate();
-		
 		// Destroy ingredients.
 		for (ItemHolder ingredient : data.getIngredients())
 		{
 			final Item item = player.getInventory().getItemByItemId(ingredient.getId());
-			ui.addItem(item);
 			player.getInventory().destroyItem("Alchemy", item, ingredient.getCount() * _craftTimes, player, null);
 		}
 		// Add success items.
 		if (successCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
-			ui.addItem(item);
 		}
 		// Add failed items.
 		if (failureCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
-			ui.addItem(item);
 		}
 		
 		player.sendPacket(new ExAlchemyConversion(successCount, failureCount));
-		player.sendInventoryUpdate(ui);
 	}
 }

L2J_Mobius_07.0_PreludeOfWar/java/org/l2jmobius/gameserver/network/clientpackets/alchemy/RequestAlchemyConversion.java

@@ -32,7 +32,6 @@ import org.l2jmobius.gameserver.network.GameClient;
 import org.l2jmobius.gameserver.network.PacketLogger;
 import org.l2jmobius.gameserver.network.SystemMessageId;
 import org.l2jmobius.gameserver.network.clientpackets.IClientIncomingPacket;
-import org.l2jmobius.gameserver.network.serverpackets.InventoryUpdate;
 import org.l2jmobius.gameserver.network.serverpackets.alchemy.ExAlchemyConversion;
 import org.l2jmobius.gameserver.taskmanager.AttackStanceTaskManager;
 
@@ -182,29 +181,23 @@ public class RequestAlchemyConversion implements IClientIncomingPacket
 			return;
 		}
 		
-		final InventoryUpdate ui = new InventoryUpdate();
-		
 		// Destroy ingredients.
 		for (ItemHolder ingredient : data.getIngredients())
 		{
 			final Item item = player.getInventory().getItemByItemId(ingredient.getId());
-			ui.addItem(item);
 			player.getInventory().destroyItem("Alchemy", item, ingredient.getCount() * _craftTimes, player, null);
 		}
 		// Add success items.
 		if (successCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
-			ui.addItem(item);
 		}
 		// Add failed items.
 		if (failureCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
-			ui.addItem(item);
 		}
 		
 		player.sendPacket(new ExAlchemyConversion(successCount, failureCount));
-		player.sendInventoryUpdate(ui);
 	}
 }

L2J_Mobius_08.2_Homunculus/java/org/l2jmobius/gameserver/network/clientpackets/alchemy/RequestAlchemyConversion.java

@@ -32,7 +32,6 @@ import org.l2jmobius.gameserver.network.GameClient;
 import org.l2jmobius.gameserver.network.PacketLogger;
 import org.l2jmobius.gameserver.network.SystemMessageId;
 import org.l2jmobius.gameserver.network.clientpackets.IClientIncomingPacket;
-import org.l2jmobius.gameserver.network.serverpackets.InventoryUpdate;
 import org.l2jmobius.gameserver.network.serverpackets.alchemy.ExAlchemyConversion;
 import org.l2jmobius.gameserver.taskmanager.AttackStanceTaskManager;
 
@@ -182,29 +181,23 @@ public class RequestAlchemyConversion implements IClientIncomingPacket
 			return;
 		}
 		
-		final InventoryUpdate ui = new InventoryUpdate();
-		
 		// Destroy ingredients.
 		for (ItemHolder ingredient : data.getIngredients())
 		{
 			final Item item = player.getInventory().getItemByItemId(ingredient.getId());
-			ui.addItem(item);
 			player.getInventory().destroyItem("Alchemy", item, ingredient.getCount() * _craftTimes, player, null);
 		}
 		// Add success items.
 		if (successCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
-			ui.addItem(item);
 		}
 		// Add failed items.
 		if (failureCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
-			ui.addItem(item);
 		}
 		
 		player.sendPacket(new ExAlchemyConversion(successCount, failureCount));
-		player.sendInventoryUpdate(ui);
 	}
 }

L2J_Mobius_09.2_ReturnOfTheQueenAnt/java/org/l2jmobius/gameserver/network/clientpackets/alchemy/RequestAlchemyConversion.java

@@ -32,7 +32,6 @@ import org.l2jmobius.gameserver.network.GameClient;
 import org.l2jmobius.gameserver.network.PacketLogger;
 import org.l2jmobius.gameserver.network.SystemMessageId;
 import org.l2jmobius.gameserver.network.clientpackets.IClientIncomingPacket;
-import org.l2jmobius.gameserver.network.serverpackets.InventoryUpdate;
 import org.l2jmobius.gameserver.network.serverpackets.alchemy.ExAlchemyConversion;
 import org.l2jmobius.gameserver.taskmanager.AttackStanceTaskManager;
 
@@ -182,29 +181,23 @@ public class RequestAlchemyConversion implements IClientIncomingPacket
 			return;
 		}
 		
-		final InventoryUpdate ui = new InventoryUpdate();
-		
 		// Destroy ingredients.
 		for (ItemHolder ingredient : data.getIngredients())
 		{
 			final Item item = player.getInventory().getItemByItemId(ingredient.getId());
-			ui.addItem(item);
 			player.getInventory().destroyItem("Alchemy", item, ingredient.getCount() * _craftTimes, player, null);
 		}
 		// Add success items.
 		if (successCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
-			ui.addItem(item);
 		}
 		// Add failed items.
 		if (failureCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
-			ui.addItem(item);
 		}
 		
 		player.sendPacket(new ExAlchemyConversion(successCount, failureCount));
-		player.sendInventoryUpdate(ui);
 	}
 }

L2J_Mobius_10.2_MasterClass/java/org/l2jmobius/gameserver/network/clientpackets/alchemy/RequestAlchemyConversion.java

@@ -32,7 +32,6 @@ import org.l2jmobius.gameserver.network.GameClient;
 import org.l2jmobius.gameserver.network.PacketLogger;
 import org.l2jmobius.gameserver.network.SystemMessageId;
 import org.l2jmobius.gameserver.network.clientpackets.IClientIncomingPacket;
-import org.l2jmobius.gameserver.network.serverpackets.InventoryUpdate;
 import org.l2jmobius.gameserver.network.serverpackets.alchemy.ExAlchemyConversion;
 import org.l2jmobius.gameserver.taskmanager.AttackStanceTaskManager;
 
@@ -182,29 +181,23 @@ public class RequestAlchemyConversion implements IClientIncomingPacket
 			return;
 		}
 		
-		final InventoryUpdate ui = new InventoryUpdate();
-		
 		// Destroy ingredients.
 		for (ItemHolder ingredient : data.getIngredients())
 		{
 			final Item item = player.getInventory().getItemByItemId(ingredient.getId());
-			ui.addItem(item);
 			player.getInventory().destroyItem("Alchemy", item, ingredient.getCount() * _craftTimes, player, null);
 		}
 		// Add success items.
 		if (successCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionSuccess().getId(), data.getProductionSuccess().getCount() * successCount, player, null);
-			ui.addItem(item);
 		}
 		// Add failed items.
 		if (failureCount > 0)
 		{
-			final Item item = player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
+			player.getInventory().addItem("Alchemy", data.getProductionFailure().getId(), data.getProductionFailure().getCount() * failureCount, player, null);
-			ui.addItem(item);
 		}
 		
 		player.sendPacket(new ExAlchemyConversion(successCount, failureCount));
-		player.sendInventoryUpdate(ui);
 	}
 }