Skip to content

Commit 921cf60

Browse files
Convert handleOpenQueryDirectory to use createSingleSelectionCommand
1 parent 21757ae commit 921cf60

1 file changed

Lines changed: 10 additions & 15 deletions

File tree

extensions/ql-vscode/src/query-history/query-history-manager.ts

Lines changed: 10 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -262,8 +262,10 @@ export class QueryHistoryManager extends DisposableObject {
262262
"query",
263263
),
264264
"codeQLQueryHistory.showQueryText": this.handleShowQueryText.bind(this),
265-
"codeQLQueryHistory.openQueryDirectory":
265+
"codeQLQueryHistory.openQueryDirectory": createSingleSelectionCommand(
266266
this.handleOpenQueryDirectory.bind(this),
267+
"query",
268+
),
267269
"codeQLQueryHistory.cancel": this.handleCancel.bind(this),
268270
"codeQLQueryHistory.exportResults": this.handleExportResults.bind(this),
269271
"codeQLQueryHistory.viewCsvResults": this.handleViewCsvResults.bind(this),
@@ -661,31 +663,24 @@ export class QueryHistoryManager extends DisposableObject {
661663
throw new Error("Unable to get query directory");
662664
}
663665

664-
async handleOpenQueryDirectory(
665-
singleItem: QueryHistoryInfo,
666-
multiSelect: QueryHistoryInfo[] | undefined,
667-
) {
668-
if (!this.assertSingleQuery(multiSelect)) {
669-
return;
670-
}
671-
666+
async handleOpenQueryDirectory(item: QueryHistoryInfo) {
672667
let externalFilePath: string | undefined;
673-
if (singleItem.t === "local") {
674-
if (singleItem.completedQuery) {
668+
if (item.t === "local") {
669+
if (item.completedQuery) {
675670
externalFilePath = join(
676-
singleItem.completedQuery.query.querySaveDir,
671+
item.completedQuery.query.querySaveDir,
677672
"timestamp",
678673
);
679674
}
680-
} else if (singleItem.t === "variant-analysis") {
675+
} else if (item.t === "variant-analysis") {
681676
externalFilePath = join(
682677
this.variantAnalysisManager.getVariantAnalysisStorageLocation(
683-
singleItem.variantAnalysis.id,
678+
item.variantAnalysis.id,
684679
),
685680
"timestamp",
686681
);
687682
} else {
688-
assertNever(singleItem);
683+
assertNever(item);
689684
}
690685

691686
if (externalFilePath) {

0 commit comments

Comments
 (0)