zhuchaowe/maxkb
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: update permission checks to remove row.id dependency and add workspace_id to knowledge retrieval

Browse Source
This commit is contained in:
CaptainB 2025-07-30 09:56:33 +08:00
parent 03fbba0952
commit a3a124d581
2 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
apps/knowledge/serializers/problem.py
View File

@ -245,6 +245,7 @@ class ProblemSerializers(serializers.Serializer):
raise AppApiException(500, _('Knowledge id does not exist')) raise AppApiException(500, _('Knowledge id does not exist'))
def get_query_set(self): def get_query_set(self):
self.is_valid()
query_set = QuerySet(model=Problem) query_set = QuerySet(model=Problem)
query_set = query_set.filter( query_set = query_set.filter(
**{'knowledge_id': self.data.get('knowledge_id')}) **{'knowledge_id': self.data.get('knowledge_id')})

14
ui/src/views/system-resource-management/KnowledgeResourceIndex.vue
View File

@ -172,7 +172,7 @@
text text
:title="$t('views.knowledge.setting.vectorization')" :title="$t('views.knowledge.setting.vectorization')"
@click.stop="reEmbeddingKnowledge(row)" @click.stop="reEmbeddingKnowledge(row)"
v-if="permissionPrecise.vector(row.id)" v-if="permissionPrecise.vector()"
> >
<AppIcon iconName="app-vectorization"></AppIcon> <AppIcon iconName="app-vectorization"></AppIcon>
</el-button> </el-button>
@ -189,14 +189,14 @@
<el-dropdown-item <el-dropdown-item
icon="Refresh" icon="Refresh"
@click.stop="syncKnowledge(row)" @click.stop="syncKnowledge(row)"
v-if="row.type === 1 && permissionPrecise.sync(row.id)" v-if="row.type === 1 && permissionPrecise.sync()"
>{{ $t('views.knowledge.setting.sync') }} >{{ $t('views.knowledge.setting.sync') }}
</el-dropdown-item> </el-dropdown-item>
<el-dropdown-item <el-dropdown-item
icon="Connection" icon="Connection"
@click.stop="openGenerateDialog(row)" @click.stop="openGenerateDialog(row)"
v-if="permissionPrecise.generate(row.id)" v-if="permissionPrecise.generate()"
>{{ $t('views.document.generateQuestion.title') }} >{{ $t('views.document.generateQuestion.title') }}
</el-dropdown-item> </el-dropdown-item>
<el-dropdown-item <el-dropdown-item
@ -206,20 +206,20 @@
path: `/knowledge/${row.id}/resource-management/setting`, path: `/knowledge/${row.id}/resource-management/setting`,
}) })
" "
v-if="permissionPrecise.edit(row.id)" v-if="permissionPrecise.edit()"
> >
{{ $t('common.setting') }} {{ $t('common.setting') }}
</el-dropdown-item> </el-dropdown-item>
<el-dropdown-item <el-dropdown-item
@click.stop="exportKnowledge(row)" @click.stop="exportKnowledge(row)"
v-if="permissionPrecise.export(row.id)" v-if="permissionPrecise.export()"
> >
<AppIcon iconName="app-export"></AppIcon <AppIcon iconName="app-export"></AppIcon
>{{ $t('views.document.setting.export') }} Excel >{{ $t('views.document.setting.export') }} Excel
</el-dropdown-item> </el-dropdown-item>
<el-dropdown-item <el-dropdown-item
@click.stop="exportZipKnowledge(row)" @click.stop="exportZipKnowledge(row)"
v-if="permissionPrecise.export(row.id)" v-if="permissionPrecise.export()"
> >
<AppIcon iconName="app-export"></AppIcon <AppIcon iconName="app-export"></AppIcon
>{{ $t('views.document.setting.export') }} ZIP</el-dropdown-item >{{ $t('views.document.setting.export') }} ZIP</el-dropdown-item
@ -228,7 +228,7 @@
icon="Delete" icon="Delete"
type="danger" type="danger"
@click.stop="deleteKnowledge(row)" @click.stop="deleteKnowledge(row)"
v-if="permissionPrecise.delete(row.id)" v-if="permissionPrecise.delete()"
> >
{{ $t('common.delete') }}</el-dropdown-item {{ $t('common.delete') }}</el-dropdown-item
> >