🆕 change form multi handler

config/default.js

@@ -153,5 +153,6 @@ module.exports = {
     dest: 'public/files',
     settings: {},
     public: '/files', // Can include {{dest}} for the dest path
+    temp: 'public/tmp/',
   },
 };

lib/routes/content.js

@@ -6,7 +6,8 @@
  */
 const config = require('config');
 const types = require('punchcard-content-types');
-const multipart = require('connect-multiparty');
+const multer = require('multer');
+const upload = multer({ dest: config.storage.temp || 'public/tmp/' });
 const uuid = require('uuid');
 const _ = require('lodash');
 const isUUID = require('validator/lib/isUUID');
@@ -17,8 +18,6 @@ const workflows = require('../workflows');
 const schedule = require('../schedule');
 const storage = require('../storage');
 
-const multipartMiddleware = multipart();
-
 /*
  * Content Route Resolution
  *
@@ -283,7 +282,7 @@ const routes = application => {
      * @param {object} res - HTTP Response
      * @param {object} next - Express callback
     */
-    app.post(`/${config.content.base}/:type/${config.content.actions.save}`, multipartMiddleware, (req, res, next) => {
+    app.post(`/${config.content.base}/:type/${config.content.actions.save}`, upload.any(), (req, res, next) => {
       const referrer = _.get(req.session, 'referrer') || req.get('Referrer');
       let audits;
       let publishable = false;
@@ -343,7 +342,7 @@ const routes = application => {
 
         // language
         let language = 'us-en';
-        if (req.body.hasOwnProperty('language')) {
+        if (req.body.language) {
           language = req.body.language;
         }
 

lib/storage/put.js

@@ -100,15 +100,15 @@ const fileDJ = file => {
       const f = new PunchcardFile({
         cwd: '/',
         base: '/',
-        path: `/${file.originalFilename}`,
+        path: `/${file.originalname}`,
         contents: data,
       });
 
-      f.field = file.fieldName;
-      f.type = file.type;
-      f.path = `/${f.uuid}${path.extname(file.originalFilename)}`;
-      f.rel = `/${f.uuid}${path.extname(file.originalFilename)}`;
-      f.original = file.originalFilename;
+      f.field = file.fieldname;
+      f.type = file.mimetype;
+      f.path = `/${f.uuid}${path.extname(file.originalname)}`;
+      f.rel = `/${f.uuid}${path.extname(file.originalname)}`;
+      f.original = file.originalname;
 
       res(f);
     });

package.json

@@ -74,6 +74,7 @@
     "moment": "^2.15.0",
     "moment-timezone": "^0.5.4",
     "morgan": "^1.6.1",
+    "multer": "^1.2.1",
     "multiparty": "^4.1.2",
     "node-dir": "^0.1.11",
     "node-schedule": "^1.1.1",

tests/storage.js

@@ -21,13 +21,13 @@ test('Storage - Default', t => {
     Object.keys(files).forEach(file => {
       const result = files[file];
       const expected = input.find(inp => {
-        return inp.fieldName === file;
+        return inp.fieldname === file;
       });
 
       t.true(result.hasOwnProperty('original'), 'Has original filename');
       t.true(result.hasOwnProperty('type'), 'Has a type');
       t.true(result.hasOwnProperty('relative'), 'Has a relative path');
-      t.is(result.type, expected.type, 'Have the same type');
+      t.is(result.type, expected.mimetype, 'Have the same type');
 
       const output = fs.readFileSync(path.join(__dirname, 'public/files', result.relative));
       const original = fs.readFileSync(expected.path);