Skip to content

Commit 92b57bf

Browse files
committed
Rename Argument[-1] to Argument[this]
See: github/codeql#12556
1 parent be3efd7 commit 92b57bf

File tree

3 files changed

+7
-11
lines changed

3 files changed

+7
-11
lines changed

extensions/ql-vscode/src/stories/data-extensions-editor/DataExtensionsEditor.stories.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -213,13 +213,13 @@ DataExtensionsEditor.args = {
213213
},
214214
"org.sql2o.Connection#createQuery(String)": {
215215
type: "summary",
216-
input: "Argument[-1]",
216+
input: "Argument[this]",
217217
output: "ReturnValue",
218218
kind: "taint",
219219
},
220220
"org.sql2o.Sql2o#open()": {
221221
type: "summary",
222-
input: "Argument[-1]",
222+
input: "Argument[this]",
223223
output: "ReturnValue",
224224
kind: "taint",
225225
},

extensions/ql-vscode/src/stories/data-extensions-editor/MethodRow.stories.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ MethodRow.args = {
4747
},
4848
modeledMethod: {
4949
type: "summary",
50-
input: "Argument[-1]",
50+
input: "Argument[this]",
5151
output: "ReturnValue",
5252
kind: "taint",
5353
},

extensions/ql-vscode/src/view/data-extensions-editor/MethodRow.tsx

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,8 @@ export const MethodRow = ({
6464
const target = e.target as HTMLSelectElement;
6565

6666
onChange(externalApiUsage, {
67-
// If there are no arguments, we will default to "this", which is Argument[-1]
68-
input: argumentsList.length === 0 ? "Argument[-1]" : "Argument[0]",
67+
// If there are no arguments, we will default to "Argument[this]"
68+
input: argumentsList.length === 0 ? "Argument[this]" : "Argument[0]",
6969
output: "ReturnType",
7070
kind: "value",
7171
...modeledMethod,
@@ -167,9 +167,7 @@ export const MethodRow = ({
167167
{modeledMethod?.type &&
168168
["sink", "summary"].includes(modeledMethod?.type) && (
169169
<Dropdown value={modeledMethod?.input} onInput={handleInputInput}>
170-
<VSCodeOption value="Argument[-1]">
171-
Argument[-1]: this
172-
</VSCodeOption>
170+
<VSCodeOption value="Argument[this]">Argument[this]</VSCodeOption>
173171
{argumentsList.map((argument, index) => (
174172
<VSCodeOption key={argument} value={`Argument[${index}]`}>
175173
Argument[{index}]: {argument}
@@ -183,9 +181,7 @@ export const MethodRow = ({
183181
["source", "summary"].includes(modeledMethod?.type) && (
184182
<Dropdown value={modeledMethod?.output} onInput={handleOutputInput}>
185183
<VSCodeOption value="ReturnValue">ReturnValue</VSCodeOption>
186-
<VSCodeOption value="Argument[-1]">
187-
Argument[-1]: this
188-
</VSCodeOption>
184+
<VSCodeOption value="Argument[this]">Argument[this]</VSCodeOption>
189185
{argumentsList.map((argument, index) => (
190186
<VSCodeOption key={argument} value={`Argument[${index}]`}>
191187
Argument[{index}]: {argument}

0 commit comments

Comments
 (0)