aspnetboilerplate#860 Should handle error status codes for abp.ajax.

Author: hikalkan

src/Abp.Web.Common/Web/Api/ProxyScripting/Generators/JQuery/JQueryProxyScriptGenerator.cs

@@ -18,8 +18,8 @@ public string CreateScript(ApplicationApiDescriptionModel model)
 
             script.AppendLine("/* This file is automatically generated by ABP framework to use MVC Controllers from javascript. */");
             script.AppendLine();
-            script.AppendLine("var abp = abp | {};");
-            script.AppendLine("abp.services = abp.services | {};");
+            script.AppendLine("var abp = abp || {};");
+            script.AppendLine("abp.services = abp.services || {};");
 
             foreach (var module in model.Modules.Values)
             {
@@ -35,7 +35,7 @@ private void AddModuleScript(StringBuilder script, ModuleApiDescriptionModel mod
             script.AppendLine($"// module '{module.Name.ToCamelCase()}'");
             script.AppendLine("(function(){");
             script.AppendLine();
-            script.AppendLine($"  abp.services.{module.Name.ToCamelCase()} = abp.services.{module.Name.ToCamelCase()} | {{}};");
+            script.AppendLine($"  abp.services.{module.Name.ToCamelCase()} = abp.services.{module.Name.ToCamelCase()} || {{}};");
 
             foreach (var controller in module.Controllers.Values)
             {
@@ -53,7 +53,7 @@ private void AddControllerScript(StringBuilder script, ModuleApiDescriptionModel
             script.AppendLine("  (function(){");
             script.AppendLine();
 
-            script.AppendLine($"    abp.services.{module.Name.ToCamelCase()}.{controller.Name.ToCamelCase()} = abp.services.{module.Name.ToCamelCase()}.{controller.Name.ToCamelCase()} | {{}};");
+            script.AppendLine($"    abp.services.{module.Name.ToCamelCase()}.{controller.Name.ToCamelCase()} = abp.services.{module.Name.ToCamelCase()}.{controller.Name.ToCamelCase()} || {{}};");
 
             foreach (var action in controller.Actions.Values)
             {

src/Abp.Web.Common/Web/Api/ProxyScripting/Generators/ProxyScriptingJsFuncHelper.cs

@@ -9,7 +9,7 @@ namespace Abp.Web.Api.ProxyScripting.Generators
 {
     internal static class ProxyScriptingJsFuncHelper
     {
-        private const string ValidJsVariableNameChars = "abcdefghijklmnopqrstuxwvyxABCDEFGHIJKLMNORQRSTUXWVYZ0123456789_";
+        private const string ValidJsVariableNameChars = "abcdefghijklmnopqrstuxwvyxABCDEFGHIJKLMNOPQRSTUXWVYZ0123456789_";
 
         public static string NormalizeJsVariableName(string name, string additionalChars = "")
         {

src/Abp.Web.Common/Web/Models/AjaxResponseOfTResult.cs

@@ -30,6 +30,11 @@ public class AjaxResponse<TResult>
         /// </summary>
         public bool UnAuthorizedRequest { get; set; }
 
+        /// <summary>
+        /// A special signature for AJAX responses. It's used in the client to detect if this is a response wrapped by ABP.
+        /// </summary>
+        public bool __abp { get; } = true;
+
         /// <summary>
         /// Creates an <see cref="AjaxResponse"/> object with <see cref="Result"/> specified.
         /// <see cref="Success"/> is set as true.

src/Abp.Web.Resources/Abp/Framework/scripts/libs/abp.jquery.js

@@ -19,9 +19,19 @@
         return $.Deferred(function ($dfd) {
             $.ajax(options)
                 .done(function (data) {
-                    abp.ajax.handleResponse(data, userOptions, $dfd);
-                }).fail(function () {
-                    $dfd.reject.apply(this, arguments);
+                    if (data.__abp) {
+                        abp.ajax.handleResponse(data, userOptions, $dfd);
+                    } else {
+                        $dfd.resolve(data);
+                        userOptions.success && userOptions.success(data);
+                    }
+                }).fail(function (err) {
+                    if (err.responseJSON && err.responseJSON.__abp) {
+                        abp.ajax.handleResponse(err.responseJSON, userOptions, $dfd);
+                    } else {
+                        $dfd.reject.apply(this, arguments);
+                        userOptions.error && userOptions.error.apply(this, arguments);
+                    }
                 });
         });
     };