Remove statusString method

This commit is contained in:
Koen Vlaswinkel
2023-12-21 16:59:33 +01:00
parent 48ff62f474
commit 2ba39647f2
4 changed files with 5 additions and 9 deletions

View File

@@ -125,12 +125,12 @@ export class CompareView extends AbstractWebview<
// only run interpolation if the label is user-defined
// otherwise we will wind up with duplicated rows
name: this.labelProvider.getShortLabel(from),
status: from.completedQuery.statusString,
status: from.completedQuery.message,
time: from.startTime,
},
toQuery: {
name: this.labelProvider.getShortLabel(to),
status: to.completedQuery.statusString,
status: to.completedQuery.message,
time: to.startTime,
},
},

View File

@@ -77,14 +77,14 @@ export class HistoryItemLabelProvider {
private getLocalInterpolateReplacements(
item: LocalQueryInfo,
): InterpolateReplacements {
const { resultCount = 0, statusString = "in progress" } =
const { resultCount = 0, message = "in progress" } =
item.completedQuery || {};
return {
t: item.startTime,
q: item.getQueryName(),
d: item.databaseName,
r: `(${resultCount} results)`,
s: statusString,
s: message,
f: item.getQueryFileName(),
l: this.getLanguageLabel(item),
"%": "%",

View File

@@ -1058,7 +1058,7 @@ export class QueryHistoryManager extends DisposableObject {
.map((item) => ({
label: this.labelProvider.getLabel(item),
description: item.databaseName,
detail: item.completedQuery.statusString,
detail: item.completedQuery.message,
query: item,
}));
if (comparableQueryLabels.length < 1) {

View File

@@ -75,10 +75,6 @@ export class CompletedQueryInfo implements QueryWithResults {
this.resultCount = value;
}
get statusString(): string {
return this.message;
}
getResultsPath(selectedTable: string, useSorted = true): string {
if (!useSorted) {
return this.query.resultsPaths.resultsPath;