Option to Disable Notify Status selector on Registration form

I'd really like to know how to fix this too. I only wanted notify so our administrators would know when anonymous users post content.

To be clear, the presence of that check box does not in any way override the permission. If a user registers with it checked, but they do not have a role that permits them to receive notifications, then they will not receive notifications.

To prevent the selector from appearing, you can comment out line 126 of notify module as follows:

/**
 * Implementation of hook_user().
 */
function notify_user($type, &$edit, &$user, $category = NULL) {
  switch ($type) {
    case 'delete':
      db_query('DELETE FROM {notify} WHERE uid = %d', $user->uid);
      break;
    
    case 'register':
      //return _notify_user_reg_fields();  //THIS IS LINE #126. See  // added at front <--------------------
      break;
    
    case 'insert':
      if (isset($edit['notify_decision']) && $edit['notify_decision'] == 1){
        db_query('INSERT INTO {notify} (uid, status, node, teasers, comment) VALUES (%d, %d, %d, %d, %d)', $user->uid, 1, 1, 0, 0);
        $edit['notify_decision'] = NULL;
      }
      break;
    
  }
}

[dbl post]

[dbl post]

[dbl post -- something must be up with drupal.org's caching or DB replication, cause I swear this didn't seem to post at first.]

Since I made such a mess of this thread, I'll atone by offering this patch. It modifies the notify admin settings to allow for the registration check box to be removed.

@matt2000, although your patch provides a workaround for people who don't want the checkbox on the reg form, I IMHO think this is not the best way to handle this issue.

Since we already have an 'access notify' permission, there's no need for an additional settings. Just check user_acess('access notify' ) to know if the user should be presented with the checkbox or not.

Patch attached.

A valid point, but what about the scenario where you want to allow registered users to manage their own settings, but you don't want it as part of the registration process? I.e., all users are added to the list by default, but have the option to opt-out. Is this too rare a scenario to consider?

Hi Matt,

With your last patch, the administrator might force new user to have notification enabled only. I wanted exactly the opposite: I wanted my new users to enable this feature later, on their own will after registration, with notification disabled by default.

Here is a new patch with best of 2 worlds:
- hide/display Notification setting on registration form
- default value for new user : notification enabled or disabled

Mike

I like #10. Has anybody tested it?

Before the patch on a vanilla install:
check box appears registration form
'Enabled' by default

After patch:
check box still appeared on registration form
'Enabled' still set by default

So the new option to disable it on the admin settings form doesn't work?

Ok so upon further investigation:

- hide/display Notification setting on registration form
Works, but is enabled by default

- default value for new user : notification enabled or disabled
Works, but is enabled by default

This doesn't apply cleanly against HEAD or the DRUPAL-6--1 branch. Please reroll, and I will commit. I'd like to do a new release with several fixes in the next couple weeks.

Hi Matt, here is the correction of the patch against your latest dev version.

Needs review I think..

On further thought, I'm taking back my comment in #9, and using a solution like the one proposed in #8.

The goal in #10 is therefore possible by granting 'access notify' permission to authenticated users, but not anonymous users.

I won't be supporting an option to sign-up users for notifications without giving them notice or opportunity to opt-out.

Matt2000, with the latest patch, users HAVE opportunity to opt-out : via their "Notification settings" panel and via the footer on each email.

I have posted a patch in http://drupal.org/node/870812 that includes options to show email notification checkbox on registration without user permission needed and to set default status as desired along with a few other UI changes.