refactoring

libs/remix-ui/editor/src/lib/providers/codeActionProvider.ts

@@ -104,51 +104,34 @@ export class RemixCodeActionProvider implements monaco.languages.CodeActionProvi
           }
         } else if (fix && nodeAtPosition && fix.nodeType !== nodeAtPosition.nodeType) return
 
-        if (Array.isArray(fix)) {
+        if (Array.isArray(fix)) 
-          for (const element of fix) {
+          for (const element of fix)
-            actions.push({
+            this.addQuickFix(actions, error, model.uri, {title: element.title, range: element.range || error, text: msg || element.message})
-              title: element.title,
+        else this.addQuickFix(actions, error, model.uri, {title: fix.title, range: fix.range || error, text: msg || fix.message})
-              diagnostics: [error],
-              kind: "quickfix",
-              edit: {
-                edits: [
-                  {
-                    resource: model.uri,
-                    edit: {
-                      range: element.range || error,
-                      text: msg || element.message
       }
     }
-                ]
+
-              },
+    return {
-              isPreferred: true
+      actions: actions,
-            })
+      dispose: () => {}
     }
-        } else
+  }
+
+  addQuickFix(actions, error, uri, fixDetails) {
+    const {title, range, text} = fixDetails
     actions.push({
-            title: fix.title,
+      title,
       diagnostics: [error],
       kind: "quickfix",
       edit: {
         edits: [
           {
-                  resource: model.uri,
+            resource: uri,
-                  edit: {
+            edit: { range, text }
-                    range: fix.range || error,
-                    text: msg || fix.message
-                  }
           }
         ]
       },
       isPreferred: true
     })
-
-      }
-    }
-
-    return {
-      actions: actions,
-      dispose: () => {}
-    }
   }
 }