Just at a quick glance:
PHP Code:
if (!this.cloner_on)
{
this.cloner_on = true;
player.chat = "Outfit Cloner: On";
}
else
{
this.cloner_on = false;
player.chat = "Outfit Cloner: Off";
}
... had might as well be:
PHP Code:
this.cloner_on = !this.cloner_on;
player.chat = "Outfit Cloner: " @ (this.cloner_on ? "On" : "Off");
Also.
PHP Code:
for (p: players)
{
for (n: npcs)
{
Do you need to scan every NPC for every player? Can't you just scan them once?
Finally, can't you prefix your temp vars with temp.?
