Thread: monkey greegree

look up "You can't wear that item." and post the code

so i checked my source and found 4 if satements for "you can't wear that" :

Spoiler for code 1:

if (item.getDefinitions().isNoted() || !item.getDefinitions().isWearItem(player.getAppear ence().isMale())) { player.getPackets().sendGameMessage("You can't wear that."); return true; }

Spoiler for code 2:

if (targetSlot == -1) { player.getPackets().sendGameMessage("You can't wear that."); return true; }

Spoiler for code 3:

if (item.getDefinitions().isNoted() || !item.getDefinitions().isWearItem(player.getAppear ence().isMale()) && t.getId() != 4084) { player.getPackets().sendGameMessage("You can't wear that."); return false; }

Spoiler for code 4:

if (targetSlot == -1) { player.getPackets().sendGameMessage("You can't wear that."); return false; }

[Only registered and activated users can see links. ]

equip slot is 3 which is the weapon slot i believe.

equip type is -1 , is that the cause?

ok so i found out that if i add them items into buttonhandler in this code it allowed them to be equipped:

[Only registered and activated users can see links. ]

its porbably a horrible way to do, maybe someone knows a better way?
but i guess if any1 else runs into this issue like me atleast this method fixes it

Originally Posted by khayam

ok so i found out that if i add them items into buttonhandler in this code it allowed them to be equipped:

[Only registered and activated users can see links. ]

its porbably a horrible way to do, maybe someone knows a better way?
but i guess if any1 else runs into this issue like me atleast this method fixes it

yeah, that's a pretty bad way of fixing it. It's just an easy cheaphax way of fixing a problem, which happens a lot in rsps, and eventually it makes everything super unorganized and hard to read.

Imagine you found a bunch of items that you couldn't equip and you had to fix them all manually, like you've just done. That if statement is then going to become unnecessarily long.

Originally Posted by fish o matic

yeah, that's a pretty bad way of fixing it. It's just an easy cheaphax way of fixing a problem, which happens a lot in rsps, and eventually it makes everything super unorganized and hard to read.

Imagine you found a bunch of items that you couldn't equip and you had to fix them all manually, like you've just done. That if statement is then going to become unnecessarily long.

yup im still a noob in coding and wanting to learn any tips on how i could improve this? or a guide?