[nickdodd25]

Well i take it you have one main timer calling that? Like just SetTimer? If so then either change all your "playerid" in that function over to "i" and get rid of the playerid param so it would be only

pawn Код:

forward BoomboxPlay();
public BoomboxPlay()

or you could set individual timers like SetTimerEx and get rid of the foreach loop and just change all the "i" over to playerid.
Cause how you have it your checking everyones positon then just checking if playerid is in a car (if you only used SetTimer then it only works for id 0).

So what i am saying is either switch to playerid or get rid of player id and just use the loop and make all the playerid's "i".