Merge pull request #275 from codefori/fix/no_history_for_non_open

worksofliam · web-flow · commit ca152ab2b83f · 2024-09-16T09:50:33.000-04:00
Stop appending to history for statements run just for results
diff --git a/src/views/results/index.ts b/src/views/results/index.ts
@@ -314,6 +314,11 @@ async function runHandler(options?: StatementInfo) {
           }
         }
 
+        // If we the API is called with no open, then don't add it to history
+        if (statementDetail.open === false) {
+          statementDetail.history = false;
+        }
+
         if ((statementDetail.qualifier === `statement` || statementDetail.qualifier === `explain`) && statementDetail.history !== false) {
           vscode.commands.executeCommand(`vscode-db2i.queryHistory.prepend`, statementDetail.content);
         }

Original file line number	Diff line number	Diff line change
`@@ -314,6 +314,11 @@ async function runHandler(options?: StatementInfo) {`
`314`	`314`	`}`
`315`	`315`	`}`
`316`	`316`
	`317`	`+ // If we the API is called with no open, then don't add it to history`
	`318`	`+ if (statementDetail.open === false) {`
	`319`	`+ statementDetail.history = false;`
	`320`	`+ }`
	`321`	`+`
`317`	`322`	if ((statementDetail.qualifier === `statement` \|\| statementDetail.qualifier === `explain`) && statementDetail.history !== false) {
`318`	`323`	vscode.commands.executeCommand(`vscode-db2i.queryHistory.prepend`, statementDetail.content);
`319`	`324`	`}`