Now that this was committed into Drupal 7 #646694: AJAX: Terrible reporting of status 0 response from AJAX request ("HTTP Error 0 has occurred")
It would be really nice if views can provide better messages when views fails than an alert message displaying: "An error occurred at /drupal/admin/build/views/ajax/preview/"
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 661676-better-notifications.patch | 6.59 KB | merlinofchaos |
| #5 | views-661676-5.patch | 5.42 KB | dagmar |
| #2 | views-661676-2.patch | 5.26 KB | dagmar |
| #1 | views-661676.patch | 1.08 KB | dagmar |
| #1 | before.png | 13.23 KB | dagmar |
Comments
Comment #1
dagmarHere is the patch. And the error messages before and after applied the patch.
Comment #2
dagmarHere is the complete patch for all error handlers in ajax.js
Comment #3
merlinofchaos commentedLet's generalize the error function so that we are not duplicating as much code.
Comment #4
dawehnerIt would be cool, if the regexes have some inline comments, what they do. But thats optional the rest is fine.
Comment #5
dagmarLets try this patch
Comment #6
merlinofchaos commentedI'm not actually sure this needs to be ported to D7. Maybe we need to submit something like this to D7 core as an improvement to the AJAX framework.
In any case, I improved the error handling slightly and am going to put this in CTools as well.
It's worth noting that behavior is going to change from server to server. For example, mine is configured in such a way that it does not actually send error messages during PHP crashes, so I added a little note to help point people to the right place in that circumstance.
Commited this patch to 2.x and 3.x.
Comment #7
damien tournoud commentedThere is nothing to port, most of that is gone.