[Su37Erich]

Hello Josh23761
Instead of the codes above, you could use the next code to avoid several 'ifs' that aren't really necessary and don't have the 'bug' of double messages
Both of above are wrong because if you use return in the message the function will end only with a message

pawn Код:

stock SendLocalClientMessage(playerid, Float:dist, color, str[])
{
    new Float:ppos[3];
    GetPlayerPos(playerid, ppos[0], ppos[1], ppos[2]);
    foreach(Player, i)//I'm not sure what foreach are you using because the foreach of ****** that I use is a little different, and it doesn't need use the 'if(IsPlayerConnected(i))'
    {
                if(IsPlayerInRangeOfPoint(i, dist, ppos[0], ppos[1], ppos[2]))
                SendClientMessage(i, color, str);
    }
}