Opened 10 months ago
Closed 10 months ago
#4706 closed Defect (wontfix)
You can add a group on the community site.
| Reported by: | Speedysnail6 | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Needs Review |
| Component: | Community Site | Version: | 1.8 |
| Severity: | minor | Keywords: | community, groups |
| Cc: | brett@… | Difficulty: |
Description
A regular user can add groups to the community site.
http://community.elgg.org/groups/add
Change History (8)
comment:1 Changed 10 months ago by Speedysnail6
- Keywords community, groups added
comment:2 Changed 10 months ago by cash
- Resolution set to worksforme
- Status changed from new to closed
comment:3 Changed 10 months ago by ewinslow
Wouldn't it be reasonable to have a permissions check at the top of this page?
comment:4 Changed 10 months ago by cash
I removed the link to the page. This means the user is manually typing in the URL. To prevent someone from accessing the page, we would have to register for the routing plugin hook and then check if an admin. I feel like I have a ton of other things to do and this is really about improving the user experience for people who are trying to bypass our restrictions - so extremely low priority for me.
comment:5 Changed 10 months ago by ewinslow
I mean more generally in Elgg, if the page is for performing some action, would it not be a good idea to check if they can actually perform that action?
Agreed about priorities.
comment:6 Changed 10 months ago by cash
I added some ideas to the Permissions Gdoc related to this. What if we changed the permissions system to be CRUD based. Then there would be a way to ask if a user can create something and add that to pages that create content. Right now all we can do is ask if the user has permissions to write to a container which is different.
comment:7 Changed 10 months ago by Speedysnail6
- Resolution worksforme deleted
- Status changed from closed to reopened
It just looks sloppy to keep the form there.
comment:8 Changed 10 months ago by cash
- Resolution set to wontfix
- Status changed from reopened to closed
You can submit a pull request to https://github.com/Elgg/community_groups

this works as expected - we stop the action