Notify turn on by default - apply to existing users

nethome - November 3, 2005 - 04:27
Project:Notify
Version:5.x-1.1
Component:Code
Category:support request
Priority:normal
Assigned:Unassigned
Status:active
Description

I tries some of the splutions found earlier, but they don't seem to work well as I keep getting various erro. Does anyone know how to change (and where tolook) to make the feature turned on automatically when a new user sign up?

#1

rfay - November 3, 2005 - 13:21

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

#2

markstos - November 10, 2005 - 02:40
Category:feature request» support request
Priority:critical» normal
Status:active» fixed

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

#3

Anonymous - November 24, 2005 - 03:00
Status:fixed» closed

#4

leon83 - April 29, 2007 - 14:33

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 ;)

AttachmentSize
notify-module.txt 21.33 KB

#5

kriptonita - September 13, 2007 - 20:19

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 = '';

#6

mdowsett - March 31, 2008 - 20:14
Version:4.6.x-1.x-dev» 5.x-1.1
Status:closed» active

any idea if these patches work for v5?

#7

mdowsett - April 1, 2008 - 15:00
Status:active» closed

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.

#8

mdowsett - April 15, 2008 - 19:13
Title:Notify turn on by default» Notify turn on by default - apply to existing users
Status:closed» active

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

 
 

Drupal is a registered trademark of Dries Buytaert.