Changeset 2115

Timestamp:

Oct 2, 2007, 12:07:47 AM (17 years ago)

Author:

rub

Message:

Resolved 0000759: email unique for each user

Location:

trunk

Files:

• include/functions_user.inc.php (modified) (3 diffs)
• language/en_UK.iso-8859-1/common.lang.php (modified) (1 diff)
• language/en_UK.iso-8859-1/install.lang.php (modified) (3 diffs)
• language/fr_FR.iso-8859-1/common.lang.php (modified) (1 diff)
• language/fr_FR.iso-8859-1/install.lang.php (modified) (1 diff)
• plugins/c13y_upgrade/initialize.inc.php (modified) (1 diff)
• plugins/c13y_upgrade/language/en_UK.iso-8859-1/plugin.lang.php (modified) (1 diff)
• plugins/c13y_upgrade/language/fr_FR.iso-8859-1/plugin.lang.php (modified) (1 diff)

trunk/include/functions_user.inc.php

@@ -25 +25 @@
 // +-----------------------------------------------------------------------+
 
-// validate_mail_address verifies whether the given mail address has the
-// right format. ie someone@domain.com "someone" can contain ".", "-" or
-// even "_". Exactly as "domain". The extension doesn't have to be
-// "com". The mail address can also be empty.
+// validate_mail_address:
+//   o verifies whether the given mail address has the
+//     right format. ie someone@domain.com "someone" can contain ".", "-" or
+//     even "_". Exactly as "domain". The extension doesn't have to be
+//     "com". The mail address can also be empty.
+//   o check if address could be empty
+//   o check if address is not used by a other user
 // If the mail address doesn't correspond, an error message is returned.
