Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-11-03 12:36:02 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix(refactor): patching helpers.tryRoute for API routes, some re-org

Browse Source 
Thanks @barisusakli for the tip
This commit is contained in:
Julian Lam
2020-03-30 13:16:29 -04:00
parent d8879d21db
commit d15d9e4492
5 changed files with 36 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
src/routes/write/users.js
View File

@@ -1,9 +1,8 @@
'use strict';
const users = require('../../user');
const middleware = require('../../middleware');
const helpers = require('../../controllers/helpers');
const controllers = require('../../controllers');
const routeHelpers = require('../../routes/helpers');
// Messaging = require.main.require('./src/messaging'),
// apiMiddleware = require('./middleware'),
// errorHandler = require('../../lib/errorHandler'),
@@ -13,16 +12,10 @@ const helpers = require('../../controllers/helpers');
module.exports = function () {
var app = require('express').Router();
const router = require('express').Router();
const setupApiRoute = routeHelpers.setupApiRoute;
app.post('/', middleware.checkRequired.bind(null, ['username']), middleware.authenticate, middleware.isAdmin, async (req, res) => {
try {
const uid = await users.create(req.body);
helpers.formatApiResponse(200, res, await users.getUserData(uid));
} catch (err) {
helpers.formatApiResponse(400, res, err);
}
});
setupApiRoute(router, '/', middleware, [middleware.checkRequired.bind(null, ['username']), middleware.isAdmin], 'post', controllers.write.users.create);
// app.route('/:uid')
// .put(apiMiddleware.requireUser, apiMiddleware.exposeAdmin, function(req, res) {
@@ -170,5 +163,5 @@ module.exports = function () {
// });
// });
return app;
return router;
};