Solved

ajax error handling

Posted on 2012-03-11
10
308 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:Steve Tinsley
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 4
  • 3
10 Comments
 
LVL 110

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 110

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:Steve Tinsley
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
Technology Partners: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 
LVL 110

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:Steve Tinsley
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:Steve Tinsley
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:Steve Tinsley
ID: 37739981
This was one of the problems.
0

Featured Post

Industry Leaders: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Title # Comments Views Activity
How can I split a variable 19 46
Php variable to be sent back 3 35
Jquery syntax 12 30
CSS Selector Chain NOT working for this situation? 3 16
Preface This is the third article about the EE Collaborative Login Project. A Better Website Login System (http://www.experts-exchange.com/A_2902.html) introduces the Login System and shows how to implement a login page. The EE Collaborative Logi…
SASS allows you to treat your CSS code in a more OOP way. Let's have a look on how you can structure your code in order for it to be easily maintained and reused.
Learn how to match and substitute tagged data using PHP regular expressions. Demonstrated on Windows 7, but also applies to other operating systems. Demonstrated technique applies to PHP (all versions) and Firefox, but very similar techniques will w…
The viewer will receive an overview of the basics of CSS showing inline styles. In the head tags set up your style tags: (CODE) Reference the nav tag and set your properties.: (CODE) Set the reference for the UL element and styles for it to ensu…

751 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