Question

Interesting. I have 2 links calling 2 JS functions – the first works , the second – doesn’t. It only calls function addns() when the argument is empty. Otherwise nope…

PHP:

 while ($row_mem = mysqli_fetch_array($mem)) { 
 $membs[] = $row_mem['username'];
 $membs_id[] = $row_mem['user_id'];

 }

 }  

 //FIRST FOREACH CALL PROPERLY
 foreach($membs_id as $val_id) {
 echo"<a href='javascript:addms($val_id)'><img src='$pcheck' width='66' height='68'
 border='1' class='currmem'/></a>&nbsp;";
 }


//THIS ONE DOESN'T 
foreach($membs as $mes_id) {
echo"<a href='javascript:addns($mes_id)'>$mes_id</a>&nbsp;";
}

JS:

 function addms(msid) { 
 var addm = msid;
 alert(addm);
 }

function addns(nsid) { 
var addn = nsid;

alert(addn);




}

I cannot see any error – thanks for comments !

Answer 1

Update to match new answer:

you need to put quotes around your variables. ($val_id) probably works because you’re retrieving an id which I’m guessing is an integer, so it’s a valid JavaScript literal.

($mes_id) probably doesn’t work because you’re getting strings back but not wrapped with quotes, so they aren’t valid JavaScript literals.

so usernames will be joe and shmoe, javascript becoems
addns(joe) and addns(shmoe), which is probably not what you want. You want addns("joe") and addns("shmoe").

Also note once you fix this is an XSS vulnerability if users can choose their username.