remove debug print. Renamed permission shortname

CodingPirates · Nov 13, 2024 · bc58e93 · bc58e93
1 parent cbe98b3
commit bc58e93
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 5 deletions.
diff --git a/members/admin/union_admin.py b/members/admin/union_admin.py
@@ -103,7 +103,6 @@ def get_fieldsets(self, request, obj=None):
         # 20241113: https://stackoverflow.com/questions/16102222/djangoremove-superuser-checkbox-from-django-admin-panel-when-login-staff-users
 
         if not obj:
-            print("not obj")
             return self.add_fieldsets
 
         info_fields = (
@@ -116,7 +115,6 @@ def get_fieldsets(self, request, obj=None):
         )
 
         if not request.user.has_perm("members.showledgeraccount"):
-            print("no perm")
             info_fields = (
                 "bank_main_org",
                 "bank_account",
@@ -125,8 +123,6 @@ def get_fieldsets(self, request, obj=None):
                 "closed_at",
             )
 
-        print(f"info:{info_fields}")
-
         return [
             (
                 "Navn og Adresse",

diff --git a/members/migrations/0060_alter_union_options.py b/members/migrations/0060_alter_union_options.py
@@ -0,0 +1,25 @@
+# Generated by Django 4.2.11 on 2024-11-13 21:20
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("members", "0059_merge_20241113_2111"),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name="union",
+            options={
+                "ordering": ["name"],
+                "permissions": (
+                    ("view_all_unions", "Can view all Foreninger"),
+                    ("show_ledger_account", "Show General Ledger Account"),
+                ),
+                "verbose_name": "Forening",
+                "verbose_name_plural": "Foreninger",
+            },
+        ),
+    ]
diff --git a/members/models/union.py b/members/models/union.py
@@ -12,7 +12,7 @@ class Meta:
         ordering = ["name"]
         permissions = (
             ("view_all_unions", "Can view all Foreninger"),
-            ("showledgeraccount", "Show General Ledger Account"),
+            ("show_ledger_account", "Show General Ledger Account"),
         )
 
     help_union = """Vi tilføjer automatisk "Coding Pirates" foran navnet når vi nævner det de fleste steder på siden."""