Make WordPress Core

Opened 3 years ago

Last modified 2 weeks ago

#20899 reviewing defect (bug)

is_home() should be able to be true when is_feed()

Reported by: nacin Owned by: johnbillion
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Query Keywords: reporter-feedback has-patch needs-testing
Focuses: Cc:


is_feed() is a special query flag that can be combined with other query flags — for example, is_author() && is_feed() for /author/admin/feed/.

But it can't be combined with is_home(), because is_home() is the fallback that is only set to true when a lot of other things are true — including is_feed(). This appears to be incorrect — is_home() should still be able to be true despite is_feed().

I tracked this down to [1449]. What kind of breakage could occur with this?

Attachments (1)

20899.diff (826 bytes) - added by stevenkword 7 months ago.

Download all attachments as: .zip

Change History (10)

comment:1 @johnbillion3 years ago

  • Cc johnbillion added

comment:2 @batmoo3 years ago

  • Cc batmoo@… added

comment:3 @ocean903 years ago

#22031 was marked as a duplicate.

comment:5 @wonderboymusic2 years ago

  • Keywords needs-patch reporter-feedback added; has-patch removed

How this make you feel now - still needed?

comment:6 @stevenkword7 months ago

  • Keywords has-patch needs-testing added; needs-patch removed

Patch 20899.diff removes the is_feed check when assigning the value for the is_home query argument.

@stevenkword7 months ago

comment:8 @stevenkword2 months ago

Anyone have an argument against this?

comment:9 @johnbillion2 weeks ago

  • Owner set to johnbillion
  • Status changed from new to reviewing
Note: See TracTickets for help on using tickets.