[wp-trac] [WordPress Trac] #31246: Notice: Undefined offset: 0 in wp-includes/query.php on line 3877

[WordPress Trac]

#31246: Notice: Undefined offset: 0 in wp-includes/query.php on line 3877
------------------------------------------+---------------------------
 Reporter:  Mediastuttgart                |       Owner:  boonebgorges
     Type:  defect (bug)                  |      Status:  reopened
 Priority:  normal                        |   Milestone:  4.1.1
Component:  Query                         |     Version:  4.1
 Severity:  trivial                       |  Resolution:
 Keywords:  has-patch fixed-major commit  |     Focuses:
------------------------------------------+---------------------------

Comment (by Chouby):

 Replying to [comment:5 boonebgorges]:

 > The trick here (and the reason you're unsetting the query var) is that
 'all' is, of course, not actually a term in the 'language' taxonomy. I
 think your trick is OK, but you might consider running it a little sooner,
 maybe at 'parse_request', so that it would be sorted out before `WP_Query`
 has initialized.
 Yes. `parse_query` is a very convenient action to modify the query when I
 need to. Here however, since the 'all' value can come only from the main
 request, you are right, it's better to use the `request` filter. Thanks
 for the suggestion.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/31246#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform