naming conventions

app/controllers/admin/newsletter.js

@@ -4,7 +4,6 @@
 
 'use strict';
 
-const DTP_COMPONENT_NAME = 'admin:newsletter';
 const express = require('express');
 
 const { SiteController, SiteError } = require('../../../lib/site-lib');
@@ -12,7 +11,7 @@ const { SiteController, SiteError } = require('../../../lib/site-lib');
 class NewsletterController extends SiteController {
 
   constructor (dtp) {
-    super(dtp, DTP_COMPONENT_NAME);
+    super(dtp, module.exports);
   }
 
   async start ( ) {
@@ -117,7 +116,11 @@ class NewsletterController extends SiteController {
   }
 }
 
-module.exports = async (dtp) => {
-  let controller = new NewsletterController(dtp);
-  return controller;
+module.exports = {
+  name: 'adminNewsletter',
+  slug: 'admin-newsletter',
+  create: async (dtp) => {
+    let controller = new NewsletterController(dtp);
+    return controller;
+  },
 };

app/controllers/comment.js

@@ -4,8 +4,6 @@
 
 'use strict';
 
-const DTP_COMPONENT_NAME = 'comment';
-
 const express = require('express');
 const numeral = require('numeral');
 
@@ -14,7 +12,7 @@ const { SiteController, SiteError } = require('../../lib/site-lib');
 class CommentController extends SiteController {
 
   constructor (dtp) {
-    super(dtp, DTP_COMPONENT_NAME);
+    super(dtp, module.exports);
   }
 
   async start ( ) {
@@ -27,7 +25,7 @@ class CommentController extends SiteController {
     dtp.app.use('/comment', router);
 
     router.use(async (req, res, next) => {
-      res.locals.currentView = DTP_COMPONENT_NAME;
+      res.locals.currentView = module.exports.slug;
       return next();
     });
 

app/controllers/content-report.js

@@ -4,8 +4,6 @@
 
 'use strict';
 
-const DTP_COMPONENT_NAME = 'content-report';
-
 const express = require('express');
 const multer = require('multer');
 
@@ -14,7 +12,7 @@ const { SiteController } = require('../../lib/site-lib');
 class ContentReportController extends SiteController {
 
   constructor (dtp) {
-    super(dtp, DTP_COMPONENT_NAME);
+    super(dtp, module.exports);
   }
 
   async start ( ) {

app/controllers/newsletter.js

@@ -4,8 +4,6 @@
 
 'use strict';
 
-const DTP_COMPONENT_NAME = 'newsletter';
-
 const express = require('express');
 const multer = require('multer');
 
@@ -14,20 +12,20 @@ const { SiteController } = require('../../lib/site-lib');
 class NewsletterController extends SiteController {
 
   constructor (dtp) {
-    super(dtp, DTP_COMPONENT_NAME);
+    super(dtp, module.exports);
   }
 
   async start ( ) {
     const { dtp } = this;
     const { limiter: limiterService } = dtp.services;
 
-    const upload = multer({ dest: `/tmp/${this.dtp.config.site.domainKey}/uploads/${DTP_COMPONENT_NAME}` });
+    const upload = multer({ dest: `/tmp/${this.dtp.config.site.domainKey}/uploads/${module.exports.slug}` });
 
     const router = express.Router();
     dtp.app.use('/newsletter', router);
 
     router.use(async (req, res, next) => {
-      res.locals.currentView = DTP_COMPONENT_NAME;
+      res.locals.currentView = module.exports.slug;
       return next();
     });
 

app/controllers/page.js

@@ -4,8 +4,6 @@
 
 'use strict';
 
-const DTP_COMPONENT_NAME = 'page';
-
 const express = require('express');
 
 const { SiteController, SiteError } = require('../../lib/site-lib');
@@ -13,7 +11,7 @@ const { SiteController, SiteError } = require('../../lib/site-lib');
 class PageController extends SiteController {
 
   constructor (dtp) {
-    super(dtp, DTP_COMPONENT_NAME);
+    super(dtp, module.exports);
   }
 
   async start ( ) {

app/controllers/post.js

@@ -4,8 +4,6 @@
 
 'use strict';
 
-const DTP_COMPONENT_NAME = 'post';
-
 const express = require('express');
 const multer = require('multer');
 
@@ -14,7 +12,7 @@ const { SiteController, SiteError } = require('../../lib/site-lib');
 class PostController extends SiteController {
 
   constructor (dtp) {
-    super(dtp, DTP_COMPONENT_NAME);
+    super(dtp, module.exports);
   }
 
   async start ( ) {