[X-Script]

You don't need the quotation marks in that place if your going to bind a key to bind those, if you want to do that you'll have to do this:

bind KEY "bind 1 slot1 ; bind 2 slot2 ; bind 3 slot3 ; bind 4 slot4 ; bind 5 slot5 ; bind 6 slot6 ; bind 7 slot7 ; bind 8 slot 8 ; bind 9 slot9 ; bind 0 slot10"

[BlackMilk]

He said unbind.

[X-Script]

Fahh. woops. I could've sworn it said bind when I read the post. Sorry, on the bright-side my post could still help someone.

[AdamR]

hehe np That unbind line is a quick way to erase what the keys to do test that the plugin works correctly.

I'd like to ask, does this plugin really count as slowhacking? Yes, it executes a command on a client, but nothing malicious. The user gets confirmation before the action is carried out, so why wouldn't this get accepted? It clearly helps people.

[X-Script]

AdamR is correct the user is being notifiied before any actions to his character are executed and it also doesn't do anything harmfull to the player.

Things i'd suggest:

1. Make a command /unbindslots unbinding them.

2.After the player said /bindslots create a menu saying "Are you sure?" "Yes" "No"

[AdamR]

Hmmm..

1: If I did /unbindslots, what would be the purpose for this other than testing? ;)

2: There already is a confirmation menu like that just worded a bit differently.

[X-Script]

Then this should be approved.

[Lee]

You should return PLUGIN_HANDLED in bindadminslots(). If you intend to cater for languages other than English, implement multi-lingual support properly.

You only need a variable if you intend to use the value later.

//Pointless:
new menuReply = str_to_num(data)
switch(menuReply)

//Correct:
switch(str_to_num(data))

It's good to see set_task() has gone.

[X-Script]

I don't know if he has implemented Lee's suggestion (which he should) but when he does, I personally think this should be approved.

[Howdy!]

Ehh, the same thing could be done with existing plugins.