-
Notifications
You must be signed in to change notification settings - Fork 15
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
Groups owners or moderators can't see reports #80
Comments
For non-system admin users, they need to be an owner, admin or moderator of the space. However, there is an exception:
See https://github.com/humhub/reportcontent/blob/master/models/ReportContent.php#L97 @luke- I think this adds confusion, mainly when testing the module to understand how it works. |
@Semir1212 What do you think? |
I've done lots of tests, including deleting the module and re-installing it. I've reported things as admin, as a regular member and both in spaces of my own and other spaces. |
@luke- @marc-farre I think there should be a way for users to report content from space admins. The behaviour is explained in the manual. Maybe we could create one report for the space admin an one for the sys admin? If not, I would leave it as is. |
I've done some tests by reporting a post in a space:
So this is the current behavior of the module, and it's working with the Report Content module version 1.1.0. @Semir1212 so do you agree, for the case n°1 (reporting a post created by a space admin), we should also notify all space admins and moderators (in addition to the global "Administrators" group members)? |
This is the kind of explanation that would have been useful in the description of the module. |
@marc-farre Yes, as long as the sys admins are also notifed and the space admins can not remove the report by themselves. |
Currently testing for this issue;
The only issues that I'm seeing is no notifications are sent to owner/administrators or moderators and both Space & Global reports aren't filtered so whatever is reported in a Space also is sent to Global but I'd also like to note that point 1 also can be considered an issue as well here. My idea when it comes to the logic that needs done here is as followed;
|
bump @marjanapesic |
Not seeing this issue exactly within HumHub v1.17.0-beta.2 and Report Content v1.1.0, also @ilMagnifico Marjana Mijatovic does not work on HumHub modules anymore from my understanding? |
@ArchBlood I'll get back to you when I download the 1.17. |
The report module doesn’t work as advertised. Only the system administrator (currently me) receives the reports, not group owners or group moderators.
The text was updated successfully, but these errors were encountered: