Context Navigation

Back to Ticket #34060

Ticket #34060: 34060.diff

File 34060.diff, 3.7 KB (added by boonebgorges, 11 years ago)

src/wp-includes/post-functions.php

diff --git a/src/wp-includes/post-functions.php b/src/wp-includes/post-functions.php
index 58961fe..34452c4 100644

                function is_post_type_viewable( $post_type_object ) {
  */
 function get_posts( $args = null ) {
         $defaults = array(
                 'numberposts' => 5, 'offset' => 0,
+                'numberposts' => 5,
                 'category' => 0, 'orderby' => 'date',
                 'order' => 'DESC', 'include' => array(),
                 'exclude' => array(), 'meta_key' => '',

new file tests/phpunit/tests/post/getPosts.php

diff --git a/tests/phpunit/tests/post/getPosts.php b/tests/phpunit/tests/post/getPosts.php
new file mode 100644
index 0000000..496a032

-              -
+<?php
+/**
+ * @group post
+ * @group query
+ */
+class Tests_Post_GetPosts extends WP_UnitTestCase {
+        public function test_offset_should_be_null_by_default() {
+                $p1 = self::factory()->post->create( array(
+                        'post_date' => '2015-04-04 04:04:04',
+                ) );
+                $p2 = self::factory()->post->create( array(
+                        'post_date' => '2014-04-04 04:04:04',
+                ) );
+                $found = get_posts( array(
+                        'numberposts' => 1,
+                        'orderby' => 'date',
+                        'order' => 'DESC',
+                        'fields' => 'ids',
+                ) );
+                $this->assertSame( array( $p1 ), $found );
+        }
+        public function test_offset_0_should_be_respected() {
+                $p1 = self::factory()->post->create( array(
+                        'post_date' => '2015-04-04 04:04:04',
+                ) );
+                $p2 = self::factory()->post->create( array(
+                        'post_date' => '2014-04-04 04:04:04',
+                ) );
+                $found = get_posts( array(
+                        'numberposts' => 1,
+                        'orderby' => 'date',
+                        'order' => 'DESC',
+                        'fields' => 'ids',
+                        'offset' => 0,
+                ) );
+                $this->assertSame( array( $p1 ), $found );
+        }
+        public function test_offset_non_0_should_be_respected() {
+                $p1 = self::factory()->post->create( array(
+                        'post_date' => '2015-04-04 04:04:04',
+                ) );
+                $p2 = self::factory()->post->create( array(
+                        'post_date' => '2014-04-04 04:04:04',
+                ) );
+                $found = get_posts( array(
+                        'numberposts' => 1,
+                        'orderby' => 'date',
+                        'order' => 'DESC',
+                        'fields' => 'ids',
+                        'offset' => 1,
+                ) );
+                $this->assertSame( array( $p2 ), $found );
+        }
+        /**
+         * @ticket 34060
+         */
+        public function test_paged_should_not_be_overridden_by_default_offset() {
+                $p1 = self::factory()->post->create( array(
+                        'post_date' => '2015-04-04 04:04:04',
+                ) );
+                $p2 = self::factory()->post->create( array(
+                        'post_date' => '2014-04-04 04:04:04',
+                ) );
+                $found = get_posts( array(
+                        'orderby' => 'date',
+                        'order' => 'DESC',
+                        'fields' => 'ids',
+                        'paged' => 2,
+                        'posts_per_page' => 1,
+                ) );
+                $this->assertSame( array( $p2 ), $found );
+        }
+        public function test_explicit_offset_0_should_override_paged() {
+                $p1 = self::factory()->post->create( array(
+                        'post_date' => '2015-04-04 04:04:04',
+                ) );
+                $p2 = self::factory()->post->create( array(
+                        'post_date' => '2014-04-04 04:04:04',
+                ) );
+                $found = get_posts( array(
+                        'orderby' => 'date',
+                        'order' => 'DESC',
+                        'fields' => 'ids',
+                        'paged' => 2,
+                        'posts_per_page' => 1,
+                        'offset' => 0,
+                ) );
+                $this->assertSame( array( $p1 ), $found );
+        }
+        public function test_explicit_offset_non_0_should_override_paged() {
+                $p1 = self::factory()->post->create( array(
+                        'post_date' => '2015-04-04 04:04:04',
+                ) );
+                $p2 = self::factory()->post->create( array(
+                        'post_date' => '2014-04-04 04:04:04',
+                ) );
+                $p3 = self::factory()->post->create( array(
+                        'post_date' => '2013-04-04 04:04:04',
+                ) );
+                $found = get_posts( array(
+                        'orderby' => 'date',
+                        'order' => 'DESC',
+                        'fields' => 'ids',
+                        'paged' => 2,
+                        'posts_per_page' => 1,
+                        'offset' => 2,
+                ) );
+                $this->assertSame( array( $p3 ), $found );
+        }
+}

Download in other formats:

Original Format

Make WordPress Core

Context Navigation

Ticket #34060: 34060.diff

src/wp-includes/post-functions.php

new file tests/phpunit/tests/post/getPosts.php

Download in other formats: