Thread: z508 staff spawn items in wild

Originally Posted by Paul

Code:

if (cmd[0].equalsIgnoreCase("item") && p.jailed == 0) {
					int itemID = Integer.parseInt(cmd[1]);
					int itemAmount = Integer.parseInt(cmd[2]);
					String itemName = Engine.items.getItemName(itemID);
					if (p.rights > 2) {
						Engine.playerItems.addItem(p, itemID, itemAmount);
						return;
					}
					if (p.wildernessZone(p.absX, p.absY)) {
						p
								.getActionSender()
								.sendMessage(p,
										"You cannot spawn items while inside of the wilderness.");
						return;

Something close to that, Post any errors.

You're going to get an error. You didn't close two brackets. And your conventions are putrid.

Code:

if (cmd[0].equalsIgnoreCase("item")) {
	if (p.jailed) {
		p.getActionSender().sendMessage(p, "You cannot spawn items while you're jailed.");
		return;
	}
	if (p.rights <= 1) {
		p.getActionSender().sendMessage(p, "You do not have sufficient rights to spawn items.");
		return;
	}
	if (p.wildernessZone(p.absX, p.absY)) {
		p.getActionSender().sendMessage(p, "You cannot spawn items while inside of the Wilderness.");
		return;
	}
	int itemID = Integer.parseInt(cmd[1]);
	int itemAmount = Integer.parseInt(cmd[2]);
	String itemName = Engine.items.getItemName(itemID);
}

This is plenty better than the garbage above this, and the garbage above this lets people with rights greater than 2 spawn. So, even admins can't spawn with that code. Also, for this to work, your jail system has to use a boolean (like it should), not an int.