Changeset 40122 for trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php

Timestamp:

02/25/2017 05:02:17 AM (8 years ago)

Author:

jnylen0

Message:

REST API: Fix behavior of sticky posts filter when no posts are sticky.

Previously, when getting posts from the API with sticky=true, if there were no sticky posts set, the query would return all posts as if the sticky argument was not set. In this situation, the query should return an empty array instead.

A sticky=true query that should return an empty array (in the previous situation, or with include and no intersecting post IDs) was also broken in that it would query the post with ID 1.

Finally, this commit significantly improves test coverage for the sticky filter argument, including direct testing of the WHERE clauses generated by WP_Query.

Props ryelle.
Fixes #39947.

File:

• trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php (modified) (2 diffs)

trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php

@@ -221 +221 @@
         if ( isset( $registered['sticky'], $request['sticky'] ) ) {
             $sticky_posts = get_option( 'sticky_posts', array() );
-            if ( $sticky_posts && $request['sticky'] ) {
+            if ( ! is_array( $sticky_posts ) ) {
+                $sticky_posts = array();
+            }
+            if ( $request['sticky'] ) {
                 /*
                  * As post__in will be used to only get sticky posts,
@@ -235 +238 @@
                  */
                 if ( ! $args['post__in'] ) {
-                    $args['post__in'] = array( -1 );
+                    $args['post__in'] = array( 0 );
                 }
             } elseif ( $sticky_posts ) {