Skip to content

Commit 85eb618

Browse files
committed
fix: Trigger button display and resource trigger api error
1 parent 1e82cbf commit 85eb618

File tree

6 files changed

+27
-19
lines changed

6 files changed

+27
-19
lines changed

ui/src/api/system-resource-management/trigger.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ const putResourceTrigger: (
108108
data: TriggerData,
109109
loading?: Ref<boolean>,
110110
) => Promise<Result<any>> = (source_type, source_id, trigger_id, data, loading) => {
111-
return post(
111+
return put(
112112
`${prefix}/${source_type}/${source_id}/trigger/${trigger_id}`,
113113
data,
114114
undefined,

ui/src/views/application/index.vue

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -253,9 +253,7 @@
253253
<el-dropdown-item
254254
@click.stop="openTriggerDrawer(item)"
255255
v-if="
256-
apiType === 'workspace' &&
257-
item.is_publish &&
258-
permissionPrecise.trigger_read(item.id)
256+
apiType === 'workspace' && permissionPrecise.trigger_read(item.id)
259257
"
260258
>
261259
<AppIcon iconName="app-trigger" class="color-secondary"></AppIcon>

ui/src/views/system-resource-management/ApplicationResourceIndex.vue

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -286,7 +286,7 @@
286286
</el-dropdown-item>
287287
<el-dropdown-item
288288
@click.stop="openTriggerDrawer(row)"
289-
v-if="row.is_publish && permissionPrecise.trigger_read()"
289+
v-if="permissionPrecise.trigger_read()"
290290
>
291291
<AppIcon iconName="app-trigger" class="color-secondary"></AppIcon>
292292
{{ $t('views.trigger.title') }}
@@ -355,7 +355,8 @@ const MoreFilledPermission = () => {
355355
return (
356356
permissionPrecise.value.export() ||
357357
permissionPrecise.value.delete() ||
358-
permissionPrecise.value.auth()
358+
permissionPrecise.value.auth() ||
359+
permissionPrecise.value.trigger_read()
359360
)
360361
}
361362

ui/src/views/system-resource-management/ToolResourceIndex.vue

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -332,11 +332,7 @@
332332

333333
<el-dropdown-item
334334
@click.stop="openTriggerDrawer(row)"
335-
v-if="
336-
row.tool_type === 'CUSTOM' &&
337-
permissionPrecise.trigger_read() &&
338-
row.is_active
339-
"
335+
v-if="row.tool_type === 'CUSTOM' && permissionPrecise.trigger_read()"
340336
>
341337
<AppIcon iconName="app-trigger" class="color-secondary"></AppIcon>
342338
{{ $t('views.trigger.title') }}
@@ -474,6 +470,7 @@ const MoreFilledPermission = (row: any) => {
474470
permissionPrecise.value.delete() ||
475471
permissionPrecise.value.auth() ||
476472
permissionPrecise.value.relate_map() ||
473+
permissionPrecise.value.trigger_read() ||
477474
(row.init_field_list?.length > 0 && permissionPrecise.value.edit())
478475
)
479476
}

ui/src/views/trigger/ResourceTriggerDrawer.vue

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -97,13 +97,20 @@ const emit = defineEmits(['refresh'])
9797
9898
const createTrigger = (trigger: any) => {
9999
if (toolId.value) {
100-
return triggerAPI.postResourceTrigger(props.source, toolId.value, trigger)
100+
return loadSharedApi({ type: 'trigger', systemType: apiType.value }).postResourceTrigger(
101+
props.source,
102+
toolId.value,
103+
trigger,
104+
)
101105
}
102106
return Promise.resolve<any>({})
103107
}
104108
const editTrigger = (trigger_id: string, trigger: any) => {
105109
if (toolId.value) {
106-
return triggerAPI.putResourceTrigger(props.source, toolId.value, trigger_id, trigger)
110+
return loadSharedApi({
111+
type: 'trigger',
112+
systemType: apiType.value,
113+
}).putResourceTrigger(props.source, toolId.value, trigger_id, trigger)
107114
}
108115
return Promise.resolve<any>({})
109116
}

ui/src/views/trigger/TriggerDrawer.vue

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -504,6 +504,7 @@ import Result from '@/request/Result'
504504
import { hasPermission } from '@/utils/permission'
505505
import permissionMap from '@/permission'
506506
import { PermissionConst, RoleConst } from '@/utils/permission/data'
507+
import { loadSharedApi } from '@/utils/dynamics-api/shared-api'
507508
508509
const emit = defineEmits(['refresh'])
509510
const props = withDefaults(
@@ -573,9 +574,11 @@ const applicationRefresh = (application_selected: any) => {
573574
application_list
574575
.filter((id) => !existApplicationIds.includes(id))
575576
.map((id) => {
576-
return applicationAPI.getApplicationDetail(id).then((ok) => {
577-
applicationDetailsDict.value[ok.data.id] = ok.data
578-
})
577+
return loadSharedApi({ type: 'application', systemType: apiType.value })
578+
.getApplicationDetail(id)
579+
.then((ok: any) => {
580+
applicationDetailsDict.value[ok.data.id] = ok.data
581+
})
579582
})
580583
const task_source_id_list = form.value.trigger_task
581584
.filter((task: any) => task.source_type === 'APPLICATION')
@@ -613,9 +616,11 @@ const toolRefresh = (tool_selected: any) => {
613616
tool_ids
614617
.filter((id) => !existToolIds.includes(id))
615618
.map((id) => {
616-
toolAPI.getToolById(id).then((ok) => {
617-
toolDetailsDict.value[ok.data.id] = ok.data
618-
})
619+
loadSharedApi({ type: 'tool', systemType: apiType.value })
620+
.getToolById(id)
621+
.then((ok: any) => {
622+
toolDetailsDict.value[ok.data.id] = ok.data
623+
})
619624
})
620625
const task_source_id_list = form.value.trigger_task
621626
.filter((task: any) => task.source_type === 'TOOL')

0 commit comments

Comments
 (0)