Changeset 7382 for trunk/bp-members/bp-members-functions.php

Timestamp:

09/17/2013 01:57:48 AM (11 years ago)

Author:

r-a-y

Message:

Clarify error message for a user attempting to register with capital
letters on a multisite install.

Because WP multisite does not allow a user to register with capital
letters, we need to do an explicit check for capital letters and throw
up a better error message to avoid confusion.

Fixes #5175.

File:

• trunk/bp-members/bp-members-functions.php (modified) (1 diff)

trunk/bp-members/bp-members-functions.php

@@ -1188 +1188 @@
         $errors->add( 'user_name', __( 'That username is not allowed', 'buddypress' ) );
 
-    if ( ! validate_username( $user_name ) )
-        $errors->add( 'user_name', __( 'Usernames can contain only letters, numbers, ., -, *, and @', 'buddypress' ) );
+    if ( ! validate_username( $user_name ) ) {
+        // Check for capital letters when on multisite.
+        //
+        // If so, throw a different error message.
+        // @see #5175
+        if ( is_multisite() ) {
+            $match = array();
+            preg_match( '/[A-Z]/', $user_name, $match );
+    
+            if ( ! empty( $match ) ) {
+                $errors->add( 'user_name', __( 'Username must be in lowercase characters.', 'buddypress' ) );
+            }
+
+        } else {
+            $errors->add( 'user_name', __( 'Usernames can contain only letters, numbers, ., -, *, and @', 'buddypress' ) );
+        }
+    }
 
     if( strlen( $user_name ) < 4 )