mirror of https://github.com/jumpserver/jumpserver
[Fix] 权限管理-> 资产与数据库 Bug (#4049)
parent
196f1654ab
commit
f8142e23cd
|
@ -57,8 +57,8 @@ class AssetPermissionSerializer(BulkOrgResourceModelSerializer):
|
||||||
def setup_eager_loading(cls, queryset):
|
def setup_eager_loading(cls, queryset):
|
||||||
""" Perform necessary eager loading of data. """
|
""" Perform necessary eager loading of data. """
|
||||||
queryset = queryset.annotate(
|
queryset = queryset.annotate(
|
||||||
users_amount=Count('users'), user_groups_amount=Count('user_groups'),
|
users_amount=Count('users', distinct=True), user_groups_amount=Count('user_groups', distinct=True),
|
||||||
assets_amount=Count('assets'), nodes_amount=Count('nodes'),
|
assets_amount=Count('assets', distinct=True), nodes_amount=Count('nodes', distinct=True),
|
||||||
system_users_amount=Count('system_users')
|
system_users_amount=Count('system_users', distinct=True)
|
||||||
)
|
)
|
||||||
return queryset
|
return queryset
|
||||||
|
|
|
@ -13,30 +13,7 @@ __all__ = [
|
||||||
]
|
]
|
||||||
|
|
||||||
|
|
||||||
class DatabaseAppPermissionSerializer(BulkOrgResourceModelSerializer):
|
class AmountMixin:
|
||||||
class Meta:
|
|
||||||
model = models.DatabaseAppPermission
|
|
||||||
list_serializer_class = AdaptedBulkListSerializer
|
|
||||||
fields = [
|
|
||||||
'id', 'name', 'users', 'user_groups',
|
|
||||||
'database_apps', 'system_users', 'comment', 'is_active',
|
|
||||||
'date_start', 'date_expired', 'is_valid',
|
|
||||||
'created_by', 'date_created'
|
|
||||||
]
|
|
||||||
read_only_fields = ['created_by', 'date_created']
|
|
||||||
|
|
||||||
|
|
||||||
class DatabaseAppPermissionListSerializer(BulkOrgResourceModelSerializer):
|
|
||||||
is_expired = serializers.BooleanField()
|
|
||||||
|
|
||||||
class Meta:
|
|
||||||
model = models.DatabaseAppPermission
|
|
||||||
fields = [
|
|
||||||
'id', 'name', 'comment', 'is_active', 'users_amount', 'user_groups_amount',
|
|
||||||
'date_start', 'date_expired', 'is_valid', 'database_apps_amount', 'system_users_amount',
|
|
||||||
'created_by', 'date_created', 'is_expired'
|
|
||||||
]
|
|
||||||
|
|
||||||
@classmethod
|
@classmethod
|
||||||
def setup_eager_loading(cls, queryset):
|
def setup_eager_loading(cls, queryset):
|
||||||
""" Perform necessary eager loading of data. """
|
""" Perform necessary eager loading of data. """
|
||||||
|
@ -46,3 +23,31 @@ class DatabaseAppPermissionListSerializer(BulkOrgResourceModelSerializer):
|
||||||
system_users_amount=Count('system_users', distinct=True)
|
system_users_amount=Count('system_users', distinct=True)
|
||||||
)
|
)
|
||||||
return queryset
|
return queryset
|
||||||
|
|
||||||
|
|
||||||
|
class DatabaseAppPermissionSerializer(AmountMixin, BulkOrgResourceModelSerializer):
|
||||||
|
class Meta:
|
||||||
|
model = models.DatabaseAppPermission
|
||||||
|
list_serializer_class = AdaptedBulkListSerializer
|
||||||
|
fields = [
|
||||||
|
'id', 'name', 'users', 'user_groups', 'database_apps', 'system_users',
|
||||||
|
'comment', 'is_active', 'date_start', 'date_expired', 'is_valid',
|
||||||
|
'created_by', 'date_created', 'users_amount', 'user_groups_amount',
|
||||||
|
'database_apps_amount', 'system_users_amount',
|
||||||
|
]
|
||||||
|
read_only_fields = [
|
||||||
|
'created_by', 'date_created', 'users_amount', 'user_groups_amount',
|
||||||
|
'database_apps_amount', 'system_users_amount',
|
||||||
|
]
|
||||||
|
|
||||||
|
|
||||||
|
class DatabaseAppPermissionListSerializer(AmountMixin, BulkOrgResourceModelSerializer):
|
||||||
|
is_expired = serializers.BooleanField()
|
||||||
|
|
||||||
|
class Meta:
|
||||||
|
model = models.DatabaseAppPermission
|
||||||
|
fields = [
|
||||||
|
'id', 'name', 'comment', 'is_active', 'users_amount', 'user_groups_amount',
|
||||||
|
'date_start', 'date_expired', 'is_valid', 'database_apps_amount', 'system_users_amount',
|
||||||
|
'created_by', 'date_created', 'is_expired'
|
||||||
|
]
|
||||||
|
|
Loading…
Reference in New Issue