Context Navigation

Ticket #25523: post-status-perm-with-test.25523.diff

File post-status-perm-with-test.25523.diff, 1.3 KB (added by oso96_2000, 12 years ago)
Been a loong week. Ok, after trying for a while I think I've come with a simple test with almost no overhead that just tests this fix. The way the code is now it doesn't even return any post, after the fix, it returns the posts created in the setUp method. Hope it works.

diff --git src/wp-includes/query.php src/wp-includes/query.php
index 417405d..fd1ae2e 100644

  class WP_Query {
                                 foreach ( $statuswheres as $index => $statuswhere )
                                         $statuswheres[$index] = "($statuswhere OR ($wpdb->posts.post_status = 'inherit' AND " . str_replace($wpdb->posts, 'p2', $statuswhere) . "))";
+                        }
+                        foreach ( $statuswheres as $statuswhere )
+                                $where .= " AND $statuswhere";
+                        $where_status = implode(' OR ', $statuswheres);
+                        if(!empty($where_status))
+                                $where .= " AND ($where_status)";
                 } elseif ( !$this->is_singular ) {
                         $where .= " AND ($wpdb->posts.post_status = 'publish'";

diff --git tests/phpunit/tests/query/results.php tests/phpunit/tests/query/results.php
index 404aea6..992d1b9 100644

  class Tests_Query_Results extends WP_UnitTestCase {
                 $this->assertFalse( $this->q->is_month );
                 $this->assertFalse( $this->q->is_year );
+        }
+        /**
+         * @ticket 25523
+         */
+        function test_query_post_readable_perm() {
+                $posts = $this->q->query( array(
+                        'post_status'   =>      array('publish', 'private'),
+                        'perm'                  =>      'readable'
+                ) );
+                $this->assertTrue($this->q->found_posts > 0);
+        }
+}