MIRRORS/layerindex-web
1
0
Fork 0
mirror of git://git.yoctoproject.org/layerindex-web.git synced 2025-07-19 12:49:01 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

admin.py: Remove has_delete_permission()

Browse Source 
The following commit has added on_delete=models.CASCADE to the ForeignKey:

===
commit 2d526f9b0d
Author: Amber Elliot <amber.n.elliot@intel.com>
Date:   Thu Jun 25 15:57:34 2020 -0700

    Updating models and imports for Django 2.2 upgrade.
===

The on_delete=models.CASCADE will remove the objects which reference the
removed objects, so there will be errors when removing a LayerBranch
with admin permissions:

===
Cannot delete layer branch
Deleting the selected layer branch would result in deleting related objects, but your account doesn't have permission to delete the following types of objects:

Append
distro
===

This is because has_delete_permission() always return False, remove it to use
Django's implementation to fix the problem.

Minor rewording.

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
Signed-off-by: Tim Orling <tim.orling@konsulko.com>
This commit is contained in:
Robert Yang 2022-12-16 08:42:52 -08:00 committed by Tim Orling
parent edb574ae7e
commit 2d7906ff3b
1 changed files with 0 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
layerindex/admin.py
View File

@ -102,8 +102,6 @@ class RecipeAdmin(admin.ModelAdmin):
return rofields return rofields
def has_add_permission(self, request, obj=None): def has_add_permission(self, request, obj=None):
return False return False
def has_delete_permission(self, request, obj=None):
return False
class PackageConfigAdmin(admin.ModelAdmin): class PackageConfigAdmin(admin.ModelAdmin):
search_fields = ['feature', 'recipe__pn'] search_fields = ['feature', 'recipe__pn']
@ -132,8 +130,6 @@ class ClassicRecipeAdmin(admin.ModelAdmin):
return rofields return rofields
def has_add_permission(self, request, obj=None): def has_add_permission(self, request, obj=None):
return False return False
def has_delete_permission(self, request, obj=None):
return False
class MachineAdmin(admin.ModelAdmin): class MachineAdmin(admin.ModelAdmin):
search_fields = ['name'] search_fields = ['name']
@ -141,8 +137,6 @@ class MachineAdmin(admin.ModelAdmin):
readonly_fields = [f.name for f in Machine._meta.get_fields()] readonly_fields = [f.name for f in Machine._meta.get_fields()]
def has_add_permission(self, request, obj=None): def has_add_permission(self, request, obj=None):
return False return False
def has_delete_permission(self, request, obj=None):
return False
class DistroAdmin(admin.ModelAdmin): class DistroAdmin(admin.ModelAdmin):
search_fields = ['name'] search_fields = ['name']
@ -150,9 +144,6 @@ class DistroAdmin(admin.ModelAdmin):
readonly_fields = [f.name for f in Distro._meta.get_fields()] readonly_fields = [f.name for f in Distro._meta.get_fields()]
def has_add_permission(self, request, obj=None): def has_add_permission(self, request, obj=None):
return False return False
def has_delete_permission(self, request, obj=None):
return False
class BBAppendAdmin(admin.ModelAdmin): class BBAppendAdmin(admin.ModelAdmin):
search_fields = ['filename'] search_fields = ['filename']
@ -160,8 +151,6 @@ class BBAppendAdmin(admin.ModelAdmin):
readonly_fields = [f.name for f in BBAppend._meta.get_fields()] readonly_fields = [f.name for f in BBAppend._meta.get_fields()]
def has_add_permission(self, request, obj=None): def has_add_permission(self, request, obj=None):
return False return False
def has_delete_permission(self, request, obj=None):
return False
class BBClassAdmin(admin.ModelAdmin): class BBClassAdmin(admin.ModelAdmin):
search_fields = ['name'] search_fields = ['name']
@ -169,8 +158,6 @@ class BBClassAdmin(admin.ModelAdmin):
readonly_fields = [f.name for f in BBClass._meta.get_fields()] readonly_fields = [f.name for f in BBClass._meta.get_fields()]
def has_add_permission(self, request, obj=None): def has_add_permission(self, request, obj=None):
return False return False
def has_delete_permission(self, request, obj=None):
return False
class PatchAdmin(admin.ModelAdmin): class PatchAdmin(admin.ModelAdmin):
search_fields = ['path'] search_fields = ['path']
@ -178,8 +165,6 @@ class PatchAdmin(admin.ModelAdmin):
readonly_fields = [f.name for f in Patch._meta.get_fields()] readonly_fields = [f.name for f in Patch._meta.get_fields()]
def has_add_permission(self, request, obj=None): def has_add_permission(self, request, obj=None):
return False return False
def has_delete_permission(self, request, obj=None):
return False
class PatchDispositionAdmin(admin.ModelAdmin): class PatchDispositionAdmin(admin.ModelAdmin):
fields = ['patch', 'user', 'disposition', 'comment'] fields = ['patch', 'user', 'disposition', 'comment']
@ -199,8 +184,6 @@ class IncFileAdmin(admin.ModelAdmin):
readonly_fields = [f.name for f in IncFile._meta.get_fields()] readonly_fields = [f.name for f in IncFile._meta.get_fields()]
def has_add_permission(self, request, obj=None): def has_add_permission(self, request, obj=None):
return False return False
def has_delete_permission(self, request, obj=None):
return False
class RecipeChangeInline(admin.StackedInline): class RecipeChangeInline(admin.StackedInline):
model = RecipeChange model = RecipeChange