web routes complete di-ized
This commit is contained in:
parent
7cb9144281
commit
db97f9fd80
2 changed files with 11 additions and 6 deletions
|
|
@ -6,8 +6,14 @@ function buildDependencies() {
|
|||
const middlewares = require('./middlewares');
|
||||
dependencies.middlewares = middlewares;
|
||||
|
||||
const invitesService = require('./services/invitesService');
|
||||
|
||||
const WebRoutesProvider = require('./routes/webRoutes');
|
||||
const webRoutesProvider = new WebRoutesProvider({ express, middlewares });
|
||||
const webRoutesProvider = new WebRoutesProvider({
|
||||
express,
|
||||
middlewares,
|
||||
invitesService,
|
||||
});
|
||||
dependencies.webRoutes = webRoutesProvider.provide();
|
||||
return dependencies;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -1,9 +1,8 @@
|
|||
const invitesService = require('../services/invitesService');
|
||||
|
||||
class WebRoutesProvider {
|
||||
constructor({ express, middlewares }) {
|
||||
constructor({ express, middlewares, invitesService }) {
|
||||
this.router = express.Router();
|
||||
this.middlewares = middlewares;
|
||||
this.invitesService = invitesService;
|
||||
}
|
||||
|
||||
provide() {
|
||||
|
|
@ -34,14 +33,14 @@ class WebRoutesProvider {
|
|||
|
||||
let invite;
|
||||
try {
|
||||
invite = await invitesService.getAppInvite(inviteUuid);
|
||||
invite = await this.invitesService.getAppInvite(inviteUuid);
|
||||
if (!invite) {
|
||||
return res
|
||||
.status(404)
|
||||
.render('error', { message: 'Invite not found.' });
|
||||
}
|
||||
|
||||
if (await invitesService.isAppInviteSpent(inviteUuid)) {
|
||||
if (await this.invitesService.isAppInviteSpent(inviteUuid)) {
|
||||
return res.status(410).render('invite_spent', { invite });
|
||||
}
|
||||
} catch (error) {
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue