feat(error handler): pass entire context nuxt object instead of only the error nuxt handler

BREAKING CHANGE: errorHandler will only have one parameter (of type Error) with the err.ctx object monkey-patched

Author: ezypeeze

lib/plugins/api.template.js

@@ -75,7 +75,11 @@ function generateAPI(controllerMapping, ctx) {
                     ctx
                 )
                     .then(runSuccessHandler)
-                    .catch(runErrorHandler);
+                    .catch(err => {
+                        err.ctx = ctx;
+
+                        return runErrorHandler(err);
+                    });
             }
         } else if (typeof controllerMapping[key] === 'object') {
             api[key] = generateAPI(controllerMapping[key], ctx);
@@ -86,8 +90,8 @@ function generateAPI(controllerMapping, ctx) {
 }
 
 export default (ctx) => {
-    const {app, req, error} = ctx;
-    const $api = process.server ? req._controllersTree(error) : generateAPI(<%= options.controllers %>, ctx);
+    const {app, req} = ctx;
+    const $api = process.server ? req._controllersTree(ctx) : generateAPI(<%= options.controllers %>, ctx);
     app.$api = $api;
     Vue.prototype.$api = $api;
 };

lib/utility/controllers.js

@@ -152,10 +152,10 @@ function controllerMappingClientSide(directory) {
  * @param directory
  * @param req
  * @param options
- * @param nuxtError
+ * @param ctx
  * @returns {Object}
  */
-function controllerMappingServerSide(directory, req, options, nuxtError) {
+function controllerMappingServerSide(directory, req, options, ctx) {
     const apiMiddleware = options.middleware;
     let SuccessHandler = options.successHandler
         ? require(options.successHandler.replace(options.aliasKey, options.srcDir))
@@ -197,7 +197,8 @@ function controllerMappingServerSide(directory, req, options, nuxtError) {
                     return SuccessHandler && SuccessHandler(result) || Promise.resolve(result);
                 })
                 .catch(function (err) {
-                    return ErrorHandler && ErrorHandler(err, nuxtError) || Promise.reject(err);
+                    err.ctx = ctx;
+                    return ErrorHandler && ErrorHandler(err) || Promise.reject(err);
                 });
         }
     });

tests/fixtures/error_handler.js

@@ -1,6 +1,6 @@
-export default function (err, nuxtError) {
+export default function (err) {
     if (err.message === "nuxtError") {
-        return nuxtError({statusCode: 500, message: err.message});
+        return err.ctx.error({statusCode: 500, message: err.message});
     }
 
     throw err;