Context Navigation

Back to Ticket #39676

Ticket #39676: 39676.3.diff

File 39676.3.diff, 2.7 KB (added by flixos90, 9 years ago)

src/wp-includes/ms-functions.php

         if ( in_array( $blogname, $illegal_names ) )
                 $errors->add('blogname',  __( 'That name is not allowed.' ) );
+        if ( strlen( $blogname ) < 4 ) {
+        /**
+         * Filters the minimum site name length required when validating a site signup.
+         *
+         * @since 4.8.0
+         *
+         * @param int $length The minimum site name length. Default 4.
+         */
+        $minimum_site_name_length = apply_filters( 'minimum_site_name_length', 4 );
+        if ( strlen( $blogname ) < $minimum_site_name_length ) {
                 $errors->add('blogname',  __( 'Site name must be at least 4 characters.' ) );
+        }

tests/phpunit/tests/multisite/wpmuValidateBlogSignup.php

         protected static $existing_blog_name = 'existingsitefoo';
         protected static $existing_blog_id;
+        protected $minimum_site_name_length = 4;
         public static function wpSetUpBeforeClass( $factory ) {
                 self::$super_admin_id = $factory->user->create();
                 grant_super_admin( self::$super_admin_id );
 …
                 $result = wpmu_validate_blog_signup( self::$existing_user_login, 'Foo Site Title', get_userdata( self::$existing_user_id ) );
                 $this->assertEmpty( $result['errors']->get_error_codes() );
+        }
+        /**
+         * @ticket 39676
+         *
+         * @dataProvider data_filter_minimum_site_name_length
+         */
+        public function test_filter_minimum_site_name_length( $site_name, $minimum_length, $expect_error ) {
+                $this->minimum_site_name_length = $minimum_length;
+                add_filter( 'minimum_site_name_length', array( $this, 'filter_minimum_site_name_length' ) );
+                $result = wpmu_validate_blog_signup( $site_name, 'Site Title', get_userdata( self::$super_admin_id ) );
+                remove_filter( 'minimum_site_name_length', array( $this, 'filter_minimum_site_name_length' ) );
+                $this->minimum_site_name_length = 4;
+                if ( $expect_error ) {
+                        $this->assertContains( 'blogname', $result['errors']->get_error_codes() );
+                } else {
+                        $this->assertEmpty( $result['errors']->get_error_codes() );
+                }
+        }
+        public function data_filter_minimum_site_name_length() {
+                return array(
+                        array( 'fooo', 5, true ),
+                        array( 'foooo', 5, false ),
+                        array( 'foo', 4, true ),
+                        array( 'fooo', 4, false ),
+                        array( 'fo', 3, true ),
+                        array( 'foo', 3, false ),
+                );
+        }
+        public function filter_minimum_site_name_length() {
+                return $this->minimum_site_name_length;
+        }
+}
 endif;

Download in other formats:

Original Format

Make WordPress Core

Context Navigation

Ticket #39676: 39676.3.diff

src/wp-includes/ms-functions.php

tests/phpunit/tests/multisite/wpmuValidateBlogSignup.php

Download in other formats: