fix: avoid duplicating the code to accept files

Author: iOvergaard

packages/uui-file-dropzone/lib/uui-file-dropzone.element.ts

@@ -114,11 +114,11 @@ export class UUIFileDropzoneElement extends LabelMixin('', LitElement) {
       queue.push(dataTransferItemList[i].webkitGetAsEntry());
     }
 
+    const acceptList: string[] = [];
+    const wildcards: string[] = [];
+
     // if the accept filer is set
     if (this.accept) {
-      const acceptList: string[] = [];
-      const wildcards: string[] = [];
-
       // Create the arrays defined above
       this.accept.split(',').forEach(item => {
         if (item.includes('*')) {
@@ -127,6 +127,7 @@ export class UUIFileDropzoneElement extends LabelMixin('', LitElement) {
           acceptList.push(item.trim().toLowerCase());
         }
       });
+    }
 
       while (queue.length > 0) {
         const entry: FileSystemFileEntry = queue.shift()!;
@@ -189,27 +190,22 @@ export class UUIFileDropzoneElement extends LabelMixin('', LitElement) {
     }
   }
 
-  private async _getFile(fileEntry: FileSystemFileEntry): Promise<File> {
-    return await new Promise<File>((resolve, reject) =>
-      fileEntry.file(resolve, reject)
-    );
-  }
+  private _isAccepted(acceptList: string[], wildcards: string[], file: File) {
+    if (acceptList.length === 0 && wildcards.length === 0) {
+      return true;
+    }
 
-  private async _isAccepted(
-    acceptList: string[],
-    wildcards: string[],
-    entry: FileSystemFileEntry
-  ) {
-    const file = await this._getFile(entry);
     const fileType = file.type.toLowerCase();
     const fileExtension = '.' + file.name.split('.')[1].toLowerCase();
 
     if (acceptList.includes(fileExtension)) {
       return true;
     }
+
     if (acceptList.includes(fileType)) {
       return true;
     }
+
     if (wildcards.some(wildcard => fileType.startsWith(wildcard))) {
       return true;
     }