if(%cl.JailRPC < $JailRP::Price::sKnife)
{
messageClient(%cl, "", "\c6You can't afford a Small Knife.");
return;
}
for(%i=0; %i<%cl.player.getDatablock().maxTools; %i++)
{
%tool = %cl.player.tool[%i];
if(%tool == 0)
{
%cl.player.tool[%i] = swordItem.getID();
%cl.player.weaponCount++;
messageClient(%cl, 'MsgItemPickup', '', %i, swordItem.getID());
%i = %cl.player.getDatablock().maxTools;
%transfersuccess = true;
}
}
if(%transferSuccess)
{
%cl.JailRPC -= $JailRP::Price::sKnife;
messageClient(%cl, "", "\c6Alright, here's your Small Knife. Keep it hidden.");
}
else
messageClient(%cl, "", "\c6I don't think you can hold another item.");
This is from a JailRP mod I made like a year back. I think it takes code from the additem event.
Looking at it, I think you need a break; after the line %transfersucess = true;