Changeset 2177

Timestamp:

Dec 11, 2007, 7:04:07 AM (16 years ago)

Author:

rub

Message:

Resolved issue 0000784: Mail notification disabled on register user

Location:

branches/branch-1_7

Files:

• admin/user_list.php (modified) (1 diff)
• include/functions_user.inc.php (modified) (4 diffs)
• include/user.inc.php (modified) (1 diff)
• register.php (modified) (2 diffs)

branches/branch-1_7/admin/user_list.php

@@ -194 +194 @@
 {
   $page['errors'] = register_user(
-    $_POST['login'], $_POST['password'], $_POST['email']);
+    $_POST['login'], $_POST['password'], $_POST['email'], false);
 
   if (count($page['errors']) == 0)

branches/branch-1_7/include/functions_user.inc.php

@@ -45 +45 @@
 }
 
-function register_user($login, $password, $mail_address, $errors = array())
+function register_user($login, $password, $mail_address,
+  $with_notification = true, $errors = array())
 {
   global $lang, $conf;
@@ -92 +93 @@
     mass_inserts(USERS_TABLE, array_keys($insert), array($insert));
 
-  // Assign by default groups
-  {
-    $query = '
+    // Assign by default groups
+    {
+      $query = '
 SELECT id
   FROM '.GROUPS_TABLE.'
@@ -100 +101 @@
   ORDER BY id ASC
 ;';
-    $result = pwg_query($query);
-
-    $inserts = array();
-    while ($row = mysql_fetch_array($result))
-    {
-      array_push
-      (
-        $inserts,
-        array
+      $result = pwg_query($query);
+
+      $inserts = array();
+      while ($row = mysql_fetch_array($result))
+      {
+        array_push
         (
-          'user_id' => $next_id,
-          'group_id' => $row['id']
-        )
-      );
+          $inserts,
+          array
+          (
+            'user_id' => $next_id,
+            'group_id' => $row['id']
+          )
+        );
+      }
     }
 
@@ -121 +123 @@
       mass_inserts(USER_GROUP_TABLE, array('user_id', 'group_id'), $inserts);
     }
-  }
 
     create_user_infos($next_id);
+
+    if ($with_notification and $conf['email_admin_on_new_user'])
+    {
+      include_once(PHPWG_ROOT_PATH.'include/functions_mail.inc.php');
+      $username = $_POST['login'];
+      $admin_url = get_absolute_root_url()
+                   .'admin.php?page=user_list&username='.$username;
+
+      $keyargs_content = array
+      (
+        get_l10n_args('User: %s', $username),
+        get_l10n_args('Email: %s', $_POST['mail_address']),
+        get_l10n_args('', ''),
+        get_l10n_args('Admin: %s', $admin_url)
+      );
+
+      pwg_mail_notification_admins
+      (
+        get_l10n_args('Registration of %s', $username),
+        $keyargs_content
+      );
+    }
 
     trigger_action('register_user',

branches/branch-1_7/include/user.inc.php

@@ -66 +66 @@
   if (!($user['id'] = get_userid($_SERVER['REMOTE_USER'])))
   {
-    register_user($_SERVER['REMOTE_USER'], '', '');
+    register_user($_SERVER['REMOTE_USER'], '', '', false);
     $user['id'] = get_userid($_SERVER['REMOTE_USER']);
   }

branches/branch-1_7/register.php

@@ -53 +53 @@
                     $_POST['password'],
                     $_POST['mail_address'],
+                    true,
                     $errors);
 
@@ -58 +59 @@
   {
     $user_id = get_userid($_POST['login']);
-    log_user( $user_id, false);
-
-    if ($conf['email_admin_on_new_user'])
-    {
-      include_once(PHPWG_ROOT_PATH.'include/functions_mail.inc.php');
-      $username = $_POST['login'];
-      $admin_url = get_absolute_root_url()
-                   .'admin.php?page=user_list&username='.$username;
-
-      $keyargs_content = array
-      (
-        get_l10n_args('User: %s', $username),
-        get_l10n_args('Email: %s', $_POST['mail_address']),
-        get_l10n_args('', ''),
-        get_l10n_args('Admin: %s', $admin_url)
-      );
-
-      pwg_mail_notification_admins
-      (
-        get_l10n_args('Registration of %s', $username),
-        $keyargs_content
-      );
-    }
+    log_user($user_id, false);
     redirect(make_index_url());
   }