When I add additional pages to a group which has a different theme to the site default, these pages do not inherit the group theme, although the group home page does display the correct theme.

This seems like the same issue too: http://drupal.org/node/585680

CommentFileSizeAuthor
#13 ga_og_panels_patch_01.txt735 bytesginc

Comments

foxtrotcharlie’s picture

foxtrotcharlie commented

I've done a bit of debugging, and it appears that in og.module line 472 there's a function: og_determine_context and this is where it get's the group context (and theme) from - if no context is returned from this function, then there's no og theme applied to that panel page. If I add in another elseif after the last one, at line 497, like so:

  elseif ((!empty($item['map'][2]) && $item['map'][2] == 'og-panel') && $item['map'][0] == 'node') {
    $node = node_load($item['map'][1]); // $item['map'][1] is the group nid
  }

This supplies the correct group context. So, I'm not sure if this is an og_panels issue - and og_panels is supposed to supply the correct information when menu_get_object(); is called at the beginning of this function, or if it's an og issue, and something like the above code needs to be added to og.module.

I'm happy to create a patch if someone can verify where the problem lies, and if my way of solving it is acceptable...

jide’s picture

jide commented

Did you try to use the og_set_group_context($group_node) function from og_panels instead ?
This function sets the context to a group node so if there is a lack of context somewhere, call this function.

pekman’s picture

pekman commented

Sorry I'm new to all of this and I have this problem. I managed to set up panels with tabs in a group. Everything is as expected on the home page (themes with group theme). But when I click on one of the tabs, the theme changes to the default site theme (I want different themes for site and group). As I said, the group homepage is ok with the groups theme set for the panel. But the tabs don't follow this. What am I doing wrong?

foxtrotcharlie’s picture

foxtrotcharlie commented

Hey Julien - From what I can tell, og_panels does call og_set_group_context in the function og_panels_page line 481, which is the funciton that displays the og panel page.

pekman’s picture

pekman commented

Thanks foxtrotcharlie - your solution worked for me!

isopharius’s picture

isopharius commented

yes very good, now only problem is the og forum tab but that's another matter

any luck on implementing this fix in og_panels instead of og itself?

ultimike’s picture

ultimike
Primary language English
Location Florida, USA
commented

I'm having the same issue with og_panels.

foxtrotcharlie: thanks for the work-around. Too bad it involves hacking OG. It definitely seems like OG Panels is setting the context correctly, but it isn't having any effect on the theming. I've played around with the code for a few hours, but haven't been able to make any progress.

Let's hope the module maintainers carve out a few minutes to jump in and lend a hand...

-mike

jimthunderbird’s picture

jimthunderbird commented

This can be fixed by adding hook_init to the og_panels module

/* implements hook_init */
function og_panels_init(){
  $group_node = node_load(arg(1));
  if(og_is_group_type($group_node->type)){
    og_set_theme($group_node);
    og_set_group_context($group_node);
    og_set_language($group_node);
  }
}
jimthunderbird’s picture

jimthunderbird commented

Or we can add the following code

og_set_theme($group_node);
og_set_group_context($group_node);
og_set_language($group_node);

in function og_panels_page

isopharius’s picture

isopharius commented

#9 works

replaced line 481 in og_panels.module, everything fine, thanks for the fix!

ultimike’s picture

ultimike
Primary language English
Location Florida, USA
commented

Hmmmm - #9 didn't work for me, but #8 did...

-mike

jimthunderbird’s picture

jimthunderbird commented

Hi mike,
I first come up with the hook_init idea and then found if the same code placed in og_panels_page function, it will also work. But my environment has just basic modules like og, views, og_panels installed, it might be the case where you have other modules install and then modify the title after og_panels, thus, i think hook_init might be a safer fix since it will be called in the beginning.

Best Regards,
jim

ginc’s picture

ginc commented
Status: Active » Needs review
StatusFileSize
new ga_og_panels_patch_01.txt735 bytes

patch implementing solution #8 credit to jimthunderbird

ginc’s picture

ginc commented

interesting... the #13 patch fixed similar issue with og_forum tab #339969: Set og theme in og forums

populist’s picture

populist commented
Status: Needs review » Fixed

cleaned it up a bit and commited.

Status: Fixed » Closed (fixed)

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