all controllers now use this.createMulter()

app/controllers/admin/content-report.js

@@ -5,9 +5,8 @@
 'use strict';
 
 const express = require('express');
-const multer = require('multer');
 
-const { /*SiteError,*/ SiteController } = require('../../../lib/site-lib');
+const { SiteController } = require('../../../lib/site-lib');
 
 class ContentReportController extends SiteController {
 
@@ -16,7 +15,7 @@ class ContentReportController extends SiteController {
   }
 
   async start ( ) {
-    const upload = multer({ dest: `/tmp/${this.dtp.config.site.domainKey}/upload` });
+    const upload = this.createMulter();
 
     const router = express.Router();
     router.use(async (req, res, next) => {

app/controllers/admin/core-node.js

@@ -16,8 +16,6 @@ class CoreNodeController extends SiteController {
   }
 
   async start ( ) {
-    // const upload = multer({ dest: `/tmp/${this.dtp.config.site.domainKey}/upload` });
-
     const router = express.Router();
     router.use(async (req, res, next) => {
       res.locals.currentView = 'admin';

app/controllers/admin/core-user.js

@@ -16,8 +16,6 @@ class CoreUserController extends SiteController {
   }
 
   async start ( ) {
-    // const upload = multer({ dest: `/tmp/${this.dtp.config.site.domainKey}/upload` });
-
     const router = express.Router();
     router.use(async (req, res, next) => {
       res.locals.currentView = 'admin';

app/controllers/auth.js

@@ -25,7 +25,8 @@ class AuthController extends SiteController {
       coreNode: coreNodeService,
       limiter: limiterService,
     } = this.dtp.services;
-    const upload = multer({ dest: `/tmp/${this.dtp.config.site.domainKey}/uploads/${this.component.slug}` });
+
+    const upload = this.createMulter();
 
     const router = express.Router();
     this.dtp.app.use('/auth', router);

app/controllers/chat.js

@@ -23,7 +23,7 @@ class ChatController extends SiteController {
       session: sessionService,
     } = this.dtp.services;
 
-    const upload = multer({ dest: `/tmp/${this.dtp.config.site.domainKey}/uploads/${this.component.slug}` });
+    const upload = this.createMulter();
 
     const router = express.Router();
     this.dtp.app.use('/chat', router);

app/controllers/image.js

@@ -26,8 +26,7 @@ class ImageController extends SiteController {
     const router = express.Router();
     dtp.app.use('/image', router);
 
-    const imageUpload = multer({
-      dest: `/tmp/${this.dtp.config.site.domainKey}/uploads/${this.component.slug}`,
+    const imageUpload = this.createMulter('uploads', {
       limits: {
         fileSize: 1024 * 1000 * 12,
       },

app/controllers/newsletter.js

@@ -19,7 +19,7 @@ class NewsletterController extends SiteController {
     const { dtp } = this;
     const { limiter: limiterService } = dtp.services;
 
-    const upload = multer({ dest: `/tmp/${this.dtp.config.site.domainKey}/uploads/${module.exports.slug}` });
+    const upload = this.createMulter();
 
     const router = express.Router();
     dtp.app.use('/newsletter', router);

app/controllers/user.js

@@ -24,7 +24,8 @@ class UserController extends SiteController {
       session: sessionService,
     } = dtp.services;
 
-    const upload = multer({ dest: `/tmp/${this.dtp.config.site.domainKey}/uploads/${this.component.slug}` });
+    const upload = this.createMulter();
+
     const router = express.Router();
     dtp.app.use('/user', router);
 

lib/site-controller.js

@@ -36,9 +36,14 @@ class SiteController extends SiteCommon {
     return pagination;
   }
 
-  createMulter (slug) {
+  createMulter (slug, options) {
     slug = slug || 'uploads';
-    return multer({ dest: `/tmp/${this.dtp.config.site.domainKey}/${slug}/${this.component.slug}` });
+
+    options = Object.assign({
+      dest: `/tmp/${this.dtp.config.site.domainKey}/${slug}/${this.component.slug}`
+    }, options || { });
+
+    return multer(options);
   }
 
   createDisplayList (name) {