fix(fix-fmt-cmd): Fix format command that would use stale query

spotify · Dec 11, 2023 · 0632241 · 0632241
1 parent ee2b3b5
commit 0632241
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/main/js/components/editor/editor.js b/src/main/js/components/editor/editor.js
@@ -35,13 +35,13 @@ function Editor({query, setQuery, handleRunQuery, handleFormatQuery}) {
             if (handleFormatQuery) {
                 commands.push({
                     name: 'format query',
-                    exec: handleFormatQuery,
+                    exec: () => handleFormatQuery(query),
                     bindKey: {mac: "Command-Option-L", win: "Ctrl-Alt-L"}
                 })
             }
             editor.commands.addCommands(commands)
         }
-    }, [editorRef?.current?.editor])
+    }, [editorRef?.current?.editor, query])
 
     return (<AceEditor
         ref={editorRef}

diff --git a/src/main/js/routes/schema/query.js b/src/main/js/routes/schema/query.js
@@ -16,7 +16,7 @@ function Query({containerUrl, schema}) {
         runQuery()
     }
 
-    function prettifyJsonQuery() {
+    function prettifyJsonQuery(query) {
         try {
             setQuery(JSON.stringify(JSON.parse(query), null, 2))
         } catch (_) {
@@ -69,7 +69,7 @@ function Query({containerUrl, schema}) {
                 data-tooltip-id="vispana-tooltip"
                 data-tooltip-content="Format Query (Cmd+Opt+L)"
                 data-tooltip-place="top"
-                onClick={prettifyJsonQuery}>
+                onClick={(e) => prettifyJsonQuery(query)}>
                 <i className="fas fa-code "/>
             </button>
             <button type="button"