[MP2]

Your code is really messy because of the way you do:

pawn Код:

if(this)
{
    if(that)
    {
        // Code
    }
    else
    {
        // Error
    }
}
else
{
    // Error
}

it's sooooooooooo much easier to read if you just use one return line:

pawn Код:

if(!this) return error;
if(!that) return error;
// Code

Reduced from 15 lines to 3.

Why are you using strtok? 'params' will contain the ENTIRE player ID/name string, there is no need to use strtok at all.