Notify turn on by default - apply to existing users

Notify does not currently do new user notification. However, the patch on this page did the job for me.

This is a support request, not a critical feature request. Since someone responded with a patch, I'm considering it resolved.

I've made some changes in Notify module for my site to solve this problem. If you want use my solution, just change default file 'notify.module' to my attached (change extension at first - just delete .txt).

New features:
1) at page /admin/content/notify there is new button - 'Subscribe all'. When you press it, all existing users, excluding these already subscribed for something, will be subscribed to teasers of materials.
2) all new users are automatically subscribed to teasers of materials.

In some way this module is breaking freedom of users, so I don't recommend it for mass use. The better way is to add checkbox 'Subscribe' (enabled by default) at user registration page - but it needs another 20 minutes of my work, I really don't need it ;)

Hi,

I wish to collaborate modifying the 'all uid' query in order to avoid to modify the setting of users that already have their own settings.

This also avoid SQL warnings from duplicate entries:

function notify_admin_settings_validate($form_id, $form_values) {
< if ($form_values['op']=='Subscribe all') {
< $result = db_query('SELECT uid FROM {users} WHERE uid not in (select uid from {notify} where 1)');
< $s = '';

any idea if these patches work for v5?

I did try this and it worked like a charm! Many thanks.

I also applied the change to show edited content (in #1 of http://drupal.org/node/150109) in the notifications and it worked as well.

are you able to post a patch for this? I've changed my notify.module already so I need to know what lines have changed. I got your module to work once but I've since used another version of the notify module to allow settings to be changed so it restricts notifications by node type and now need to apply your fix to set all users settings to what the admin sets.

If you cant' provide a proper patch, can you document what lines you changed to what?

MANY thanks

I think there are 2 or 3 different needs being talked about here. If current versions don't resolve your issue, please post a new issue. If you're trying to make bulk changes to settings on an existing installation, your best bet is to write a SQL query for it. Notify module is unlikely to support bulk operations on user settings.