Skip to content
Snippets Groups Projects
Commit 9eaa858c authored by Nik | Klampfradler's avatar Nik | Klampfradler
Browse files

Merge branch '773-editdashboardview-permission-check-broken' into 'master'

Resolve "EditDashboardView permission check broken"

Closes #773

See merge request !1162
parents aaf21e78 ce540fe7
No related branches found
No related tags found
1 merge request!1162Resolve "EditDashboardView permission check broken"
Pipeline #111218 canceled
......@@ -50,6 +50,7 @@ Changed
Fixed
~~~~~
* The permission check for the dashboard edit page failed when the user had no person assigned.
* In some cases, the IFrame for legacy pages was not properly sized for its content.
* When accessing the person overview page without a person ID, the avatar image was not displayed properly.
* The system tried to send notifications for done background tasks
......
......@@ -971,7 +971,7 @@ class EditDashboardView(PermissionRequiredMixin, View):
if (
self.default_dashboard
and not request.user.has_perm("core.edit_default_dashboard_rule")
or getattr(request.user.person, "is_dummy", False)
or getattr(request.user, "person", True) and getattr(request.user.person, "is_dummy", False)
):
raise PermissionDenied()
......
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