Context Navigation

← Previous Change
Next Change →

class-wp-rest-posts-controller.php

Timestamp:

02/27/2017 07:53:43 PM (9 years ago)

Author:

ocean90

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.

Merge of [40037] and [40122] to the 4.7 branch.

Props ryelle, jnylen0.
See #39079.
Fixes #39947.

Location:

branches/4.7

Files:

: 2 edited

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

Legend:

: Unmodified
: Added
: Removed

branches/4.7
- Property svn:mergeinfo changed
  /trunk merged: 40037,40122

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

-              r40115
+              r40136
                 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,
 …
                                  */
                                 if ( ! $args['post__in'] ) {
                                         $args['post__in'] = array( -1 );
+                                        $args['post__in'] = array( 0 );
+                                }
                         } elseif ( $sticky_posts ) {

Note: See TracChangeset for help on using the changeset viewer.

Make WordPress Core

Context Navigation

Changeset 40136 for branches/4.7/src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php

Legend:

branches/4.7

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

Download in other formats: