Changeset 27647

Timestamp:

03/21/2014 05:20:52 PM (11 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 );
+    }
 }