Changeset 27647

Timestamp:

03/21/2014 05:20:52 PM (13 years ago)

Author:

nacin

Message:

In WP_Query::get_queried_object(), account for pre_get_posts by checking for tag when tag_id isn't present.

Merges [27511] to the 3.8 branch.

props mattonomics, wonderboymusic.
fixes #27362.

Location:

branches/3.8

Files:

• . (modified) (1 prop)
• src/wp-includes/query.php (modified) (1 diff)
• tests/phpunit/tests/query.php (modified) (1 diff)

branches/3.8/src/wp-includes/query.php

@@ -3274 +3274 @@
                                 }
                         } elseif ( $this->is_tag ) {
-                                $term = get_term( $this->get( 'tag_id' ), 'post_tag' );
+                                if ( $this->get( 'tag_id' ) ) {
+                                        $term = get_term( $this->get( 'tag_id' ), 'post_tag' );
+                                } elseif ( $this->get( 'tag' ) ) {
+                                        $term = get_term_by( 'slug', $this->get( 'tag' ), 'post_tag' );
+                                }
                         } else {
                                 $tax_query_in_and = wp_list_filter( $this->tax_query->queries, array( 'operator' => 'NOT IN' ), 'NOT' );

branches/3.8/tests/phpunit/tests/query.php

@@ -79 +79 @@
 
         }
+
+        /**
+         * @ticket 26627
+         */
+        function test_tag_queried_object() {
+                $slug = 'tag-slug-26627';
+                $this->factory->tag->create( array( 'slug' => $slug ) );
+                $tag = get_term_by( 'slug', $slug, 'post_tag' );
+
+                add_action( 'pre_get_posts', array( $this, '_tag_queried_object' ), 11 );
+
+                $this->go_to( get_term_link( $tag ) );
+
+                $this->assertQueryTrue( 'is_tag', 'is_archive' );
+                $this->assertNotEmpty( get_query_var( 'tag_id' ) );
+                $this->assertNotEmpty( get_query_var( 'tag' ) );
+                $this->assertEmpty( get_query_var( 'tax_query' ) );
+                $this->assertCount( 1, get_query_var( 'tag_slug__in' ) );
+                $this->assertEquals( get_queried_object(), $tag );
+
+                remove_action( 'pre_get_posts', array( $this, '_tag_queried_object' ), 11 );
+        }
+
+        function _tag_queried_object( &$query ) {
+                $tag = get_term_by( 'slug', 'tag-slug-26627', 'post_tag' );
+                $this->assertTrue( $query->is_tag() );
+                $this->assertTrue( $query->is_archive() );
+                $this->assertNotEmpty( $query->get( 'tag' ) );
+                $this->assertCount( 1, $query->get( 'tag_slug__in' ) );
+                $this->assertEquals( $query->get_queried_object(), $tag );
+        }
 }