When using beta12 of subscriptions the widgets to subscribe to a group do not display, when at og/manage/NID

However, this works fine with beta11.

I supplied the patch to show the problematic changes in subscriptions_ui, but I think that subscriptions_og may need to be refactored to accommodate the changes in beta12.

CommentFileSizeAuthor
beta12.patch1.05 KBjgraham

Comments

gustav’s picture

gustav commented

Also all subscriptionsfor groups seem to be ignored. After clicking "Save" in the subscription form the checkboxes revert to unchecked.

develcuy’s picture

develcuy commented

I've fixed it but can't commit, I'm waiting for some changes from gustav.

So the change in _subscriptions_og_node_options() is from:

      $options['group_nid'][] = array(
        'name' => t('To this group'),
        'params' => array('value' => $node->nid),
        'link' => 'node/'. $node->nid,
      );

To:

$options['group_nid'][] = array(
    'name' => $label,
    'params' => array(
      'module' => 'node', // specify module
      'field' => 'group_nid', // specify field
      'value' => $group_nid
    ),
    'link' => 'node/'. $group_nid,
  );

Blessing!

develcuy’s picture

develcuy commented

@jgraham, thanks for your patch :)

Just committed a fix, please check at CVS and be sure to use original beta12, no need to patch.

Blessings!

develcuy’s picture

develcuy commented
Version: 5.x-1.0-beta2 » 5.x-1.x-dev
Status: Active » Fixed
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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