Changeset 6480 for trunk/bp-blogs/bp-blogs-template.php

Timestamp:

10/31/2012 10:57:55 AM (12 years ago)

Author:

djpaul

Message:

Increase the maxlength attribute on the site registration template from 50 to 63.

• 63 characters is the maximum length of domains per ​RFC1034.
• While not universally enforced, some systems such as some DNS servers do follow the RFC.
• Server-side validation has been intentionally omitted as this should be handled upstream in WordPress; see ​#WP21994

Fixes #4560, props humanshell

File:

• trunk/bp-blogs/bp-blogs-template.php (modified) (1 diff)

trunk/bp-blogs/bp-blogs-template.php

@@ -646 +646 @@
 
     if ( !is_subdomain_install() )
-        echo '<span class="prefix_address">' . $current_site->domain . $current_site->path . '</span> <input name="blogname" type="text" id="blogname" value="'.$blogname.'" maxlength="50" /><br />';
+        echo '<span class="prefix_address">' . $current_site->domain . $current_site->path . '</span> <input name="blogname" type="text" id="blogname" value="'.$blogname.'" maxlength="63" /><br />';
     else
-        echo '<input name="blogname" type="text" id="blogname" value="'.$blogname.'" maxlength="50" /> <span class="suffix_address">.' . bp_blogs_get_subdomain_base() . '</span><br />';
+        echo '<input name="blogname" type="text" id="blogname" value="'.$blogname.'" maxlength="63" /> <span class="suffix_address">.' . bp_blogs_get_subdomain_base() . '</span><br />';
 
     if ( !is_user_logged_in() ) {