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

Timestamp:

01/08/2019 03:46:58 AM (6 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;
+    }
 }