Skip to content
Snippets Groups Projects
Commit 343e9c28 authored by Jonathan Weth's avatar Jonathan Weth :keyboard:
Browse files

Merge branch '996-person-editing-form-disabled' into 'master'

Resolve "Person editing form disabled"

Closes #996

See merge request !1442
parents e728a80c 48be35eb
No related branches found
No related tags found
1 merge request!1442Resolve "Person editing form disabled"
Pipeline #178475 failed
...@@ -51,6 +51,7 @@ Changed ...@@ -51,6 +51,7 @@ Changed
Fixed Fixed
~~~~~ ~~~~~
* Persons could not be edited by non-superusers with global person editing permission.
* GraphQL mutations did not return errors in case of exceptions. * GraphQL mutations did not return errors in case of exceptions.
* Make email field unique over all persons. * Make email field unique over all persons.
* Third-party login buttons now directly open external login page. * Third-party login buttons now directly open external login page.
......
...@@ -113,6 +113,7 @@ class PersonForm(ExtensibleForm): ...@@ -113,6 +113,7 @@ class PersonForm(ExtensibleForm):
if ( if (
request request
and self.instance and self.instance
and not request.user.has_perm("core.change_person")
and not request.user.has_perm("core.change_person", self.instance) and not request.user.has_perm("core.change_person", self.instance)
): ):
# Disable non-editable fields # Disable non-editable fields
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment