Set notify default

logintoboggan set the notify default for users

i'm afraid i don't know what you're referring to. can you be more specific?

I'll take a wild guess that he's referring to the Notify module. At least it's a feature I want for that module. It doesn't feel right to implement it in Logintoboggan. I'd expect that to be a setting in Notify (so that it would be available with or without Logintoboggan).

Oops, slippery mouse.

this doesn't seem like a logintoboggan feature request. recategorizing.

This would be a great feature. I have hacked my current copy of notify.module to do just this. It was as easy as updating notify_user to the following:

<?php
function notify_user($type, &$edit, &$user, $category = NULL) {
  switch ($type) {
    case 'insert':
      db_query('INSERT INTO {notify} VALUES(%d,1,1,1,0,0)', $user->uid);
      break;
    case 'delete':
      db_query('DELETE FROM {notify} WHERE uid = %d', $user->uid);
      break;
  }
}
?>

I have hard coded the values of the different options but breaking this out to an administrative form should be pretty darn simple. If I get the time I may do this and provide the code.

Hi there,
This is a great idea, and would make Notify much more useful to non-technical users, who, on my site, don't know how to set this up, or are not aware it exists.
I'll try the hard coding, but would feel much safer if it was included in the module itself, with an admin toggle.

Okay, here is a patch to the latest stable release (5.x-1-1) that should provide admin options to allow for default user notify settings. I have not tested at all yet (should be able to tomorrow), so use at your own risk.

I have not tested the patch.
Just a small review on the code style:
* don't use tabs, but 2 spaces.
* put the '#type' => 'radios', on their own line, aligned with the other array elements.

Note: I will not review this patch further, nor consider it for inclusion until the second list of issues (bugs) is properly dealt with:
http://drupal.org/node/159427

Ok, I fixed the formatting issues (FYI...I had simply copied and pasted the code from elsewhere in the same file) and tested it and it seems to work just great!

will this work if you don't use logintobbaggan?

I would like to have an option I can set to turn notification on by default. Could this be included when upgrades are done for version 6? I looked at the patch but I don't know php or MySQL and I'm hesitant to start applying patches.

Applied the patch on #9 against 5.x-1.x-dev and it is working fine for about two months in my live site. It's a nice feature addition that should get in to the module.

Answering #12: the patch works independently of logintoboggan.

This feature request is almost five years old. Drupal 5 is no longer supported.

Default settings for notifications for new users is implemented and works for Drupal 6 and Drupal 7 versions (I've now idea when it was fixed, but has verified that it works now. It is part of the Notify Admin UI and is idependent of " logintoboggan").

The description of the option can be improved. I will change "Notification checkbox default on new user registration form" to something like "Receive e-mail notifications checked as default for new users" in the next dev snapshot.

If someone wants this backported to Drupal 5, they need to re-open in two weeks.

Duplicate: http://drupal.org/node/123504