Is that right?

dcmd_ban(playerid,params[])
{
    #pragma unused params
    new pname[MAX_PLAYER_NAME];
    new Spname[MAX_PLAYER_NAME];
	new string[128];
	new cmdreason[100];
	new id;
	if(sscanf(params,"us[100]",id,cmdreason))
	{
	    SendClientMessage(playerid,COLOR_ERROR,"Usage: /ban (Player Name/ID) (Reason)");
	    return 1;
	}
    if(!IsPlayerConnected(id))
	{
	    SendClientMessage(playerid, COLOR_RED, "The player is not connected to the server");
	    return 1;
	}
    GetPlayerName(id, pname, sizeof(pname));
	format(string,sizeof(string),"***An Administrator has banned %s(%d) from the server.  (Reason: %s)",pname,id,cmdreason);
	SendClientMessageToAll(COLOR_PINK,string);
	PlayerInfo[id][pBanned] =1;
    GameTextForPlayer(id, "~r~BANNED",1000000,4);
	Kick(id);

    GetPlayerName(id, pname, sizeof(pname));
	format(string,sizeof(string),"***An Administrator has banned %s(%d) from the server.  (Reason: %s)",pname,id,cmdreason);
    IRC_GroupSay(groupID,IRC_CHANNEL,string);
    IRC_GroupSay(groupID,ADMIN_CHANNEL,string);

	GetPlayerName(id,pname,MAX_PLAYER_NAME);
    GetPlayerName(playerid,Spname,MAX_PLAYER_NAME);
	format(string,sizeof(string),"[ADMIN UPDATE] Administrator %s(%d) has banned %s(%d). (Reason: %s)",Spname,playerid,pname,id,cmdreason);
	SendAdminMessage(COLOR_PINK, string);
	return 1;
}
//============================================================================//
dcmd_unban(playerid,params[])
{
    #pragma unused params
    new pname[MAX_PLAYER_NAME];
    new Spname[MAX_PLAYER_NAME];
	new string[128];
	new cmdreason[100];
	new id;
	if(sscanf(params,"us[100]",id,cmdreason))
	{
	    SendClientMessage(playerid,COLOR_ERROR,"Usage: /ban (Player Name/ID) (Reason)");
	    return 1;
	}
    if(fexist(UserPath(id)))
    {
        new INI:File = INI_Open(UserPath(id));
        GetPlayerName(id, pname, sizeof(pname));
    	format(string,sizeof(string),"***An Administrator has un-banned %s(%d) from the server.",pname,id);
        IRC_GroupSay(groupID,IRC_CHANNEL,string);
        IRC_GroupSay(groupID,ADMIN_CHANNEL,string);
    	PlayerInfo[id][pBanned] =0;

        GetPlayerName(playerid,Spname,MAX_PLAYER_NAME);
    	format(string,sizeof(string),"[ADMIN UPDATE] Administrator %s(%d) has un-banned %s(%d).",Spname,playerid,cmdreason);
    	SendAdminMessage(COLOR_PINK, string);
        INI_Close(File);
    }
	return 1;
}
//============================================================================//
//Command BAN.
IRCCMD:iban(botid, channel[], user[], host[], params[])
{
	// Check if the user is at least an op in the channel
	if (IRC_IsOp(botid, channel, user))
	{
		new playerid, reason[64];
		new id;
		// Check if the user at least entered a player ID
        if (sscanf(params, "dS(No reason)[64]", playerid, reason))
		{
			return 1;
		}
		// Check if the player is connected
		if (IsPlayerConnected(id))
		{
			// Echo the formatted message
			new msg[128], name[MAX_PLAYER_NAME];
            GetPlayerName(id, name, sizeof(name));
			format(msg, sizeof(msg), "02***[IRC BAN] %s(%d) has been banned by an IRC Administrator. (Reason: %s)", name, id, reason);
			IRC_GroupSay(groupID, ADMIN_CHANNEL, msg);
			IRC_GroupSay(groupID, IRC_CHANNEL, msg);
			format(msg, sizeof(msg), "***[IRC BAN] %s(%d) has been banned by an IRC Administrator. (Reason: %s)", name, id, reason);
			SendClientMessageToAll(0xFF0080FF, msg);
			PlayerInfo[id][pBanned] =1;
            GameTextForPlayer(id, "~r~BANNED",1000000,4);
 			Kick(id);
		}
	}
	return 1;
}
//============================================================================//
//Command UINBAN.
IRCCMD:iunban(botid, channel[], user[], host[], params[])
{
	// Check if the user is at least an op in the channel
	if (IRC_IsOp(botid, channel, user))
	{
		new playerid, reason[64];
		new id;
		// Check if the user at least entered a player ID
		if (sscanf(params, "dS(No reason)[64]", playerid, reason))
		{
			return 1;
		}
		// Check if the player is connected
		if (IsPlayerConnected(id))
		{
            if(fexist(UserPath(id)))
           {
			// Echo the formatted message
            new INI:File = INI_Open(UserPath(id));
			new msg[128], name[MAX_PLAYER_NAME];
            GetPlayerName(id, name, sizeof(name));
			PlayerInfo[id][pBanned] =0;
			format(msg, sizeof(msg), "02***[IRC BAN] IRC Administrator %s has unbanned %s.", user, reason);
			IRC_GroupSay(groupID, ADMIN_CHANNEL, msg);
			IRC_GroupSay(groupID, IRC_CHANNEL, msg);
            INI_Close(File);
		   }
		}
	}
	return 1;
}