Wrong description for script function pcblockmove(<id>,<flag>);

Originally posted by [b]_Okuz_[/b]
Hello,

I'm here to report some issues related to this command.

script_commands.txt:

[quote]*pcblockmove <id>,<option>;

Prevents the given ID from moving when the option != 0, 0 enables the ID to move again.
ID should be able to be GID of a monster/NPC or AID from a character.

Examples:

//prevents the current char from moving away;
pcblockmove getcharid(3),1;

//enables the current char to move again.
pcblockmove getcharid(3),0;[/quote]

Source Code (script.c):

[CODE]BUILDIN_FUNC(pcblockmove)
{
int id, flag;
TBL_PC *sd = NULL;

id = script_getnum(st,2);
flag = script_getnum(st,3);

if (id)
sd = map_id2sd(id);
else
sd = script_rid2sd(st);

if (sd)
sd->state.blockedmove = flag > 0;

return 0;
}

BUILDIN_DEF(pcblockmove,"ii"),[/CODE]

According to source code, ID is not really necessary because it'll use player's RID instead of ID if ID was not found. But, WTF? The BUILDIN_DEF says that I must put ID.. So, something is wrong here.. And besides that, I'm not sure if it'll block Monsters and NPCs because sd is a pointer to TBL_PC.

Anyways, can someone take a look please?

Thanks.

PS: I'm sorry about my grammar haha