Skip to content

Commit ca93f0e

Browse files
eysk-ioaeisenberg
authored andcommitted
Add link to language guides for empty query results
1 parent d9ff5bd commit ca93f0e

File tree

3 files changed

+13
-3
lines changed

3 files changed

+13
-3
lines changed

extensions/ql-vscode/src/view/alert-table.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import * as React from 'react';
33
import * as Sarif from 'sarif';
44
import * as Keys from '../pure/result-keys';
55
import * as octicons from './octicons';
6-
import { className, renderLocation, ResultTableProps, zebraStripe, selectableZebraStripe, jumpToLocation, nextSortDirection } from './result-table-utils';
6+
import { className, renderLocation, ResultTableProps, zebraStripe, selectableZebraStripe, jumpToLocation, nextSortDirection, emptyQueryResultsMessage } from './result-table-utils';
77
import { onNavigation, NavigationEvent } from './results';
88
import { PathTableResultSet } from '../pure/interface-types';
99
import {
@@ -79,7 +79,7 @@ export class PathTable extends React.Component<PathTableProps, PathTableState> {
7979
if (this.props.nonemptyRawResults) {
8080
return <span>No Alerts. See <a href='#' onClick={this.props.showRawResults}>raw results</a>.</span>;
8181
} else {
82-
return <span>No Alerts</span>;
82+
return emptyQueryResultsMessage();
8383
}
8484
}
8585

extensions/ql-vscode/src/view/raw-results-table.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import * as React from 'react';
2-
import { ResultTableProps, className } from './result-table-utils';
2+
import { ResultTableProps, className, emptyQueryResultsMessage } from './result-table-utils';
33
import { RAW_RESULTS_LIMIT, RawResultsSortState } from '../pure/interface-types';
44
import { RawTableResultSet } from '../pure/interface-types';
55
import RawTableHeader from './RawTableHeader';
@@ -21,6 +21,10 @@ export class RawTable extends React.Component<RawTableProps, {}> {
2121
const { resultSet, databaseUri } = this.props;
2222

2323
let dataRows = resultSet.rows;
24+
if (dataRows.length === 0) {
25+
return emptyQueryResultsMessage();
26+
}
27+
2428
let numTruncatedResults = 0;
2529
if (dataRows.length > RAW_RESULTS_LIMIT) {
2630
numTruncatedResults = dataRows.length - RAW_RESULTS_LIMIT;

extensions/ql-vscode/src/view/result-table-utils.tsx

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -140,3 +140,9 @@ export function nextSortDirection(direction: SortDirection | undefined, includeU
140140
return assertNever(direction);
141141
}
142142
}
143+
144+
export function emptyQueryResultsMessage(): JSX.Element {
145+
return <span>
146+
This query returned no results. If this isn&apos;t what you&apos;re expecting, and for effective query-writing tips, check out the <a href="https://codeql.github.com/docs/codeql-language-guides/">CodeQL language guides</a>.
147+
</span>;
148+
}

0 commit comments

Comments
 (0)