Problem/Motivation
When the entity isn't a post comment "extra checks" take place. These extra checks are potentially going to load all the Group Content of the entity which can be a lot, then it's going to loop on all that loaded group content to check permissions. This can probably be done differently in a better optimised way.
Proposed resolution
Use 'break' statement to stop altering at first encounter of permission conflict for a group.
Comments
Comment #2
navneet0693 commentedHere is a pull request for this: https://github.com/goalgorilla/open_social/pull/1488
Comment #3
kingdutchSee PR
Comment #4
navneet0693 commentedOn it!
Comment #5
navneet0693 commentedUpdated the PR.
Comment #7
jaapjan commented