Elgg / community_plugins

Elgg community plugin repository
17 stars 16 forks source link

canComment() permissions #129

Open hypeJunction opened 9 years ago

hypeJunction commented 9 years ago

I am noticing that the core doesn't really check canComment() permissions before rendering a form or when saving a comment. Seems that we rely entirely on canWriteToContainer() permissions. We need to align these. Perhaps it would make sense to rewrite canComment() to default to canWriteToContainer()

hypeJunction commented 9 years ago

There is a difference in how these two behave with regards to group content.

hypeJunction commented 9 years ago

@juho-jaakkola