From 9e7f7df7218722079b0afd5bf38b5c5b3901138b Mon Sep 17 00:00:00 2001 From: thomasl Date: Thu, 13 Feb 2025 20:09:45 +0100 Subject: [PATCH] Some improvements --- apps/member/forms.py | 1 - apps/member/views.py | 6 +++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/apps/member/forms.py b/apps/member/forms.py index e578c837..ecf7fb59 100644 --- a/apps/member/forms.py +++ b/apps/member/forms.py @@ -15,7 +15,6 @@ from django.utils.translation import gettext_lazy as _ from note.models import NoteSpecial, Alias from note_kfet.inputs import Autocomplete, AmountInput from permission.models import PermissionMask, Role -from permission.backends import PermissionBackend from PIL import Image, ImageSequence from .models import Profile, Club, Membership diff --git a/apps/member/views.py b/apps/member/views.py index 7f266529..3bbc167f 100644 --- a/apps/member/views.py +++ b/apps/member/views.py @@ -79,13 +79,17 @@ class UserUpdateView(ProtectQuerysetMixin, LoginRequiredMixin, UpdateView): del profile_form.fields["last_report"] fields_to_check = list(profile_form.fields.keys()) + fields_modifiable = False # Delete the fields for which the user does not have the permission to modify for field_name in fields_to_check: if not PermissionBackend.check_perm(self.request, f"member.change_profile_{field_name}", context['user_object'].profile): profile_form.fields[field_name].widget = forms.HiddenInput() + else : + fields_modifiable = True - context['profile_form'] = profile_form + if fields_modifiable : + context['profile_form'] = profile_form return context