Make WordPress Core

Opened 12 years ago

Closed 9 years ago

#21821 closed defect (bug) (worksforme)

wp_title spits warnings and returns a blank when a query is both is_post_type_archive and is_category/is_tax/etc

Reported by: devesine's profile devesine Owned by:
Milestone: Priority: normal
Severity: minor Version: 3.4
Component: Themes Keywords: needs-patch
Focuses: template Cc:

Description

When a query is made against both a post type and a category / taxonomy / et cetera, the query is both is_category and is_post_type_archive. The queried_object is set to the first object type it comes to; category / taxonomy come before post_type, so the queried_object is a term here.

wp_title walks through each is_ option in turn and resets the title with every one it finds being true. When there is both a category and a post type archive, single_term_title gets called first, successfully setting the title, but then post_type_archive_title is called, which assumes the queried_object is in fact a post type object; it spits out a couple of warnings and returns a blank title.

Probably, since its subordinate title functions depend on the queried_object being as expected, wp_title should use else if rather than just if (in the same way that get_queried_object does), but I'm not sure if that would have implications elsewhere.

(Tested against trunk r21768 as well as 3.4.1)

Attached is a unit test that demonstrates the problem.

Attachments (1)

isVarious.php (1.5 KB) - added by devesine 12 years ago.

Download all attachments as: .zip

Change History (11)

@devesine
12 years ago

#1 @SergeyBiryukov
12 years ago

Related/duplicate: #18614, #19035, #20994

#2 @greenshady
12 years ago

  • Cc justin@… added

#3 @DrewAPicture
12 years ago

  • Cc xoodrew@… added

#4 @SergeyBiryukov
12 years ago

  • Version changed from trunk to 3.4

#5 @serons
12 years ago

  • Cc serons added

#7 @lkraav
12 years ago

  • Cc leho@… added

Yeap, hitting this in 3.5.2

#8 @nacin
11 years ago

  • Component changed from Template to Themes
  • Focuses template added

#9 @obenland
10 years ago

  • Keywords needs-patch added

#10 @wonderboymusic
9 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to worksforme
  • Status changed from new to closed

This got fixed at some point

Note: See TracTickets for help on using tickets.