My site is now sending notifications on "Blocked content types" to all users subscribed to a taxonomy term, when a new node is created with that taxonomy present even though it is on the "Blocked content types"

CommentFileSizeAuthor
#5 subscriptions.zip140.99 KBbreak9

Comments

salvis’s picture

salvis
he/his
commented
Component: subscriptions_event » Code
Status: Active » Needs work

Now that I look at it, I don't think this has ever worked right. Are you implying it did work in BETA5?

break9’s picture

break9 commented

yes it worked for me since Dec 2008 when I first added this module.

salvis’s picture

salvis
he/his
commented

I also thought it worked, but I cannot find any code in BETA5 (nor 1.1) that would actually do make it work...

break9’s picture

break9 commented

Yep, it has worked great for the last year until I updated. I have thousands of users that are affected by the change. they all noticed immediately. Hence me in a deep pile of ___.

Any and all help is appreciated on both issues.

break9’s picture

break9 commented
StatusFileSize
new subscriptions.zip140.99 KB

here is the complete working as expected version of subscriptions I had on my site. Niether this nor the http://drupal.org/node/610106 issues are present in the version. zip is attached

The only code changes i made were:

subscriptions.taxonomy.module;
********remove the words "to content in"

subscription.module;
$items['user/%user/subscriptions/'. $stype] = array(
'title' => $data['title'],
********'type' => MENU_CALLBACK,
'file' => 'subscriptions.admin.inc',
'page callback' => 'subscriptions_page',
'page arguments' => array(1, $stype),
'access callback' => '_subscriptions_access',
'access arguments' => array(1, $data['access']),
'weight' => (isset($data['weight']) ? $data['weight'] : 0),
);

salvis’s picture

salvis
he/his
commented
Status: Needs work » Fixed

I've committed a fix to the -dev version (give it up to 12h to be repackaged).

Please let us know if this works for you now.

(Unfortunately, I don't know what to do about your other issue...)

break9’s picture

break9 commented

I will test and report back.

Yes, the other issue is very strange? i don't know why it doesn't want the $account parameter passed to the function call. It is unfortunate since that is the primary interface for this module on our site. I will have to use the older release until I can sort out what is happening.

Thanks for the update on this issue, and the module in general it is great!!

Status: Fixed » Closed (fixed)

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