I just installed this on my local Drupal 7 test bed environment. I enabled a number of the sub-modules. And now when I go to load my node pages I get the following error: Fatal error: Call to undefined function notifications_ui_node_subscriptions()

It points to: /sites/all/modules/contrib/notifications/notifications_ui/notifications_ui.module on line 140

I have also found the function call on line 223. It seems there is something incomplete in here. I have tried this in the Alpha2 and Dev downloads.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jason.fisher’s picture

Priority: Major » Critical

Same issue. I can't find the function anywhere?

riverole’s picture

Hello.

I'm facing the same issue when trying to print a comment form. I was searching through internet and I find that the function in Drupal 6 was:

function notifications_ui_node_subscriptions($node, $display) {
if (notifications_ui_user_access() && notifications_ui_node_options($node->type, $display)) {
return notifications_ui_object_subscriptions('node', $node);
}
}

I searched through the Notifications module and the three functions that are used exists and are defined, so I just added this function to Notifications.ui.module. I know this means hacking the module and I will go to hell doing that, but I need this to work and it seems that this bug was not solved. I have no idea about the implications about these changes (I have not analyzed accurately how the module works and so on) and I'm still not so confident in order to make a "patch" so If anybody wants to do that, that would be great, sure. I not, and you don't matter to go to hell, just copy this to notifications.ui.module and be happy.

DamienMcKenna’s picture

jacob.embree’s picture

Version: 7.x-1.0-alpha2 » 7.x-1.x-dev
jacob.embree’s picture

There is a lot of code that was broken by e86557d75e3cdf88dbac056b9c46576db06caf76. This patch attempts to fix calls to notifications_ui_node_subscriptions and related things. Anything that depends on notifications_ui_node_options() seems to be useless now.