kallithea Changeset - 7f2a5a6d732c

Changeset - 7f2a5a6d732c

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

Mads Kiilerich - 11 years ago 2014-07-18 18:44:54
madski@unity3d.com

javascript: don't just alert with 'error' - be slightly more informative

1 file changed with 6 insertions and 6 deletions:

kallithea/public/js/base.js

0 comments (0 inline, 0 general)

kallithea/public/js/base.js

➞

Show inline comments

@@ @@ -409,13 +409,13 @@ var ajaxGET = function(url,success) { @@
     var sUrl = url;
     var callback = {
         success: success,
         failure: function (o) {
             if (o.status != 0) {
                 alert("error: " + o.statusText);
+                alert("Ajax GET error: " + o.statusText);
             };
         },
     };
     var request = YAHOO.util.Connect.asyncRequest('GET', sUrl, callback);
     return request;
@@ @@ -426,13 +426,13 @@ var ajaxPOST = function(url,postData,suc @@
     YUC.initHeader('X-PARTIAL-XHR',true);
     var sUrl = url;
     var callback = {
         success: success,
         failure: function (o) {
             alert("error");
+            alert("Ajax POST error: " + o.statusText);
         },
     };
     var postData = _toQueryString(postData);
     var request = YAHOO.util.Connect.asyncRequest('POST', sUrl, callback, postData);
     return request;
 };
@@ @@ -694,17 +694,17 @@ var injectInlineForm = function(tr){ @@
     var $inlineform = $form.find('.inline-form');
     $form.submit(function(e){
         e.preventDefault();
         if(lineno === undefined){
-            alert('missing line !');
+            alert('Error submitting, line ' + lineno + ' not found.');
             return
+        }
         if(f_path === undefined){
-            alert('missing file path !');
+            alert('Error submitting, file path ' + f_path + ' not found.');
             return
+        }
         var text = $('#text_'+lineno).val();
         if(text == ""){
             return
@@ @@ -1063,13 +1063,13 @@ var deleteNotification = function(url, n @@
     var callback = {
         success:function(o){
             $("#notification_"+notification_id).remove();
             _run_callbacks(callbacks);
         },
         failure:function(o){
-            alert("error");
+            alert("deleteNotification failure");
         },
     };
     var postData = '_method=delete';
     var sUrl = url.replace('__NOTIFICATION_ID__',notification_id);
     var request = YAHOO.util.Connect.asyncRequest('POST', sUrl,
                                                   callback, postData);
@@ @@ -1081,13 +1081,13 @@ var readNotification = function(url, not @@
             var $obj = $("#notification_"+notification_id);
             $obj.removeClass('unread');
             $obj.find('.read-notification').remove();
             _run_callbacks(callbacks);
         },
         failure:function(o){
-            alert("error");
+            alert("readNotification failure");
         },
     };
     var postData = '_method=put';
     var sUrl = url.replace('__NOTIFICATION_ID__',notification_id);
     var request = YAHOO.util.Connect.asyncRequest('POST', sUrl,
                                                   callback, postData);

0 comments (0 inline, 0 general)