Changeset 44454 for trunk/tests/phpunit/tests/post.php

Timestamp:

01/08/2019 03:46:58 AM (8 years ago)

Author:

pento

Message:

Permalinks: Add a pre_wp_unique_post_slug filter.

Returning a non-null value on this fillter will cause wp_unique_post_slug() to return early with that value, skipping potentially expensive database queries on some sites.

Props coffee2code, javorszky, iCaleb.
Fixes #21112.

File:

• trunk/tests/phpunit/tests/post.php (modified) (1 diff)

trunk/tests/phpunit/tests/post.php

@@ -1358 +1358 @@
         }
 
+        /**
+         * Test ensuring that the post_slug can be filtered with a custom value short circuiting the built in
+         * function that tries to create a unique name based on the post name.
+         *
+         * @see wp_unique_post_slug()
+         * @ticket 21112
+         */
+        function test_pre_wp_unique_post_slug_filter() {
+                add_filter( 'pre_wp_unique_post_slug', array( $this, 'filter_pre_wp_unique_post_slug' ), 10, 6 );
+
+                $post_id = $this->factory->post->create(
+                        array(
+                                'title'       => 'An example',
+                                'post_status' => 'publish',
+                                'post_type'   => 'page',
+                        )
+                );
+                $post    = get_post( $post_id );
+                $this->assertEquals( 'override-slug-' . $post->post_type, $post->post_name );
+
+                remove_filter( 'pre_wp_unique_post_slug', array( $this, 'filter_pre_wp_unique_post_slug' ), 10, 6 );
+        }
+
+        function filter_pre_wp_unique_post_slug( $default, $slug, $post_ID, $post_status, $post_type, $post_parent ) {
+                return 'override-slug-' . $post_type;
+        }
 }