/news cmd
#7

pawn Code:
format(string, sizeof(string), "[SAN News] %s %s: %s", rtext, sendername, cmdtext);
You forget the GetPlayerName, and you replaced cmdtext with result. So this must be:
PHP Code:
new rtext[20];
if(
PlayerInfo[playerid][pRank] == 1) { rtext "Intern Worker"; }
else if(
PlayerInfo[playerid][pRank] == 2) { rtext "Journalist"; }
else if(
PlayerInfo[playerid][pRank] == 3) { rtext "Head Journalist"; }
else if(
PlayerInfo[playerid][pRank] == 4) { rtext "Company Secretary"; }
else if(
PlayerInfo[playerid][pRank] == 5) { rtext "ABC Manager"; }
else if(
PlayerInfo[playerid][pRank] == 6) { rtext "Network Producer"; }
else { 
rtext "Intern Worker"; }
GetPlayerName(playeridsendernameMAX_PLAYER_NAME);
format(stringsizeof(string), "[SAN News] %s %s: %s"rtextsendernameresult);
OOCNews(COLOR_NEWSstring); 
And at OOCNews function, check that player vals: pRadio & gNews. Maybe gNews at OnPlayerConnect is setted to 1, and / or pRadio to 0 ...

And this routine, can be optimizied a bit ...
pawn Code:
for(new i = 1; i < MAX_VEHICLES; i++)
                {
                    if(IsASANCar(i))
                    {
                        new dist = CheckDistanceToVehicle(7, playerid, i);
                        if(dist)
                        {
                            counter++;
                        }
                    }
                }
IN:
PHP Code:
                for(new 1MAX_VEHICLESi++)
                {
                    if(
IsASANCar(i))
                    {
                        new 
dist CheckDistanceToVehicle(7playeridi);
                        if(
dist)
                        {
                            
counter++;
                            break;
                        }
                    }
                } 
So, the whole command, looks like this:
PHP Code:
if(strcmp(cmd"/news"true) == 0)
{
    if(
IsPlayerConnected(playerid))
    {
        if(
PlayerInfo[playerid][pMember] == || PlayerInfo[playerid][pLeader] == 8)
        {
            if(
PlayerInfo[playerid][pMuted] == 1)
            {
                
SendClientMessage(playeridTEAM_CYAN_COLOR"Your muted.");
                return 
1;
            }
            new 
bool:counter = !true;
            for(new 
1MAX_VEHICLESi++)
            {
                if(
IsASANCar(i))
                {
                    if(
CheckDistanceToVehicle(7playeridi))
                    {
                        
counter true;
                        break;
                    }
                }
            }
            if(!
counter)
            {
                
SendClientMessage(playeridCOLOR_GREY"Your not inside/near the Chopper/Van.");
                return 
1;
            }
            new 
length strlen(cmdtext);
            while ((
idx length) && (cmdtext[idx] <= ' '))
            {
                
idx++;
            }
            new 
offset idx;
            new 
result[128];
            while ((
idx length) && ((idx offset) < (sizeof(result) - 1)))
            {
                
result[idx offset] = cmdtext[idx];
                
idx++;
            }
            
result[idx offset] = EOS;
            if(!
strlen(result))
            {
                
SendClientMessage(playeridCOLOR_GRAD2"USAGE: /news [newstext]");
                return 
1;
            }
            
            new 
rtext[20];
            if(
PlayerInfo[playerid][pRank] == 1) { rtext "Intern Worker"; }
            else if(
PlayerInfo[playerid][pRank] == 2) { rtext "Journalist"; }
            else if(
PlayerInfo[playerid][pRank] == 3) { rtext "Head Journalist"; }
            else if(
PlayerInfo[playerid][pRank] == 4) { rtext "Company Secretary"; }
            else if(
PlayerInfo[playerid][pRank] == 5) { rtext "ABC Manager"; }
            else if(
PlayerInfo[playerid][pRank] == 6) { rtext "Network Producer"; }
            else { 
rtext "Intern Worker"; }
            
            
GetPlayerName(playeridsendernameMAX_PLAYER_NAME);
            
format(stringsizeof(string), "[SAN News] %s %s: %s"rtextsendernameresult);
            
OOCNews(COLOR_NEWSstring);
        }
        else
        {
            
SendClientMessage(playeridCOLOR_GREY"You are not a News Reporter !");
        }
    }
    return 
1;

Reply


Messages In This Thread
/news cmd - by marwan - 22.07.2011, 19:26
Re: /news cmd - by Roko_foko - 22.07.2011, 20:54
Re: /news cmd - by MoroDan - 22.07.2011, 21:14
Re: /news cmd - by marwan - 23.07.2011, 07:24
Re: /news cmd - by marwan - 23.07.2011, 07:37
Re: /news cmd - by marwan - 23.07.2011, 10:02
Re: /news cmd - by MoroDan - 23.07.2011, 10:15
Re: /news cmd - by marwan - 23.07.2011, 11:07
Re: /news cmd - by MoroDan - 23.07.2011, 11:09
Re: /news cmd - by marwan - 23.07.2011, 11:18

Forum Jump:


Users browsing this thread: 1 Guest(s)