Changeset 26012 for trunk/src/wp-includes/query.php

Timestamp:

11/05/2013 01:34:44 AM (13 years ago)

Author:

wonderboymusic

Message:

Respect paged when offset is used in WP_Query. Adds unit tests.

Props coffee2code.
Fixes #18897.

File:

• trunk/src/wp-includes/query.php (modified) (1 diff)

trunk/src/wp-includes/query.php

@@ -2787 +2787 @@
                                 $page = 1;
 
-                        if ( empty($q['offset']) ) {
-                                $pgstrt = ($page - 1) * $q['posts_per_page'] . ', ';
-                        } else { // we're ignoring $page and using 'offset'
+                        $pgstrt = ($page - 1) * $q['posts_per_page'];
+
+                        if ( ! empty( $q['offset'] ) ) {
                                 $q['offset'] = absint($q['offset']);
-                                $pgstrt = $q['offset'] . ', ';
-                        }
-                        $limits = 'LIMIT ' . $pgstrt . $q['posts_per_page'];
+                                $pgstrt += $q['offset'];
+                        }
+                        $limits = 'LIMIT ' . $pgstrt . ', ' . $q['posts_per_page'];
                 }