From 02576e003778c9e06fb252d90a9a232db6b6c367 Mon Sep 17 00:00:00 2001 From: e107steved Date: Sat, 30 Dec 2006 22:39:43 +0000 Subject: [PATCH] Impose minimum length on display name --- e107_languages/English/lan_signup.php | 5 +++-- e107_languages/English/lan_usersettings.php | 7 ++++--- signup.php | 12 ++++++++++-- usersettings.php | 12 +++++++++--- 4 files changed, 26 insertions(+), 10 deletions(-) diff --git a/e107_languages/English/lan_signup.php b/e107_languages/English/lan_signup.php index f726ac7c9..346665fbc 100644 --- a/e107_languages/English/lan_signup.php +++ b/e107_languages/English/lan_signup.php @@ -4,8 +4,8 @@ | e107 website system - Language File. | | $Source: /cvs_backup/e107_0.8/e107_languages/English/lan_signup.php,v $ -| $Revision: 1.2 $ -| $Date: 2006-12-27 22:26:47 $ +| $Revision: 1.3 $ +| $Date: 2006-12-30 22:39:43 $ | $Author: e107steved $ +----------------------------------------------------------------------------+ */ @@ -115,6 +115,7 @@ define("LAN_SIGNUP_52", "Incorrect Password"); define("LAN_SIGNUP_53", "field failed validation test"); define("LAN_SIGNUP_54", "Click here to fill in your details to register"); define("LAN_SIGNUP_55", "That display name is too long. Please choose another"); +define("LAN_SIGNUP_56", "That display name is too short. Please choose another"); diff --git a/e107_languages/English/lan_usersettings.php b/e107_languages/English/lan_usersettings.php index 5216dd57e..289a617d9 100644 --- a/e107_languages/English/lan_usersettings.php +++ b/e107_languages/English/lan_usersettings.php @@ -4,9 +4,9 @@ | e107 website system - Language File. | | $Source: /cvs_backup/e107_0.8/e107_languages/English/lan_usersettings.php,v $ -| $Revision: 1.1.1.1 $ -| $Date: 2006-12-02 04:34:40 $ -| $Author: mcfly_e107 $ +| $Revision: 1.2 $ +| $Date: 2006-12-30 22:39:43 $ +| $Author: e107steved $ +----------------------------------------------------------------------------+ */ define("PAGE_NAME", "User Settings"); @@ -106,5 +106,6 @@ define("LAN_USET_8", "Signature / Timezone"); define("LAN_USET_9", "Some of the required fields (marked with a *) are missing from your settings."); define("LAN_USET_10","Please update your settings now, in order to proceed."); define("LAN_USET_11", "That user name cannot be accepted as valid, please choose a different user name"); +define("LAN_USET_12", "That display name is too short. Please choose another"); ?> \ No newline at end of file diff --git a/signup.php b/signup.php index eb7916e5f..d3ab5d41d 100644 --- a/signup.php +++ b/signup.php @@ -11,8 +11,8 @@ | GNU General Public License (http://gnu.org). | | $Source: /cvs_backup/e107_0.8/signup.php,v $ -| $Revision: 1.3 $ -| $Date: 2006-12-27 22:26:41 $ +| $Revision: 1.4 $ +| $Date: 2006-12-30 22:39:43 $ | $Author: e107steved $ +----------------------------------------------------------------------------+ */ @@ -394,6 +394,14 @@ if (isset($_POST['register'])) $_POST['name'] = $_POST['loginname']; } + // Impose a minimum length on display name + $_POST['name'] = trim($_POST['name']); + if (strlen($_POST['name']) < 2) + { + $error_message .= LAN_SIGNUP_56."\\n"; + $error = TRUE; + } + // Check for disallowed names. if(isset($pref['signup_disallow_text'])) { diff --git a/usersettings.php b/usersettings.php index 91e302eaa..27a2e117a 100644 --- a/usersettings.php +++ b/usersettings.php @@ -11,9 +11,9 @@ | GNU General Public License (http://gnu.org). | | $Source: /cvs_backup/e107_0.8/usersettings.php,v $ -| $Revision: 1.1.1.1 $ -| $Date: 2006-12-02 04:33:10 $ -| $Author: mcfly_e107 $ +| $Revision: 1.2 $ +| $Date: 2006-12-30 22:39:43 $ +| $Author: e107steved $ +----------------------------------------------------------------------------+ */ @@ -181,6 +181,12 @@ if (isset($_POST['updatesettings'])) } // $username = strip_tags($_POST['username']); + // Impose a minimum length on display name + $_POST['username'] = trim($_POST['username']); + if (strlen($_POST['username']) < 2) + { + $error .= LAN_USET_12."\\n"; + } $user_sess = ""; if ($file_userfile['error'] != 4)