recode

Author: JemyCheung

src/upload.js

@@ -5,7 +5,7 @@ import {
   setLocalFileInfo,
   removeLocalFileInfo,
   getLocalFileInfo,
-  isContainFileMimeType,
+  findMimeType,
   sum,
   getDomainFromUrl,
   getPortFromUrl,
@@ -65,14 +65,14 @@ export class UploadManager {
       this.putExtra.fname = this.file.name;
     }
     if (this.putExtra.mimeType && this.putExtra.mimeType.length) {
-      var compareMimeType = isContainFileMimeType(this.file.type, this.putExtra.mimeType);
-      if (compareMimeType == null){
-        let err = new Error("file type doesn't match with what you specify");
-        this.onError(err);
-        return;
-      }else{
-        this.putExtra.mimeType = [compareMimeType];
-      }
+      var compareMimeType = findMimeType(this.file.type, this.putExtra.mimeType);
+      if (compareMimeType == null || compareMimeType == undefined) {
+         let err = new Error("file type doesn't match with what you specify");
+         this.onError(err);
+         return;
+       } else {
+         this.putExtra.mimeType = [compareMimeType];
+       }
     }
     let upload = getUploadUrl(this.config, this.token).then(res => {
       this.uploadUrl = res;

src/utils.js

@@ -264,15 +264,12 @@ function getUpHosts(token) {
   }
 }
 
-
-export function isContainFileMimeType(fileType, mimeType){
-    var rtType = null;
-    mimeType.forEach(elem => {
-        if(fileType == elem){
-            rtType = fileType;
-        }
-    });
-    return rtType;
+export function findMimeType(fileType, mimeType) {
+  return mimeType.find((elem) => {
+    if (fileType == elem) {
+      return elem;
+    }
+  });
 }
 
 export function createObjectURL(file) {