[S4t3K]

@Greentarch : This one looks clearer :

PHP Code:


public OnPlayerCommandPerformed( playerid, cmdtext[ ], success )
{
      return success ? 1 : SendClientMessage(playerid, -1, ">> That command doesn't exist! (/cmds)");
} 


The wiki says that "SendClientMessage" returns 1 if the function executes successful. This way, returning 1 just after that would be a bit useless.

@Author : Some parts of your "COMMON ERRORS" are very bad coded.

I take this code as an example :

pawn Code:

public OnPlayerCommandPerformed( playerid, cmdtext[ ], success ) 
{ 
    if( success == 0 ) 
    { 
        
     } 
     return SendClientMessage( playerid, COLOR, "Your new unknown command text" ); 
     return 0; 
}

It won't work as a solution ! You'll even get an error during the compilation !

This one would work :

pawn Code:

public OnPlayerCommandPerformed( playerid, cmdtext[ ], success ) 
{ 
    if(!success) return SendClientMessage( playerid, COLOR, "Your new unknown command text" ); 
    return 1;
}

Because as Zeex said :

Quote:

Originally Posted by Zeex pawn Code: OnPlayerCommandPerformed(playerid, cmdtext[], success) And this one gets called after command execution, here if you do "return 0" the player will see standard "Unknown command" message. The "success" parameter is equal to value returned by command function returns (if it doesn't exist success will be 0).

Correct your whole tutorial please.