Closed (won't fix)
Project:
Drupal core
Version:
8.0.x-dev
Component:
forum.module
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
10 Aug 2010 at 06:11 UTC
Updated:
29 Jul 2014 at 18:57 UTC
Jump to comment: Most recent file
Comments
Comment #1
dddave commentedFor 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.
Comment #2
realityloop commentedddave, 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.
Comment #3
realityloop commentedDrupal 7 patch attached
Comment #4
dddave commentedSetting to D7 so that folks see it in the queue...
Comment #6
realityloop commentedApplied required changes to forum.test
Comment #8
andypostStill think that this task for contrib, because setting access to forum (taxonomy field) is useless if forum nodes are accessible without forum access dependency
Comment #9
realityloop commentedUpdated test file
Comment #10
andypostLet's allow a bot to test a patch. But I strongly disagree that core forum have additional access
Comment #12
realityloop commentedchanging version
Comment #13
alexweber commentedpatch attached addressing all of the above issues :)
Comment #14
andypostThis 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.
Comment #15
alexweber commentedfair enough!