Changeset 31323 for trunk/tests/phpunit/tests/admin/includesPost.php

Timestamp:

02/02/2015 04:49:23 PM (10 years ago)

Author:

boonebgorges

Message:

In get_sample_permalink(), override 'future' status before generating permalink.

In [31114], get_permalink() was modified to prevent pretty permalinks from
being generated for posts with the 'future' post status. This inadvertently
broke the pretty permalink preview for scheduled posts. The fix is to include
the 'future' status in the list of statuses that get_sample_permalink() fakes
as 'publish' before it fetches a permalink.

Props DrewAPicture.
Fixes #30910.

File:

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

trunk/tests/phpunit/tests/admin/includesPost.php

@@ -179 +179 @@
     }
 
+    /**
+     * @ticket 30910
+     */
+    public function test_get_sample_permalink_should_return_pretty_permalink_for_posts_with_post_status_future() {
+        global $wp_rewrite;
+
+        $old_permalink_structure = get_option( 'permalink_structure' );
+        $permalink_structure = '%postname%';
+        $wp_rewrite->set_permalink_structure( "/$permalink_structure/" );
+        flush_rewrite_rules();
+
+        $future_date = date( 'Y-m-d H:i:s', time() + 100 );
+        $p = $this->factory->post->create( array( 'post_status' => 'future', 'post_name' => 'foo', 'post_date' => $future_date ) );
+
+        $found = get_sample_permalink( $p );
+        $expected = trailingslashit( home_url( $permalink_structure ) );
+
+        $this->assertSame( $expected, $found[0] );
+
+        $wp_rewrite->set_permalink_structure( $old_permalink_structure );
+        flush_rewrite_rules();
+    }
 }