Remove not supported moderation-permission
(MGNLFORUM-250)
|
|
| Status: | Closed |
| Project: | Forum (closed) |
| Component/s: | security |
| Affects Version/s: | None |
| Fix Version/s: | 3.3 |
| Type: | Sub-task | Priority: | Neutral |
| Reporter: | Christoph Meier | Assignee: | Roman Kovařík |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Template: |
|
| Date of First Response: |
| Comments |
| Comment by Roman Kovařík [ 06/Mar/14 ] |
String createRolesForNewForumsPropertyValue = PropertyUtil.getString(installContext.getJCRSession(RepositoryConstants.CONFIG).getNode("/modules/forum/config"), "createRolesForNewForums"); assertEquals("false", createRolesForNewForumsPropertyValue); You can use: // THEN assertFalse(configSession.getProperty("/modules/forum/config/createRolesForNewForums").getBoolean()); Otherwise looks OK. |
| Comment by Christoph Meier [ 11/Mar/14 ] |
|
The commit onto master was done against the parent ticket |
| Comment by Milan Divilek [ 11/Mar/14 ] |
|
Reopen: This is more like suggestion of different approach than reopening of ticket. If /modules/forum/config/createRolesForNewForums is set to true then info.magnolia.module.forum.DefaultForumManager#createForumRoles will create three roles "forum" + forumName + "user", "forum" + forumName + "moderator" and "forum" + forumName + "admin". "forum" + forumName + "moderator" and "forum" + forumName + "admin" are obsolete because of |