simplify admin register

fet2020/finance/admin.py

@@ -23,6 +23,7 @@ class BillInline(admin.TabularInline):
     show_change_link = True
 
 
+@admin.register(BankData)
 class BankDataAdmin(admin.ModelAdmin):
     form = BankDataAdminForm
     model = BankData
@@ -52,6 +53,7 @@ class BankDataAdmin(admin.ModelAdmin):
         )
 
 
+@admin.register(Bill)
 class BillAdmin(admin.ModelAdmin):
     form = BillAdminForm
     model = Bill
@@ -142,6 +144,7 @@ class BillAdmin(admin.ModelAdmin):
         obj.author = request.user
         super().save_model(request, obj, form, change)
 
+    @admin.display(description="Kontoinhaber:in")
     def get_bankdata_name(self, obj):
         try:
             tmp = BankData.objects.get(id=obj.bankdata.id)
@@ -150,8 +153,7 @@ class BillAdmin(admin.ModelAdmin):
 
         return f"{ tmp.name }"
 
-    get_bankdata_name.short_description = "Kontoinhaber:in"
-
+    @admin.display(description="IBAN")
     def get_bankdata_iban(self, obj):
         try:
             tmp = BankData.objects.get(id=obj.bankdata.id)
@@ -160,8 +162,7 @@ class BillAdmin(admin.ModelAdmin):
 
         return f"{ tmp.iban }"
 
-    get_bankdata_iban.short_description = "IBAN"
-
+    @admin.display(description="BIC")
     def get_bankdata_bic(self, obj):
         try:
             tmp = BankData.objects.get(id=obj.bankdata.id)
@@ -170,8 +171,7 @@ class BillAdmin(admin.ModelAdmin):
 
         return f"{ tmp.bic }"
 
-    get_bankdata_bic.short_description = "BIC"
-
+    @admin.display(description="Status")
     def status_colored(self, obj):
         # TODO: if there is a status without color, set nothing.
         colors = {
@@ -184,8 +184,6 @@ class BillAdmin(admin.ModelAdmin):
             f'<b style="background:{colors[obj.status]};">{obj.get_status_display()}</b>'
         )
 
-    status_colored.short_description = "Status"
-
     @admin.action(description="Als 'Abgerechnet' markieren.")
     def make_cleared(self, request, queryset):
         updated = queryset.update(status="C")
@@ -215,6 +213,7 @@ class BillAdmin(admin.ModelAdmin):
         )
 
 
+@admin.register(Resolution)
 class ResolutionAdmin(admin.ModelAdmin):
     form = ResolutionAdminForm
     model = Resolution
@@ -285,6 +284,7 @@ class ResolutionAdmin(admin.ModelAdmin):
             extra_context=extra_context,
         )
 
+    @admin.display(description="Gesamtsumme (EUR)")
     def total(self, obj):
         total = 0
         bills = Bill.objects.filter(resolution=obj)
@@ -293,9 +293,8 @@ class ResolutionAdmin(admin.ModelAdmin):
 
         return f"{ total }"
 
-    total.short_description = "Gesamtsumme (EUR)"
-
 
+@admin.register(Wiref)
 class WirefAdmin(admin.ModelAdmin):
     form = WirefAdminForm
     model = Wiref
@@ -327,6 +326,7 @@ class WirefAdmin(admin.ModelAdmin):
             extra_context=extra_context,
         )
 
+    @admin.display(description="Gesamtsumme (EUR)")
     def total(self, obj):
         total = 0
         bills = Bill.objects.filter(wiref=obj)
@@ -334,11 +334,3 @@ class WirefAdmin(admin.ModelAdmin):
             total += elem.amount
 
         return f"{ total }"
-
-    total.short_description = "Gesamtsumme (EUR)"
-
-
-admin.site.register(BankData, BankDataAdmin)
-admin.site.register(Bill, BillAdmin)
-admin.site.register(Resolution, ResolutionAdmin)
-admin.site.register(Wiref, WirefAdmin)