Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add permission to use custom reports (LANDA-375) #107

Closed
wants to merge 1 commit into from

Conversation

barredterra
Copy link
Member

@barredterra barredterra commented Nov 4, 2021

Add permission to create custom reports to role LANDA Regional Organization Management.

Bildschirmfoto 2021-11-04 um 19 38 45

Frappe validiert selber, dass hier niemand heimlich SQL-Abfragen einbaut.

For LANDA Regional Organization Management
Copy link
Contributor

@melanieroedel melanieroedel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bei mir geht es leider gar nicht.
Wenn ich als RV Mitarbeiter einen fertigen Report aufrufe (z. Adress- oder Geburtstagsliste) kommt ein Fehler (Unzureichende Berechtigung)

10:03:31 web.1            | Traceback (most recent call last):
10:03:31 web.1            |   File "/home/frappe/bench-landa/apps/frappe/frappe/app.py", line 68, in application
10:03:31 web.1            |     response = frappe.api.handle()
10:03:31 web.1            |   File "/home/frappe/bench-landa/apps/frappe/frappe/api.py", line 55, in handle
10:03:31 web.1            |     return frappe.handler.handle()
10:03:31 web.1            |   File "/home/frappe/bench-landa/apps/frappe/frappe/handler.py", line 31, in handle
10:03:31 web.1            |     data = execute_cmd(cmd)
10:03:31 web.1            |   File "/home/frappe/bench-landa/apps/frappe/frappe/handler.py", line 67, in execute_cmd
10:03:31 web.1            |     return frappe.call(method, **frappe.form_dict)
10:03:31 web.1            |   File "/home/frappe/bench-landa/apps/frappe/frappe/__init__.py", line 1199, in call
10:03:31 web.1            |     return fn(*args, **newargs)
10:03:31 web.1            |   File "/home/frappe/bench-landa/apps/frappe/frappe/desk/form/load.py", line 40, in getdoc
10:03:31 web.1            |     raise frappe.PermissionError(("read", doctype, name))
10:03:31 web.1            | frappe.exceptions.PermissionError: ('read', 'Report', 'Member Birthday List')

Wenn ich als RV Mitarbeiter einen normalen Bericht (z. B. Mitglied) aufrufe, gibt es im Menü nicht die Option "Speichern" wie es beim Administrator der Fall wäre.

@melanieroedel
Copy link
Contributor

@barredterra Oh Mann, es hat eine ganze Weile gedauert, bis ich gemerkt habe, was das problem ist. Dein Hack scheint die Datenbank zu überschreiben. Deshalb sind die Berechtigungen für Report auch noch genauso falsch gesetzt, wenn ich wieder version-13 auschecke wie vorher.
Dafür habe ich aber vermutlich auch das Problem identifiziert.
Deine Änderung setzt nur die Berechtigungen für LANDA Regional Organization Management:
image
Dafür werden aber die Standard Einstellungen entfernt:
image

@barredterra
Copy link
Member Author

@melanieroedel das scheint ein tieferes Problem in frappe zu sein, daher würde ich die Lösung nach dort verlagern: frappe/frappe#15074

@barredterra barredterra deleted the landa-375 branch November 24, 2021 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants