[wp-trac] [WordPress Trac] #39676: Remove restriction of minimum site title length of 4 characters

WordPress Trac noreply at wordpress.org
Tue Mar 28 21:26:28 UTC 2017


#39676: Remove restriction of minimum site title length of 4 characters
--------------------------------------------------+------------------------
 Reporter:  flixos90                              |       Owner:  flixos90
     Type:  enhancement                           |      Status:  reopened
 Priority:  normal                                |   Milestone:  4.8
Component:  Networks and Sites                    |     Version:
 Severity:  normal                                |  Resolution:
 Keywords:  has-patch has-unit-tests 2nd-opinion  |     Focuses:  multisite
--------------------------------------------------+------------------------

Comment (by jeremyfelt):

 It feels a little strange to add a new function just to enforce this as a
 default filter. What if we applied the filter directly to the character
 limit? (Ignore the filter name)

 {{{
 $min_char_count = absint( apply_filters( 'minimum_character_count', 4 ) );

 if ( strlen( $blogname ) < $min_char_count ) {
     $errors->add('blogname',  __( "Site name must be at least
 $min_char_count characters." ) );
 }
 }}}

 Also - we could just revert most of this and remove the `is_super_admin()`
 check. The existing `wpmu_validate_blog_signup` filter, while somewhat
 painful, makes it possible to filter this already. It'd sure be nice if
 those error keys were unique though. :)

--
Ticket URL: <https://core.trac.wordpress.org/ticket/39676#comment:10>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list