New command: !removeitem

Status
Not open for further replies.
Newbie Spellweaver
Joined
Mar 6, 2009
Messages
8
Reaction score
0
Code:
		} else if (splitted[0].equals("!removeitem")) {
			int iditem = Integer.parseInt(splitted[1]);
			MapleInventory iv = c.getPlayer().getInventory(MapleInventoryType.EQUIP);
			int possessed = iv.countById(iditem);
			if (possessed > 0) {
				MapleInventoryManipulator.removeById(c, MapleInventoryType.EQUIP, iditem, possessed, true, false);
				mc.dropMessage("All of item " + iditem + " in your inventory have been removed. (" + possessed + " Total)");
			} else if (possessed == 0) {
				MapleInventory iv1 = c.getPlayer().getInventory(MapleInventoryType.USE);
				int possessed1 = iv1.countById(iditem);
				if (possessed1 > 0) {
					MapleInventoryManipulator.removeById(c, MapleInventoryType.USE, iditem, possessed1, true, false);
					mc.dropMessage("All of item " + iditem + " in your inventory have been removed. (" + possessed1 + " Total)");
				} else if (possessed1 == 0) {
					MapleInventory iv2 = c.getPlayer().getInventory(MapleInventoryType.SETUP);
					int possessed2 = iv2.countById(iditem);
					if (possessed2 > 0) {
						MapleInventoryManipulator.removeById(c, MapleInventoryType.SETUP, iditem, possessed2, true, false);
						mc.dropMessage("All of item " + iditem + " in your inventory have been removed. (" + possessed2 + " Total)");
					} else if (possessed2 == 0) {
						MapleInventory iv3 = c.getPlayer().getInventory(MapleInventoryType.ETC);
						int possessed3 = iv3.countById(iditem);
						if (possessed3 > 0) {
							MapleInventoryManipulator.removeById(c, MapleInventoryType.ETC, iditem, possessed3, true, false);
							mc.dropMessage("All of item " + iditem + " in your inventory have been removed. (" + possessed3 + " Total)");
						} else if (possessed3 == 0) {
							MapleInventory iv4 = c.getPlayer().getInventory(MapleInventoryType.CASH);
							int possessed4 = iv1.countById(iditem);
							if (possessed4 > 0) {
								MapleInventoryManipulator.removeById(c, MapleInventoryType.CASH, iditem, possessed4, true, false);
								mc.dropMessage("All of item " + iditem + " in your inventory have been removed. (" + possessed4 + " Total)");
							} else if (possessed4 == 0) {
								mc.dropMessage("You do not currently own any of " + iditem);
							}
						}
					}
				}
			}

!removeitem <itemid>

Removes all of <itemid> from your inventory. Doesn't affect equipped items :).
 
PHP:
		} else if (splitted[0].equals("!removeitem")) {
			int iditem = Integer.parseInt(splitted[1]);
			MapleInventory iv = c.getPlayer().getInventory(MapleItemInformationProvider.getInstance().getInventoryType(iditem));
			int possessed = iv.countById(iditem);
			MapleInventoryManipulator.removeById(c, MapleItemInformationProvider.getInstance().getInventoryType(iditem), iditem, possessed, true, false);
		}
 
Last edited:
PHP:
		} else if (splitted[0].equals("!removeitem")) {
			int iditem = Integer.parseInt(splitted[1]);
			MapleInventory iv = c.getPlayer().getInventory(MapleItemInformationProvider.getInstance().getInventoryType(iditem));
			int possessed = iv.countById(iditem);
			MapleInventoryManipulator.removeById(c, MapleInventoryType.EQUIP, iditem, possessed, true, false);
		}

The difference between mine and yours (other than size) is that mine can remove items from any inventory type, yours will only remove equips. Mine also tells you where it removed the item from and how many you had.
 
PHP:
else if (splitted[0].equals("!removeitem")) {
            MapleInventoryManipulator.removeById(c, MapleInventoryType.EQUIP, Integer.parseInt(splitted[1]), c.getPlayer().getInventory(MapleItemInformationProvider.getInstance().getInventoryType(iditem)).countById(iditem), true, false);
        }

Anyways, MrMysterious is right.
 
ORLY ? Go try it out.

I already did test yours... I tested removing items in my equip inventory, worked fine. I then tested removing items in my etc inventory, nothing happened. Your code is set to check the item type but then it ignores that and tries to remove from the equip inventory.
 
1st: MR mysterious is right
2nd: same with moogra
3rd: your long ass command wastes so much bandwith when u can make it smaller
 
Status
Not open for further replies.
Back