Changeset 45286

Timestamp:

05/04/2019 09:06:14 PM (5 years ago)

Author:

azaozz

Message:

Fix setup_postdata() to set the (inside the loop) globals before the_post action is fired. Follow-up from #42814 and [44941].

Props david.binda, spacedmonkey, boonebgorges, birgire, jorbin, azaozz.
Merges [45285] from trunk.
Fixes #47114 for 5.2.

Location:

branches/5.2

Files:

• . (modified) (1 prop)
• src/wp-includes/class-wp-query.php (modified) (2 diffs)
• tests/phpunit/tests/query/setupPostdata.php (modified) (2 diffs)

branches/5.2/src/wp-includes/class-wp-query.php

@@ -4209 +4209 @@
         $numpages     = $elements['numpages'];
 
+
+
+
+
+
+
+
+
+
+
+
         return true;
     }
@@ -4299 +4310 @@
         }
 
-        /**
-         * Fires once the post data has been setup.
-         *
-         * @since 2.8.0
-         * @since 4.1.0 Introduced `$this` parameter.
-         *
-         * @param WP_Post  $post The Post object (passed by reference).
-         * @param WP_Query $this The current Query object (passed by reference).
-         */
-        do_action_ref_array( 'the_post', array( &$post, &$this ) );
-
         $elements = compact( 'id', 'authordata', 'currentday', 'currentmonth', 'page', 'pages', 'multipage', 'more', 'numpages' );
 

branches/5.2/tests/phpunit/tests/query/setupPostdata.php

@@ -9 +9 @@
 
     protected $global_data = array();
+
+
 
     public function setUp() {
@@ -417 +419 @@
     }
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }