fix(editor): Allow importing the same workflow multiple times (#7458)

Fixes #7457
n8n-io · Oct 23, 2023 · 15e1737 · 15e1737
1 parent cb3c62e
commit 15e1737
Showing 1 changed file with 19 additions and 19 deletions.
diff --git a/packages/editor-ui/src/components/MainHeader/WorkflowDetails.vue b/packages/editor-ui/src/components/MainHeader/WorkflowDetails.vue
@@ -475,27 +475,27 @@ export default defineComponent({
 			cb(saved);
 		},
 		async handleFileImport(): Promise<void> {
-			const reader = new FileReader();
-			reader.onload = (event: ProgressEvent) => {
-				const data = (event.target as FileReader).result;
-
-				let workflowData: IWorkflowDataUpdate;
-				try {
-					workflowData = JSON.parse(data as string);
-				} catch (error) {
-					this.showMessage({
-						title: this.$locale.baseText('mainSidebar.showMessage.handleFileImport.title'),
-						message: this.$locale.baseText('mainSidebar.showMessage.handleFileImport.message'),
-						type: 'error',
-					});
-					return;
-				}
-
-				nodeViewEventBus.emit('importWorkflowData', { data: workflowData });
-			};
-
 			const inputRef = this.$refs.importFile as HTMLInputElement | undefined;
 			if (inputRef?.files && inputRef.files.length !== 0) {
+				const reader = new FileReader();
+				reader.onload = () => {
+					let workflowData: IWorkflowDataUpdate;
+					try {
+						workflowData = JSON.parse(reader.result as string);
+					} catch (error) {
+						this.showMessage({
+							title: this.$locale.baseText('mainSidebar.showMessage.handleFileImport.title'),
+							message: this.$locale.baseText('mainSidebar.showMessage.handleFileImport.message'),
+							type: 'error',
+						});
+						return;
+					} finally {
+						reader.onload = undefined;
+						inputRef.value = null;
+					}
+
+					nodeViewEventBus.emit('importWorkflowData', { data: workflowData });
+				};
 				reader.readAsText(inputRef.files[0]);
 			}
 		},