Hy,
I download notifications module and install. I want enable taxonomy notifications via the administrator menu but it's write : "This version is incompatible with the 6.2 version of Drupal core."
Thus I open notifications_tags.info and i replace :

dependencies[] = notifications  notifications_content taxonomy

by

dependencies[] = notifications
dependencies[] = notifications_content
dependencies[] = taxonomy

After that, I can enable it.
But when I try to subscribe at a term taxonomy (in /user/1/notifications/taxonomy), it doesn't take my modification into consideration.
After research in notifications_tags.module, I replaced in notifications_tags_user_form($account) [line 200]:

$form['#submit']['notifications_content_form_submit'] = array();

by

$form['#submit'][] = 'notifications_content_form_submit';

After my modification, I can subscribe (my subscription is well save in database).
But the uid isn't corresponding. It's because the $account variable is null.
So in looking the function notifications_tags_user_page($account = NULL), I saw a call at the function drupal_get_form with 2 args (the second argument is $account). Before the call, the variable $account is good, but not after. So I think that the function drupal_get_form is not good called.
I modified the function notifications_tags_user_page($account = NULL) [line 126]:

return drupal_get_form('notifications_tags_user_form',$account);

by

return drupal_get_form('notifications_tags_user_form');

and I had add at beginning of the function notifications_tags_user_form() :

global $user;
  
  if (is_null($account)) {
    $account = $user;
  }

And now, I can use this pluging completely.
Sorry if you don't me understand very well, I don't speak very well English but I hope I explain me the better than possible.
If you can tell me if my modifications are good or not..
Thank you in advance

Comments

jose reyero’s picture

Status: Active » Fixed

Oh, it seems that module was forgotten during the upgrade.

A patch is much more useful and would save us both a lot of reading and writing ;-)... anyway, I followed (most of) your instructions and got it working. Thanks

Anonymous’s picture

Status: Fixed » Closed (fixed)

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