Solved

ajax error handling

Posted on 2012-03-11
10
300 Views
Last Modified: 2012-03-19
I have created an ajax call which submits some php code to a mysql db and it works ok.
I now want to add some error handling so if the mysql query fails then the user gets told so.

The code below is how far I have got with it but the array always comes back undefined.

Could someone take a look and perhaps give me a clue why it doesn't work.

Many Thanks

Steve


javascript
function saveCanvas(frm) {
	
    $.blockUI({ message: '<h1><img src="images/busy.gif" /><br>Submitting...<br>Please Wait...</h1>' }); 
&#9;
    var strImageData = canvas.toDataURL();  
&#9;
    $.ajax({
      &#9;type: 'POST',
      &#9;url: 'draw/submitimage.php',
&#9;  &#9;dataType : 'html',
&#9;&#9;data: frm.serialize() + "&imageData="+encodeURIComponent(strImageData),
&#9;&#9;cache: false,
&#9;&#9;async: true,

&#9;&#9;success: function(data) {
&#9;&#9;&#9;if (data.status() == "success") {
&#9;&#9;&#9;&#9;alert("SUCCESS. " + data.message);

&#9;&#9;&#9;} else {
&#9;&#9;&#9;&#9;alert("Oops. " + data.message);
&#9;&#9;&#9;}
&#9;&#9;},
&#9;&#9;
&#9;&#9;error: function(jqXHR, exception) {
            if (jqXHR.status === 0) {
                alert('Not connect.\n Verify Network.');
            } else if (jqXHR.status == 404) {
                alert('Requested page not found. [404]');
            } else if (jqXHR.status == 500) {
                alert('Internal Server Error [500].');
            } else if (exception === 'parsererror') {
                alert('Requested JSON parse failed.');
            } else if (exception === 'timeout') {
                alert('Time out error.');
            } else if (exception === 'abort') {
                alert('Ajax request aborted.');
            } else {
                alert('Uncaught Error.\n' + jqXHR.responseText);
            }
        }
&#9;});
&#9;
&#9;    return false;
};

Open in new window



submit php
<?php require_once '../includes/config.php'; ?>

  <?php
  
  &#9;  $ajaxResponse=array();
  
&#9;  //convert all the posts to variables:
&#9;  $terminalNumber = $_COOKIE['terminalNumber'];
  &#9;  $submissionType = "image";
  &#9;  $name = $_POST['name'];
&#9;  $subject = $_POST['subject'];
&#9;  $message = $_POST['description'];&#9;  
&#9;  $imageData = $_POST['imageData'];
&#9; 
   //Insert the values into the correct database with the right fields
   $result=mysql_query("INSERT INTO messages (dateTime, terminalNumber, submissionType, name, subject, message, imageData)".
      "VALUES (NOW(), '$terminalNumber', '$submissionType', '$name', '$subject', '$message', '$imageData')");&#9;  
&#9; 
&#9;//handle your query.
if ($result) {
     $ajaxResponse['status'] = "success";
     $ajaxResponse['message'] = "Your feedback has been submitted";
} else {
     $ajaxResponse['status'] = "failure";
     $ajaxResponse['message'] = "There was a problem: " . mysql_error();
}

echo json_encode($ajaxResponse);

?>

Open in new window

0
Comment
Question by:sjtinsley83
  • 4
  • 3
10 Comments
 
LVL 108

Expert Comment

by:Ray Paseur
ID: 37707597
In MySQL datetime is a reserved word.  Not sure if that is the problem or not, but it could be a risk.
0
 
LVL 108

Expert Comment

by:Ray Paseur
ID: 37707600
What happens if you remove the AJAX part of it and just make a form that posts directly to submit.php?  If you try that it will help isolate the problem to the server side vs. the client side.

Also, please learn about this function before you get hacked.
http://us3.php.net/manual/en/function.mysql-real-escape-string.php
0
 

Author Comment

by:sjtinsley83
ID: 37707636
Just so you know... even tho the alert that comes back is: Oops undefined it still successfully entering the record into the mysql db.

if (data.status() == "success") {
	alert("SUCCESS. " + data.message);

} else {
	alert("Oops. " + data.message);
}

Open in new window


I could remove the code above and it all works fine. I would just like to inform the user if there was an error.
0
Top 6 Sources for Identifying Threat Actor TTPs

Understanding your enemy is essential. These six sources will help you identify the most popular threat actor tactics, techniques, and procedures (TTPs).

 
LVL 108

Accepted Solution

by:
Ray Paseur earned 500 total points
ID: 37707649
It seems like data.status() is not defined as a method.  How about data.status without the parentheses, so it is a property?
0
 

Author Comment

by:sjtinsley83
ID: 37707666
I have altered the function so that all it does it alert("SUCCESS. " + data.message);
But it still alerts undefined.
Any other ideas?

function saveCanvas(frm) {

    $.blockUI({ message: '<h1><img src="images/busy.gif" /><br>Submitting...<br>Please Wait...</h1>' }); 

    var strImageData = canvas.toDataURL();  
	
    $.ajax({
      	type: 'POST',
      	url: 'draw/submitimage.php',
	  	dataType : 'html',
		data: frm.serialize() + "&imageData="+encodeURIComponent(strImageData),
		cache: false,
		async: true,
		

		success: function(data) {
				alert("SUCCESS. " + data.message);
				window.location.href = "index.php";
		},
		
		error: function(jqXHR, exception) {
            if (jqXHR.status === 0) {
                alert('Not connect.\n Verify Network.');
            } else if (jqXHR.status == 404) {
                alert('Requested page not found. [404]');
            } else if (jqXHR.status == 500) {
                alert('Internal Server Error [500].');
            } else if (exception === 'parsererror') {
                alert('Requested JSON parse failed.');
            } else if (exception === 'timeout') {
                alert('Time out error.');
            } else if (exception === 'abort') {
                alert('Ajax request aborted.');
            } else {
                alert('Uncaught Error.\n' + jqXHR.responseText);
            }
        }
	});
	    return false;
};

Open in new window

0
 

Author Comment

by:sjtinsley83
ID: 37715217
Has anyone else got any ideas why the variable I am setting in the PHP script is showing as undefined in the javascript success statement?
0
 

Author Closing Comment

by:sjtinsley83
ID: 37739981
This was one of the problems.
0

Featured Post

Do You Know the 4 Main Threat Actor Types?

Do you know the main threat actor types? Most attackers fall into one of four categories, each with their own favored tactics, techniques, and procedures.

Join & Write a Comment

Browsers only know CSS so your awesome SASS code needs to be translated into normal CSS. Here I'll try to explain what you should aim for in order to take full advantage of SASS.
This article demonstrates how to create a simple responsive confirmation dialog with Ok and Cancel buttons using HTML, CSS, jQuery and Promises
HTML5 has deprecated a few of the older ways of showing media as well as offering up a new way to create games and animations. Audio, video, and canvas are just a few of the adjustments made between XHTML and HTML5. As we learned in our last micr…
The viewer will learn the basics of jQuery, including how to invoke it on a web page. Reference your jQuery libraries: (CODE) Include your new external js/jQuery file: (CODE) Write your first lines of code to setup your site for jQuery.: (CODE)

705 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

19 Experts available now in Live!

Get 1:1 Help Now