I thought having a permission for access to Forums would be useful, this allows you to assign access based on role

Comments

dddave’s picture

Version: 6.x-dev » 8.x-dev

For D6 we have to live with the contrib solutions like http://drupal.org/project/forum_access. Feature requests have to be against D8 by now. I don't know how D7 handles forum access so this might already be obsolete for future releases.

realityloop’s picture

ddave, I figured this would be the case but thought it might be useful for someone, I'll have a look at D7 tomorrow and submit a patch for that too if relevant.

realityloop’s picture

StatusFileSize
new9.28 KB

Drupal 7 patch attached

dddave’s picture

Version: 8.x-dev » 7.x-dev

Setting to D7 so that folks see it in the queue...

Status: Needs review » Needs work

The last submitted patch, forum-role-based-accessd7.patch, failed testing.

realityloop’s picture

Status: Needs work » Needs review
StatusFileSize
new10.88 KB

Applied required changes to forum.test

Status: Needs review » Needs work

The last submitted patch, forum-role-based-accessd7.patch, failed testing.

andypost’s picture

Version: 7.x-dev » 8.x-dev

Still think that this task for contrib, because setting access to forum (taxonomy field) is useless if forum nodes are accessible without forum access dependency

realityloop’s picture

StatusFileSize
new10.96 KB

Updated test file

andypost’s picture

Version: 8.x-dev » 7.x-dev
Status: Needs work » Needs review

Let's allow a bot to test a patch. But I strongly disagree that core forum have additional access

Status: Needs review » Needs work

The last submitted patch, forum-role-based-accessd7.patch, failed testing.

realityloop’s picture

Version: 7.x-dev » 8.x-dev

changing version

alexweber’s picture

Status: Needs work » Needs review
StatusFileSize
new18.2 KB

patch attached addressing all of the above issues :)

andypost’s picture

Status: Needs review » Closed (won't fix)

This patch just adds additional access for /forum/* but still does not care about content that has forum field attached.

If this patch will introduce node access for nodes that posted to forum so we had a BIG confusion|collision within node access! Users could have 'access content' permission but 'access forum' should override this. So better off this to contrib

Also there will be performance degradation of node access will be introduced in core.

alexweber’s picture

fair enough!