Skip to content

Commit 94d3bf3

Browse files
committed
fix: add null checks before filtering tool IDs in ApplicationSetting.vue
1 parent 589a165 commit 94d3bf3

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

ui/src/views/application/ApplicationSetting.vue

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -789,11 +789,15 @@ const openParamSettingDialog = () => {
789789
}
790790
791791
function removeTool(id: any) {
792-
applicationForm.value.tool_ids = applicationForm.value.tool_ids.filter((v: any) => v !== id)
792+
if (applicationForm.value.tool_ids) {
793+
applicationForm.value.tool_ids = applicationForm.value.tool_ids.filter((v: any) => v !== id)
794+
}
793795
}
794796
795797
function removeMcpTool(id: any) {
796-
applicationForm.value.mcp_tool_ids = applicationForm.value.mcp_tool_ids.filter((v: any) => v !== id)
798+
if (applicationForm.value.mcp_tool_ids) {
799+
applicationForm.value.mcp_tool_ids = applicationForm.value.mcp_tool_ids.filter((v: any) => v !== id)
800+
}
797801
}
798802
799803
const mcpServersDialogRef = ref()
@@ -828,7 +832,7 @@ function getToolSelectOptions() {
828832
? {
829833
scope: 'WORKSPACE',
830834
tool_type: 'CUSTOM',
831-
workspace_id: application.value?.workspace_id,
835+
workspace_id: applicationForm.value?.workspace_id,
832836
}
833837
: {
834838
scope: 'WORKSPACE',
@@ -851,7 +855,7 @@ function getMcpToolSelectOptions() {
851855
? {
852856
scope: 'WORKSPACE',
853857
tool_type: 'MCP',
854-
workspace_id: application.value?.workspace_id,
858+
workspace_id: applicationForm.value?.workspace_id,
855859
}
856860
: {
857861
scope: 'WORKSPACE',

0 commit comments

Comments
 (0)