-function validate_mail_address( $mail_address )
-{
-  global $lang, $conf;
+// 
+function validate_mail_address($mail_address)
+{
+  global $conf;
 
   if (empty($mail_address) and
@@ -39 +43 @@
     return '';
   }
+
   $regex = '/^[\w-]+(\.[\w-]+)*@[\w-]+(\.[\w-]+)*\.[a-z]+$/';
   if ( !preg_match( $regex, $mail_address ) )
   {
-    return $lang['reg_err_mail_address'];
+    return l10n('reg_err_mail_address');
+  }
+  
+  if (defined("PHPWG_INSTALLED") and !empty($mail_address))
+  {
+    $query = '
+select count(*)
+from '.USERS_TABLE.'
+where upper('.$conf['user_fields']['email'].') = upper(\''.$mail_address.'\')
+;';
+    list($count) = mysql_fetch_array(pwg_query($query));
+    if ($count != 0)
+    {
+      return l10n('reg_err_mail_address_dbl');
+    }
   }
 }
@@ -48 +67 @@
 function register_user($login, $password, $mail_address, $errors = array())
 {
-  global $lang, $conf;
+  global $conf;
 
   if ($login == '')
   {
-    array_push($errors, $lang['reg_err_login1']);
+    array_push($errors, l10n('reg_err_login1'));
   }
   if (ereg("^.* $", $login))
   {
-    array_push($errors, $lang['reg_err_login2']);
+    array_push($errors, l10n('reg_err_login2'));
   }
   if (ereg("^ .*$", $login))
   {
-    array_push($errors, $lang['reg_err_login3']);
+    array_push($errors, l10n('reg_err_login3'));
   }
   if (get_userid($login))
   {
-    array_push($errors, $lang['reg_err_login5']);
+    array_push($errors, l10n('reg_err_login5'));
   }
   $mail_error = validate_mail_address($mail_address);

trunk/language/en_UK.iso-8859-1/common.lang.php

@@ -638 +638 @@
 // --------- Starting below: New or revised $lang ---- from Butterfly (1.8)
 $lang['Administrator, webmaster and special user cannot use this method'] = 'Administrator, webmaster and special user cannot use this method';
+$lang['reg_err_mail_address_dbl'] = 'a user use already this mail address';
 ?>

trunk/language/en_UK.iso-8859-1/install.lang.php

@@ -5 +5 @@
 // | Copyright (C) 2003-2007 PhpWebGallery Team - http://phpwebgallery.net |
 // +-----------------------------------------------------------------------+
-// | branch        : BSF (Best So Far)
-// | file          : $RCSfile$
+// | file          : $Id$
 // | last update   : $Date$
 // | last modifier : $Author$
@@ -33 +32 @@
 $lang['Start_Install'] = 'Start Install';
 $lang['reg_err_mail_address'] = 'mail address must be like xxx@yyy.eee (example : jack@altern.org)';
-$lang['mail_webmaster'] = 'Webmaster mail adress';
-$lang['mail_webmaster_info'] = 'Visitors will be able to contact site administrator with this mail';
-$lang['reg_err_mail_address'] = 'e-mail address refused, it must be like name@server.com';
 
 $lang['install_webmaster'] = 'Webmaster login';
@@ -73 +69 @@
 <li>this login will enable you to access to the administration panel and to the instructions in order to place pictures in your directories</li>
 </ul>';
-$lang['conf_mail_webmaster'] = 'Webmaster mail adress';
+$lang['conf_mail_webmaster'] = 'Webmaster mail address';
 $lang['conf_mail_webmaster_info'] = 'Visitors will be able to contact site administrator with this mail';
 ?>

trunk/language/fr_FR.iso-8859-1/common.lang.php

@@ -637 +637 @@
 // --------- Starting below: New or revised $lang ---- from Butterfly (1.8)
 $lang['Administrator, webmaster and special user cannot use this method'] = 'Administrateur, webmestre et utilisateur spécial ne peuvent pas utiliser cette méthode';
+$lang['reg_err_mail_address_dbl'] = 'un utilisateur utilise déjà cette adresse e-mail';
 ?>

trunk/language/fr_FR.iso-8859-1/install.lang.php

@@ -5 +5 @@
 // | Copyright (C) 2003-2007 PhpWebGallery Team - http://phpwebgallery.net |
 // +-----------------------------------------------------------------------+
-// | branch        : BSF (Best So Far)
-// | file          : $RCSfile$
+// | file          : $Id$
 // | last update   : $Date$
 // | last modifier : $Author$

trunk/plugins/c13y_upgrade/initialize.inc.php

@@ -40 +40 @@
   $result = array();
 
+  /* Check user with same e-mail */
+  $query = '
+select count(*)
+from '.USERS_TABLE.'
+group by upper('.$conf['user_fields']['email'].')
+having count(*) > 1
+limit 0,1
+;';
+
+  if (mysql_fetch_array(pwg_query($query)))
+  {
+    $result[] = get_c13y(
+      l10n('c13y_exif_dbl_email_user'),
+      null,
+      null,
+      l10n('c13y_exif_correction_dbl_email_user'));
+  }
+
+
+  /* Check if this plugin must deactivate */
   if (count($result) === 0)
   {

trunk/plugins/c13y_upgrade/language/en_UK.iso-8859-1/plugin.lang.php

@@ -28 +28 @@
 $lang['c13y_upgrade_no_anomaly'] = 'No anomaly detected after application upgrade';
 $lang['c13y_upgrade_deactivate'] = 'You can deactivate "Check upgrades" plugin';
-
+$lang['c13y_exif_dbl_email_user'] = 'Users with same email address';
+$lang['c13y_exif_correction_dbl_email_user'] = 'Delete duplicate users';
 ?>

trunk/plugins/c13y_upgrade/language/fr_FR.iso-8859-1/plugin.lang.php

@@ -28 +28 @@
 $lang['c13y_upgrade_no_anomaly'] = 'Pas d\'anomalie détectée après la mise à jour de l\'application';
 $lang['c13y_upgrade_deactivate'] = 'Vous pouvez désactiver le plugin "Check upgrades"';
+$lang['c13y_exif_dbl_email_user'] = 'Utilisateurs avec la même adresse email';
+$lang['c13y_exif_correction_dbl_email_user'] = 'Supprimez les utilisateurs en double';
 
 ?>