views:

4933

answers:

4

I am trying to pass a dataString to to an ajax call using JQuery. In the call, I construct the get parameters and then send them to the php page on the receiving end. The trouble is that the data string has ampersands in them and the HTML strict validator is chocking on it.

Here is the code:

$(document).ready(function(){
 $("input#email").focus();
 $('#login_submit').submit(function(){
  var username = $('input#email').val();
  var password = $('input#password').val();
  var remember = $('input#remember').attr("checked");
  var dataString = "email="+username+"&password="+password+"&remember="+remember;
  $.post('login.php', dataString, function(data) {
   if (data == 'Login Succeeded.') {
    location.reload(true);
   } else {
    $("input#email").focus();
    $("#login_msg").html(data).effect("pulsate", {times: 2}, 1000); 
   }
  });   
  return false;
 });
});

and here is an example of the validator message: cannot generate system identifier for general entity "password".

var dataString = "email="+username+"&password="+password+"&remember="+rememb…

(in the validator the "p" after the first ampersand is marked red indicating the point of the failure).

+9  A: 

Try putting your javascript inside a CDATA block like this:

<script type="text/javascript">
<![CDATA[
// content of your Javascript goes here
]]>
</script>

which should make it pass validation. To be extra safe you can add Javascript comments around the CDATA tags to hide them from older browsers who don't understand the CDATA tag:

<script type="text/javascript">
/* <![CDATA[ */
// content of your Javascript goes here
/* ]]> */
</script>
Marc Novakowski
Strelok
That was perfect. The first answer didn't work but the second one did. Thanks for the help!
Mike Farmer
A: 

i would try:

var dataString = "email="+username+"&amp;password="+password+"&amp;remember="+remember;
Luis Melgratti
yeah, I tried that already and no go. Thanks for the help.
Mike Farmer
+1  A: 

Note: before one goes blindly wrapping text in CDATA blocks, be aware that CDATA's purpose is NOT for making invalid characters valid.

See: http://www.flightlab.com/~joe/sgml/cdata.html

BryanH
+1  A: 

"\u0026" works!