@azteam/express 1.2.125 → 1.2.126

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@azteam/express",
-    "version": "1.2.125",
+    "version": "1.2.126",
     "description": "",
     "main": "src/index.js",
     "repository": {

package/src/AdminController.js

@@ -1,8 +1,8 @@
-import { NOT_EXISTS } from '@azteam/error';
+import {NOT_EXISTS} from '@azteam/error';
 
-import { REQUEST_TYPE, USER_LEVEL } from './constant';
-import { paginateMiddleware, roleMiddleware, validateMiddleware } from './middleware';
-import { rulesID } from './validate';
+import {REQUEST_TYPE, USER_LEVEL} from './constant';
+import {paginateMiddleware, roleMiddleware, validateMiddleware} from './middleware';
+import {rulesID} from './validate';
 import Controller from './Controller';
 
 
@@ -29,9 +29,9 @@ class AdminController extends Controller {
         });
     }
 
-    getPaginate() {
+    getPaginatePublic() {
         return {
-            path: '/public',
+            path: '/',
             method: [
                 roleMiddleware([this.options.roles.READ], USER_LEVEL.ADMIN),
                 paginateMiddleware(this.options.paginateOptions),
@@ -46,7 +46,7 @@ class AdminController extends Controller {
 
     getPaginateTrash() {
         return {
-            path: '/trash',
+            path: '/',
             method: [
                 roleMiddleware([this.options.roles.READ], USER_LEVEL.ADMIN),
                 paginateMiddleware(this.options.paginateOptions),
@@ -59,9 +59,9 @@ class AdminController extends Controller {
         };
     };
 
-    getOne() {
+    getOnePubic() {
         return {
-            path: '/public/:id',
+            path: '/:id',
             method: [
                 roleMiddleware([this.options.roles.READ], USER_LEVEL.ADMIN),
                 validateMiddleware(REQUEST_TYPE.PARAMS, rulesID),
@@ -77,7 +77,7 @@ class AdminController extends Controller {
 
     getOneTrash() {
         return {
-            path: '/trash/:id',
+            path: '/:id',
             method: [
                 roleMiddleware([this.options.roles.READ], USER_LEVEL.ADMIN),
                 validateMiddleware(REQUEST_TYPE.PARAMS, rulesID),
@@ -96,9 +96,9 @@ class AdminController extends Controller {
         return data;
     };
 
-    postCreate() {
+    postCreatePublic() {
         return {
-            path: '/public',
+            path: '/',
             method: [
                 roleMiddleware([this.options.roles.CREATE], USER_LEVEL.ADMIN),
                 validateMiddleware(REQUEST_TYPE.BODY, this.options.rulesCreate),
@@ -119,9 +119,9 @@ class AdminController extends Controller {
         return data;
     };
 
-    putModify() {
+    putModifyPublic() {
         return {
-            path: '/public/:id',
+            path: '/:id',
             method: [
                 roleMiddleware([this.options.roles.UPDATE], USER_LEVEL.ADMIN),
                 validateMiddleware(REQUEST_TYPE.PARAMS, rulesID),
@@ -141,9 +141,9 @@ class AdminController extends Controller {
         };
     };
 
-    delete() {
+    deletePublic() {
         return {
-            path: '/public/:id',
+            path: '/:id',
             method: [
                 roleMiddleware([this.options.roles.DELETE], USER_LEVEL.ADMIN),
                 validateMiddleware(REQUEST_TYPE.PARAMS, rulesID),
@@ -158,7 +158,7 @@ class AdminController extends Controller {
         };
     };
 
-    postRestore() {
+    postRestoreTrash() {
         return {
             path: '/trash/:id',
             method: [
@@ -175,9 +175,9 @@ class AdminController extends Controller {
         };
     };
 
-    deleteDestroy() {
+    deleteDestroyTrash() {
         return {
-            path: '/destroy/:id',
+            path: '/trash/:id',
             method: [
                 roleMiddleware([this.options.roles.DESTROY], USER_LEVEL.ADMIN),
                 validateMiddleware(REQUEST_TYPE.PARAMS, rulesID),

package/src/ApiServer.js

@@ -1,4 +1,5 @@
 import fs from 'fs';
+import path from 'path';
 import http from 'http';
 import express from 'express';
 import helmet from 'helmet';
@@ -229,33 +230,31 @@ class ApiServer {
             });
 
             const msg = [];
-            _.map(this.controllers, (obj) => {
+            _.map(this.controllers, (data) => {
+                const controller = data.controller;
+                const controllerName = data.name;
+                const controllerVersion = data.version;
 
-                const {name, controller, version} = obj;
+                const listPublicRouter = controller.publicRouter();
 
-                const listPublicMethod = obj.controller.publicMethod();
+                _.map(listPublicRouter, (method) => {
+                    const {name, type} = method;
 
-                _.map(listPublicMethod, (methodName) => {
+                    const router = controller[name]();
 
-                    const matches = methodName.match(/get|post|put|patch|delete/);
-                    if (matches) {
-                        const type = matches[0];
-                        const item = controller[methodName]();
+                    router.path = `/${method.path}/${router.path}`;
+                    router.path = controller.pathName ? `/${controller.pathName}${router.path}` : router.path;
+                    router.path = controllerVersion.startsWith('v') ? `/${controllerVersion}${router.path}` : router.path;
 
-                        item.path = controller.pathName ? `/${controller.pathName}${item.path}` : item.path;
-
-                        item.path = version.startsWith('v') ? `/${version}${item.path}` : item.path;
-
-                        msg.push({
-                            controller: name,
-                            version,
-                            type,
-                            method: methodName,
-                            path: item.path
-                        });
+                    msg.push({
+                        controller: controllerName,
+                        version: controllerVersion,
+                        type,
+                        method: name,
+                        path: path.resolve(router.path)
+                    });
 
-                        app[type](item.path, ...item.method);
-                    }
+                    app[type](router.path, ...router.method);
                 });
             });
 

package/src/Controller.js

@@ -8,7 +8,7 @@ class Controller {
         };
     }
 
-    publicMethod() {
+    publicRouter() {
         let child = this;
         let result = [];
 
@@ -16,11 +16,23 @@ class Controller {
 
             const data = Object.getOwnPropertyNames(Object.getPrototypeOf(child));
 
-            result = [
-                ...result,
-                ...data.filter((methodName) => methodName.match(/^get|post|put|patch|delete/))
-            ]
+            data.map((methodName) => {
+                const matches = methodName.match(/get|post|put|patch|delete/);
+                if (matches) {
+                    let path = '/';
+                    if (methodName.match(/Public/)) {
+                        path += 'public';
+                    } else if (methodName.match(/Trash/)) {
+                        path += 'trash';
+                    }
 
+                    result.push({
+                        type: matches[0],
+                        name: methodName,
+                        path
+                    });
+                }
+            });
             child = child.__proto__;
         }
         return result;