As of simplenews_roles_nodeapi you only support $node-type=='simplenews'.
Simplenews allows us to enable multiple newsletter issue content types.
See corresponding code from simplenews.module

  // Operate only on node types set in 'simplenews_content_types' variable.
  if (!in_array($node->type, variable_get('simplenews_content_types', array('simplenews')))) {
    return;
  }

Comments

joachim’s picture

Status: Active » Fixed
StatusFileSize
new772 bytes

Thanks for reporting this :)

Committed this patch.

Though as you can probably see from the code, our hook_nodeapi doesn't actually do anything. It used to request a sync, but when I took over this module I couldn't think of a reason why one would be needed there. If there's a case I've overlooked, please file an issue!

miro_dietiker’s picture

oh.. didn't realize that... just well... gimme another minute to think about that ;-)

Status: Fixed » Closed (fixed)

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