We have moved to Github. Please open tickets there.

#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

this works as expected - we stop the action

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

Note: See TracTickets for help on using tickets.