diff options
author | Étienne Loks <etienne.loks@iggdrasil.net> | 2024-10-16 17:57:13 +0200 |
---|---|---|
committer | Étienne Loks <etienne.loks@iggdrasil.net> | 2025-02-19 14:43:48 +0100 |
commit | c93dd3812c53d21ab8517dc7af72e1d4b70a1b04 (patch) | |
tree | 2153d8fd121f7ecd08a31e4867d58a2eb3c9aab7 /archaeological_context_records/views.py | |
parent | b8eef9b6aaed7ee097f8ea86174067f9ca42abd8 (diff) | |
download | Ishtar-c93dd3812c53d21ab8517dc7af72e1d4b70a1b04.tar.bz2 Ishtar-c93dd3812c53d21ab8517dc7af72e1d4b70a1b04.zip |
♻ permissions refactoring: refactor has_permission methods
Diffstat (limited to 'archaeological_context_records/views.py')
-rw-r--r-- | archaeological_context_records/views.py | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/archaeological_context_records/views.py b/archaeological_context_records/views.py index 15b2aee30..fdee2c459 100644 --- a/archaeological_context_records/views.py +++ b/archaeological_context_records/views.py @@ -51,11 +51,13 @@ revert_contextrecord = revert_item(models.ContextRecord) def autocomplete_contextrecord(request): - if not request.user.has_perm( - "archaeological_context_records.view_contextrecord", models.ContextRecord - ) and not request.user.has_perm( + ishtaruser = getattr(request.user, "ishtaruser", None) + if not ishtaruser: + return HttpResponse(content_type="text/plain") + if not ishtaruser.has_permission( + "archaeological_context_records.view_contextrecord" + ) and not ishtaruser.has_permission( "archaeological_context_records.view_own_contextrecord", - models.ArchaeologicalSite, ): return HttpResponse(content_type="text/plain") if not request.GET.get("term"): @@ -258,7 +260,8 @@ class QAOperationContextRecordView(QAItemForm): icon_class="fa fa-plus", text=_("Add context record"), target="one", - rights=["add_contextrecord", "add_own_contextrecord"], + rights=["archaeological_context_records.add_contextrecord", + "archaeological_context_records.add_own_contextrecord"], ) def form_valid(self, form): |