Add change note

Also, avoid a code scanning warning.

Author: aeisenberg

extensions/ql-vscode/CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - Sorted result set filenames now include a hash of the result set name instead of the full name. [#2955](https://github.com/github/vscode-codeql/pull/2955)
 - The "Install Pack Dependencies" will now only list CodeQL packs located in the workspace. [#2960](https://github.com/github/vscode-codeql/pull/2960)
+- Fix a bug where the "View Query Log" action for a query history item was not working. [#2984](https://github.com/github/vscode-codeql/pull/2984)
 
 ## 1.9.2 - 12 October 2023
 

extensions/ql-vscode/src/query-results.ts

@@ -74,10 +74,7 @@ export class CompletedQueryInfo implements QueryWithResults {
      * Map from result set name to SortedResultSetInfo.
      */
     public sortedResultsInfo: Record<string, SortedResultSetInfo> = {},
-  ) {
-    // The log path is only set when loading from the query history.
-    this.logFileLocation = logFileLocation ?? this.query.logPath;
-  }
+  ) {}
 
   setResultCount(value: number) {
     this.resultCount = value;
@@ -293,7 +290,7 @@ export class LocalQueryInfo {
     this.completedQuery = new CompletedQueryInfo(
       info.query,
       info.result,
-      info.logFileLocation,
+      info.query.logPath,
       info.successful,
       info.message,
       undefined,