fix: handle createQuery throwing an exception in the TransformModal

josdejong · Sep 25, 2024 · 4b1f3f6 · 4b1f3f6
1 parent 5dc2262
commit 4b1f3f6
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 4 deletions.
diff --git a/src/lib/components/modals/TransformModal.scss b/src/lib/components/modals/TransformModal.scss
@@ -47,6 +47,10 @@
         }
       }
 
+      .query-error {
+        color: $error-color;
+      }
+
       textarea.jse-query {
         flex: 1;
         outline: none;

diff --git a/src/lib/components/modals/TransformModal.svelte b/src/lib/components/modals/TransformModal.svelte
@@ -83,6 +83,7 @@
           state.queryOptions || {}
         )
   let isManual = state.isManual || false
+  let queryError: string | undefined = undefined
 
   let previewError: string | undefined = undefined
   let previewContent: Content = { text: '' }
@@ -96,11 +97,17 @@
   }
 
   function updateQueryByWizard(newQueryOptions: QueryLanguageOptions) {
-    queryOptions = newQueryOptions
-    query = getSelectedQueryLanguage(queryLanguageId).createQuery(selectedJson, newQueryOptions)
-    isManual = false
+    try {
+      queryOptions = newQueryOptions
+
+      query = getSelectedQueryLanguage(queryLanguageId).createQuery(selectedJson, newQueryOptions)
+      queryError = undefined
+      isManual = false
 
-    debug('updateQueryByWizard', { queryOptions, query, isManual })
+      debug('updateQueryByWizard', { queryOptions, query, isManual })
+    } catch (err) {
+      queryError = String(err)
+    }
   }
 
   function handleChangeQuery(event: Event) {
@@ -271,6 +278,11 @@
                   json={selectedJson}
                   onChange={updateQueryByWizard}
                 />
+                {#if queryError}
+                  <div class="query-error">
+                    {queryError}
+                  </div>
+                {/if}
               {:else}
                 (Only available for arrays, not for objects)
               {/if}