The talk module allows the administrator to move the comments to a separate page at node/###/talk. It seems like og_user_roles does not apply on this page, because all the role permissions are not applying. For instance, if I grant the permission to create a 'page' content, the "Create page" link suddenly disappears when navigating to the talk page.

CommentFileSizeAuthor
#1 ogur-350747.patch6.44 KBsomebodysysop

Comments

somebodysysop’s picture

somebodysysop commented
Status: Active » Needs review
StatusFileSize
new ogur-350747.patch6.44 KB

Try the attached patch. Please let me know if it works.

joewang’s picture

joewang commented

Wow! Thanks for the quick reply-unfortunately, I won't be able to test this until the new year, since I do not currently have access to the server I am working on.

However, in looking through the code, I did notice that the patch contained patches in addition to fixing this current one. Could you roll a new release, and then upload a patch just for this specific problem? I have the og_forum fix currently installed, but nothing else, and I don't want to go through to change every bug.

Thanks!

somebodysysop’s picture

somebodysysop commented

I actually planned on submitting a new dev release, but decided I should wait until I was sure this update fixed the problem. So I went through the exercise of creating this patch.

If you just want to fix this particular problem, here's the patch segment:

-    if (arg(0) == 'node' && is_numeric(arg(1)) && (arg(2) == 'edit' || arg(2) == 'outline' || arg(2) == 'track' || arg(2) == 'results' || arg(2) == 'access' || arg(2) == 'modr8' || arg(2) == 'ogmodr8' || arg(2) == 'delete' || arg(2) == 'galleries' || arg(2) == 'email' || arg(2) == 'revisions' || arg(2) == 'signups' || arg(2) == 'agenda' || arg(2) == 'workflow' || arg(2) == 'votes' || arg(2) == 'refresh' || arg(2) == 'purge')) {
+	// Modified 2008-12-24: How about just looking for nid in /node/<nid> instead of trying to
+	//                      anticipate each and every module.
+    // if (arg(0) == 'node' && is_numeric(arg(1)) && (arg(2) == 'edit' || arg(2) == 'outline' || arg(2) == 'track' || arg(2) == 'results' || arg(2) == 'access' || arg(2) == 'modr8' || arg(2) == 'ogmodr8' || arg(2) == 'delete' || arg(2) == 'galleries' || arg(2) == 'email' || arg(2) == 'revisions' || arg(2) == 'signups' || arg(2) == 'agenda' || arg(2) == 'workflow' || arg(2) == 'votes' || arg(2) == 'refresh' || arg(2) == 'purge')) {
+    if (arg(0) == 'node' && is_numeric(arg(1))) {

Have fun. Just let us know if it works.

joewang’s picture

joewang commented
Status: Needs review » Fixed

works great! thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.