WordPress.org

Make WordPress Core

Opened 4 months ago

#41196 new defect (bug)

add _doing_it_wrong to get_query_var and prevent fatal errors

Reported by: netweblogic Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: trunk
Component: Query Keywords:
Focuses: Cc:

Description

The following scenario will throw a fatal error:

<?php
function parse_query_fail_test(){
        get_query_var('some_var_name', false);
}
add_action('parse_query', 'parse_query_fail_test');

function parse_query_fail_trigger(){
        get_posts();
}
add_action('plugins_loaded', 'parse_query_fail_trigger');

this could easily be avoided with a check and _doing_it_wrong function, similar to functions further down such as is_tax();

I would argue that the same could be done for get_queried_object(), get_queried_object_id() and set_query_var()

Attachments (2)

query.php.patch (503 bytes) - added by netweblogic 4 months ago.
Suggested patch for get_query_var
41196.patch (665 bytes) - added by nileshdudakiya94 4 months ago.
has-patch has-unit-tests added

Download all attachments as: .zip

Change History (2)

@netweblogic
4 months ago

Suggested patch for get_query_var

@nileshdudakiya94
4 months ago

has-patch has-unit-tests added

Note: See TracTickets for help on using tickets.