@azteam/express 1.2.123 → 1.2.126

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@azteam/express",
-    "version": "1.2.123",
+    "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';
 
 
@@ -14,7 +14,8 @@ class AdminController extends Controller {
                 CREATE: 2,
                 UPDATE: 3,
                 DELETE: 4,
-                RESTORE: 5
+                RESTORE: 5,
+                DESTROY: 6
             }, // system roles
 
             paginateOptions: {},
@@ -28,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),
@@ -45,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),
@@ -58,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),
@@ -76,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),
@@ -95,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),
@@ -118,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),
@@ -132,7 +133,7 @@ class AdminController extends Controller {
 
                     const data = await this.beforeModify(req.body, req.user);
 
-                    item = await this.repository.modifyByUser(req.user.id, item, rdata);
+                    item = await this.repository.modifyByUser(req.user.id, item, data);
 
                     return res.success(item, this.options.guardResponse, this.options.allowResponse);
                 }
@@ -140,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),
@@ -157,7 +158,7 @@ class AdminController extends Controller {
         };
     };
 
-    postRestore() {
+    postRestoreTrash() {
         return {
             path: '/trash/:id',
             method: [
@@ -173,6 +174,23 @@ class AdminController extends Controller {
             ]
         };
     };
+
+    deleteDestroyTrash() {
+        return {
+            path: '/trash/:id',
+            method: [
+                roleMiddleware([this.options.roles.DESTROY], USER_LEVEL.ADMIN),
+                validateMiddleware(REQUEST_TYPE.PARAMS, rulesID),
+                async (req, res) => {
+                    const item = await this.repository.findOneTrashById(req.params.id);
+                    if (!item) return res.error(NOT_EXISTS);
+
+                    await this.repository.destroy(item);
+                    return res.success(true);
+                }
+            ]
+        };
+    };
 }
 
 export default AdminController;

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;