Skip to content

Commit 8952874

Browse files
Merge branch 'main' into robertbrignull/ResultTables-Header
2 parents 892f052 + 36f92da commit 8952874

File tree

6 files changed

+40
-42
lines changed

6 files changed

+40
-42
lines changed

extensions/ql-vscode/CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
# CodeQL for Visual Studio Code: Changelog
22

3+
## [UNRELEASED]
4+
35
## 1.8.9 - 3 August 2023
46

57
- Remove "last updated" information and sorting from variant analysis results view. [#2637](https://github.com/github/vscode-codeql/pull/2637)

extensions/ql-vscode/package-lock.json

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

extensions/ql-vscode/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"description": "CodeQL for Visual Studio Code",
55
"author": "GitHub",
66
"private": true,
7-
"version": "1.8.9",
7+
"version": "1.8.10",
88
"publisher": "GitHub",
99
"license": "MIT",
1010
"icon": "media/VS-marketplace-CodeQL-icon.png",
Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,33 @@
1+
import * as React from "react";
2+
import { AlertTable } from "./alert-table";
3+
import { Graph } from "./graph";
4+
import { RawTable } from "./raw-results-table";
5+
import { ResultTableProps } from "./result-table-utils";
6+
7+
export function ResultTable(props: ResultTableProps) {
8+
const { resultSet } = props;
9+
switch (resultSet.t) {
10+
case "RawResultSet":
11+
return <RawTable {...props} resultSet={resultSet} />;
12+
case "InterpretedResultSet": {
13+
const data = resultSet.interpretation.data;
14+
switch (data.t) {
15+
case "SarifInterpretationData": {
16+
const sarifResultSet = {
17+
...resultSet,
18+
interpretation: { ...resultSet.interpretation, data },
19+
};
20+
return <AlertTable {...props} resultSet={sarifResultSet} />;
21+
}
22+
case "GraphInterpretationData": {
23+
return (
24+
<Graph
25+
graphData={data?.dot[props.offset]}
26+
databaseUri={props.databaseUri}
27+
/>
28+
);
29+
}
30+
}
31+
}
32+
}
33+
}

extensions/ql-vscode/src/view/results/result-tables.tsx renamed to extensions/ql-vscode/src/view/results/ResultTables.tsx

Lines changed: 1 addition & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -14,18 +14,15 @@ import {
1414
ParsedResultSets,
1515
IntoResultsViewMsg,
1616
} from "../../common/interface-types";
17-
import { AlertTable } from "./alert-table";
18-
import { Graph } from "./graph";
19-
import { RawTable } from "./raw-results-table";
2017
import {
21-
ResultTableProps,
2218
tableHeaderClassName,
2319
tableHeaderItemClassName,
2420
toggleDiagnosticsClassName,
2521
alertExtrasClassName,
2622
} from "./result-table-utils";
2723
import { vscode } from "../vscode-api";
2824
import { sendTelemetry } from "../common/telemetry";
25+
import { ResultTable } from "./ResultTable";
2926
import { ResultTablesHeader } from "./ResultTablesHeader";
3027

3128
/**
@@ -169,12 +166,6 @@ export class ResultTables extends React.Component<
169166
}
170167
}
171168

172-
untoggleProblemsView() {
173-
this.setState({
174-
problemsViewSelected: false,
175-
});
176-
}
177-
178169
private onTableSelectionChange = (
179170
event: React.ChangeEvent<HTMLSelectElement>,
180171
): void => {
@@ -338,34 +329,6 @@ export class ResultTables extends React.Component<
338329
}
339330
}
340331

341-
function ResultTable(props: ResultTableProps) {
342-
const { resultSet } = props;
343-
switch (resultSet.t) {
344-
case "RawResultSet":
345-
return <RawTable {...props} resultSet={resultSet} />;
346-
case "InterpretedResultSet": {
347-
const data = resultSet.interpretation.data;
348-
switch (data.t) {
349-
case "SarifInterpretationData": {
350-
const sarifResultSet = {
351-
...resultSet,
352-
interpretation: { ...resultSet.interpretation, data },
353-
};
354-
return <AlertTable {...props} resultSet={sarifResultSet} />;
355-
}
356-
case "GraphInterpretationData": {
357-
return (
358-
<Graph
359-
graphData={data?.dot[props.offset]}
360-
databaseUri={props.databaseUri}
361-
/>
362-
);
363-
}
364-
}
365-
}
366-
}
367-
}
368-
369332
function getDefaultResultSet(resultSets: readonly ResultSet[]): string {
370333
return getDefaultResultSetName(
371334
resultSets.map((resultSet) => resultSet.schema.name),

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import {
1515
ResultSet,
1616
} from "../../common/interface-types";
1717
import { EventHandlers as EventHandlerList } from "./event-handler-list";
18-
import { ResultTables } from "./result-tables";
18+
import { ResultTables } from "./ResultTables";
1919

2020
import "./resultsView.css";
2121
import { useCallback, useEffect } from "react";

0 commit comments

Comments
 (0)