Skip to content

Commit 5fa338e

Browse files
Return early if method cannot be modeled
1 parent 8529c05 commit 5fa338e

File tree

1 file changed

+31
-6
lines changed
  • extensions/ql-vscode/src/view/data-extensions-editor

1 file changed

+31
-6
lines changed

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

Lines changed: 31 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,36 @@ export const MethodRow = ({
174174
const methodCanBeModeled =
175175
!externalApiUsage.supported ||
176176
(modeledMethod && modeledMethod?.type !== "none");
177+
178+
const externalApiUsageName = (
179+
<span>
180+
{externalApiUsage.packageName}.{externalApiUsage.typeName}.
181+
{externalApiUsage.methodName}
182+
{externalApiUsage.methodParameters}
183+
</span>
184+
);
185+
186+
if (!methodCanBeModeled) {
187+
return (
188+
<VSCodeDataGridRow>
189+
<ApiOrMethodCell gridColumn={1}>
190+
<VSCodeCheckbox />
191+
{externalApiUsageName}
192+
{mode === Mode.Application && (
193+
<UsagesButton onClick={jumpToUsage}>
194+
{externalApiUsage.usages.length}
195+
</UsagesButton>
196+
)}
197+
<ViewLink onClick={jumpToUsage}>View</ViewLink>
198+
</ApiOrMethodCell>
199+
<VSCodeDataGridCell gridColumn={2} />
200+
<VSCodeDataGridCell gridColumn={3} />
201+
<VSCodeDataGridCell gridColumn={4} />
202+
<VSCodeDataGridCell gridColumn={5} />
203+
</VSCodeDataGridRow>
204+
);
205+
}
206+
177207
const showInputCell =
178208
modeledMethod?.type && ["sink", "summary"].includes(modeledMethod?.type);
179209
const showOutputCell =
@@ -188,11 +218,7 @@ export const MethodRow = ({
188218
<VSCodeDataGridRow>
189219
<ApiOrMethodCell gridColumn={1}>
190220
<VSCodeCheckbox />
191-
<span>
192-
{externalApiUsage.packageName}.{externalApiUsage.typeName}.
193-
{externalApiUsage.methodName}
194-
{externalApiUsage.methodParameters}
195-
</span>
221+
{externalApiUsageName}
196222
{mode === Mode.Application && (
197223
<UsagesButton onClick={jumpToUsage}>
198224
{externalApiUsage.usages.length}
@@ -204,7 +230,6 @@ export const MethodRow = ({
204230
<Dropdown
205231
value={modeledMethod?.type ?? "none"}
206232
options={modelTypeOptions}
207-
disabled={!methodCanBeModeled}
208233
onChange={handleTypeInput}
209234
/>
210235
</VSCodeDataGridCell>

0 commit comments

Comments
 (0)