Skip to content

Commit 98da123

Browse files
committed
Add evaluation results to query object
Ensures better backwards compatibility with legacy query objects.
1 parent db0fbaa commit 98da123

3 files changed

Lines changed: 5 additions & 3 deletions

File tree

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ export class CompletedQueryInfo implements QueryWithResults {
8484
this.query = evaluation.query;
8585
this.logFileLocation = evaluation.logFileLocation;
8686
if (evaluation.result) {
87-
this.result = evaluation.result;
87+
this.result = evaluation.result as legacyMessages.EvaluationResult;
8888
}
8989
this.message = evaluation.message;
9090
this.sucessful = evaluation.sucessful;

extensions/ql-vscode/src/query-server/run-queries.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ export async function compileAndRunQueryAgainstDatabase(
8181
target,
8282
};
8383
await query.createTimestampFile();
84-
let result;
84+
let result: messages.RunQueryResult | undefined;
8585
try {
8686
result = await qs.sendRequest(messages.runQuery, queryToRun, token, progress);
8787
if (qs.config.customLogDirectory) {
@@ -126,6 +126,7 @@ export async function compileAndRunQueryAgainstDatabase(
126126

127127
return {
128128
query,
129+
result,
129130
message,
130131
sucessful: result.resultType === messages.QueryResultType.SUCCESS,
131132
dispose: () => {

extensions/ql-vscode/src/run-queries-shared.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import * as messages from './pure/messages-shared';
22
import * as legacyMessages from './pure/legacy-messages';
3+
import * as newMessages from './pure/new-messages';
34
import { DatabaseInfo, QueryMetadata } from './pure/interface-types';
45
import * as path from 'path';
56
import { createTimestampFile, showAndLogWarningMessage } from './helpers';
@@ -375,7 +376,7 @@ export interface QueryWithResults {
375376
readonly dispose: () => void;
376377
readonly sucessful?: boolean;
377378
readonly message?: string;
378-
readonly result?: legacyMessages.EvaluationResult
379+
readonly result?: legacyMessages.EvaluationResult | newMessages.RunQueryResult
379380
}
380381

381382

0 commit comments

Comments
 (0)