Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

SyntaxError: identifier starts immediately after numeric literal in Firebug

I'm getting that error when I call this javascript function:

function kickUser(id_userChat){
$.post("chatFuncs.php", { action: "kick", id_user: id_userChat });  
}

this "kickUser" function is generated for every user connected to my chat box, like this

$listUsers .= '<img src="imgUsers/'.$DBClass->nomImg($rowUsers['id_user'],$posImg).'" height="'.$heightImg.'" width="'.$widhImg.'"/>
<span class="styleMsg">'.$rowUser['nameUser'].'</span>&nbsp;
<a href="#" class="BtnKick" onClick="kickUser('.$rowUsers['id_user'].')">Kick</a></br>';

and the action "kick" is just an update to my database where I remove the user from my chatUsers table

If I change $rowUsers['id_user'] for $rowUsers['userName'] the error changes to: ReferenceError: 'userName' is not defined (i changed the real name of the user for 'userName' just for this example).

like image 299
Mollo Avatar asked Dec 04 '22 00:12

Mollo


2 Answers

Identifiers in JavaScript can't begin with a number; they must begin with a letter, $ or _.


I'm guessing it's coming from this:

onclick="kick_user('.$rowUsers['id_user'].')">Kick</a>

If you mean to pass a string, then you need to quote the value being passed.

onclick="kick_user(\"'.$rowUsers['id_user'].'\")">Kick</a>

I don't know PHP, so maybe you need different escaping, but this should give you the idea.

like image 109
the system Avatar answered Dec 05 '22 13:12

the system


The resulting JavaScript code will be

kickUser(userName)

…and obviously there is no js variable userName. You want to pass a string instead:

kickUser('userName');

So add the quotes/apostrophes to the output, and don't forget to escape the $rowUsers['userName'] properly. It's quite the same for $rowUsers['id_user'], which seems to have output even an invalid identifier.

like image 23
Bergi Avatar answered Dec 05 '22 12:12

Bergi