fix(mts): fix files with the same name for package export

packages/component-manuscript/src/components/ReviewerReportForm.js

@@ -215,7 +215,7 @@ export default compose(
       changeForm('reviewerReport', field, value)
     },
     addFile: ({ formValues = {}, uploadFile, changeForm, version }) => file => {
-      uploadFile(file, 'review', version.id)
+      uploadFile(file, 'review', version)
         .then(file => {
           const files = formValues.files || []
           const newFiles = [...files, file]

packages/component-manuscript/src/components/SubmitRevision.js

@@ -183,7 +183,7 @@ export default compose(
   ),
   withHandlers({
     addFile: ({ formValues = {}, uploadFile, changeForm, version }) => file => {
-      uploadFile(file, 'responseToReviewers', version.id)
+      uploadFile(file, 'responseToReviewers', version)
         .then(file => {
           const { files } = formValues
           const newFiles = files.responseToReviewers

packages/components-faraday/src/components/Files/Files.js

@@ -145,7 +145,7 @@ export default compose(
       changeForm,
       parentForm = 'wizard',
     }) => type => file => {
-      uploadFile(file, type, version.id)
+      uploadFile(file, type, version)
         .then(file => {
           const newFiles = {
             ...files,

packages/components-faraday/src/redux/files.js

@@ -32,10 +32,11 @@ const uploadSuccess = () => ({
   type: UPLOAD_SUCCESS,
 })
 
-const createFileData = (file, type, fragmentId) => {
+const createFileData = ({ file, type, fragmentId, newName }) => {
   const data = new FormData()
   data.append('fileType', type)
   data.append('fragmentId', fragmentId)
+  data.append('newName', newName)
   data.append('file', file)
 
   return {
@@ -47,6 +48,26 @@ const createFileData = (file, type, fragmentId) => {
   }
 }
 
+const setFileName = (file, { files }) => {
+  let newFilename = file.name
+  const fileCount = Object.values(files)
+    .reduce((acc, f) => [...acc, ...f], [])
+    .map(f => f.originalName)
+    .reduce((count, name) => {
+      if (file.name === name) {
+        count += 1
+      }
+      return count
+    }, 0)
+  if (fileCount > 0) {
+    const ext = file.name.split('.').reverse()[0]
+    const filename = file.name.replace(`.${ext}`, '')
+    newFilename = `${filename} (${fileCount}).${ext}`
+  }
+
+  return newFilename
+}
+
 const removeRequest = () => ({
   type: REMOVE_REQUEST,
 })
@@ -66,9 +87,13 @@ export const getRequestStatus = state => state.files.isFetching
 export const getFileError = state => state.files.error
 
 // thunked actions
-export const uploadFile = (file, type, fragmentId) => dispatch => {
+export const uploadFile = (file, type, fragment) => dispatch => {
   dispatch(uploadRequest(type))
-  return request('/files', createFileData(file, type, fragmentId)).then(
+  const newName = setFileName(file, fragment)
+  return request(
+    '/files',
+    createFileData({ file, type, fragmentId: fragment.id, newName }),
+  ).then(
     r => {
       dispatch(uploadSuccess())
       return r

packages/xpub-faraday/config/validations.js

@@ -41,6 +41,7 @@ module.exports = {
           Joi.object({
             id: Joi.string(),
             name: Joi.string().required(),
+            originalName: Joi.string(),
             type: Joi.string(),
             size: Joi.number(),
             url: Joi.string(),
@@ -51,6 +52,7 @@ module.exports = {
           Joi.object({
             id: Joi.string(),
             name: Joi.string().required(),
+            originalName: Joi.string(),
             type: Joi.string(),
             size: Joi.number(),
             url: Joi.string(),
@@ -61,6 +63,7 @@ module.exports = {
           Joi.object({
             id: Joi.string(),
             name: Joi.string().required(),
+            originalName: Joi.string(),
             type: Joi.string(),
             size: Joi.number(),
             url: Joi.string(),
@@ -71,6 +74,7 @@ module.exports = {
           Joi.object({
             id: Joi.string(),
             name: Joi.string().required(),
+            originalName: Joi.string(),
             type: Joi.string(),
             size: Joi.number(),
             url: Joi.string(),