Skip to:
Content

bbPress.org

Changeset 7068


Ignore:
Timestamp:
02/07/2020 05:47:55 PM (2 years ago)
Author:
johnjamesjacoby
Message:

Templates: avoid $wp_query global touches.

This commit replaces direct $wp_query touches with calls to bbp_get_wp_query() which is designed for this purpose. Combined with the filter inside of bbp_get_global_object() this enables originally intended flexibility inside of the template stack.

See #3334. For 2.6.15, branches/2.6.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • branches/2.6/src/includes/common/template.php

    r7067 r7068  
    135135 */
    136136function bbp_is_forum_archive() {
    137     global $wp_query;
    138137
    139138    // Default to false
    140139    $retval = false;
     140
     141    // Get the main query global
     142    $wp_query = bbp_get_wp_query();
    141143
    142144    // In forum archive
     
    183185 */
    184186function bbp_is_forum_edit() {
    185     global $wp_query, $pagenow;
    186 
    187     // Assume false
    188     $retval = false;
     187    global $pagenow;
     188
     189    // Assume false
     190    $retval = false;
     191
     192    // Get the main query global
     193    $wp_query = bbp_get_wp_query();
    189194
    190195    // Check query
     
    278283 */
    279284function bbp_is_topic_edit() {
    280     global $wp_query, $pagenow;
    281 
    282     // Assume false
    283     $retval = false;
     285    global $pagenow;
     286
     287    // Assume false
     288    $retval = false;
     289
     290    // Get the main query global
     291    $wp_query = bbp_get_wp_query();
    284292
    285293    // Check query
     
    377385 */
    378386function bbp_is_topic_tag_edit() {
    379     global $wp_query, $pagenow, $taxnow;
     387    global $pagenow, $taxnow;
    380388
    381389    // Bail if topic-tags are off
     
    386394    // Assume false
    387395    $retval = false;
     396
     397    // Get the main query global
     398    $wp_query = bbp_get_wp_query();
    388399
    389400    // Check query
     
    457468 */
    458469function bbp_is_reply_edit() {
    459     global $wp_query, $pagenow;
    460 
    461     // Assume false
    462     $retval = false;
     470    global $pagenow;
     471
     472    // Assume false
     473    $retval = false;
     474
     475    // Get the main query global
     476    $wp_query = bbp_get_wp_query();
    463477
    464478    // Check query
     
    528542 */
    529543function bbp_is_favorites() {
    530     global $wp_query;
    531 
    532     // Assume false
    533     $retval = false;
     544
     545    // Assume false
     546    $retval = false;
     547
     548    // Get the main query global
     549    $wp_query = bbp_get_wp_query();
    534550
    535551    // Check query
     
    550566 */
    551567function bbp_is_subscriptions() {
    552     global $wp_query;
    553 
    554     // Assume false
    555     $retval = false;
     568
     569    // Assume false
     570    $retval = false;
     571
     572    // Get the main query global
     573    $wp_query = bbp_get_wp_query();
    556574
    557575    // Check query
     
    573591 */
    574592function bbp_is_topics_created() {
    575     global $wp_query;
    576 
    577     // Assume false
    578     $retval = false;
     593
     594    // Assume false
     595    $retval = false;
     596
     597    // Get the main query global
     598    $wp_query = bbp_get_wp_query();
    579599
    580600    // Check query
     
    596616 */
    597617function bbp_is_replies_created() {
    598     global $wp_query;
    599 
    600     // Assume false
    601     $retval = false;
     618
     619    // Assume false
     620    $retval = false;
     621
     622    // Get the main query global
     623    $wp_query = bbp_get_wp_query();
    602624
    603625    // Check query
     
    618640 */
    619641function bbp_is_user_home() {
    620     global $wp_query;
    621 
    622     // Assume false
    623     $retval = false;
     642
     643    // Assume false
     644    $retval = false;
     645
     646    // Get the main query global
     647    $wp_query = bbp_get_wp_query();
    624648
    625649    // Check query
     
    660684 */
    661685function bbp_is_single_user() {
    662     global $wp_query;
    663 
    664     // Assume false
    665     $retval = false;
     686
     687    // Assume false
     688    $retval = false;
     689
     690    // Get the main query global
     691    $wp_query = bbp_get_wp_query();
    666692
    667693    // Check query
     
    682708 */
    683709function bbp_is_single_user_edit() {
    684     global $wp_query;
    685 
    686     // Assume false
    687     $retval = false;
     710
     711    // Assume false
     712    $retval = false;
     713
     714    // Get the main query global
     715    $wp_query = bbp_get_wp_query();
    688716
    689717    // Check query
     
    704732 */
    705733function bbp_is_single_user_profile() {
    706     global $wp_query;
    707 
    708     // Assume false
    709     $retval = false;
     734
     735    // Assume false
     736    $retval = false;
     737
     738    // Get the main query global
     739    $wp_query = bbp_get_wp_query();
    710740
    711741    // Check query
     
    726756 */
    727757function bbp_is_single_user_topics() {
    728     global $wp_query;
    729 
    730     // Assume false
    731     $retval = false;
     758
     759    // Assume false
     760    $retval = false;
     761
     762    // Get the main query global
     763    $wp_query = bbp_get_wp_query();
    732764
    733765    // Check query
     
    748780 */
    749781function bbp_is_single_user_replies() {
    750     global $wp_query;
    751 
    752     // Assume false
    753     $retval = false;
     782
     783    // Assume false
     784    $retval = false;
     785
     786    // Get the main query global
     787    $wp_query = bbp_get_wp_query();
    754788
    755789    // Check query
     
    770804 */
    771805function bbp_is_single_user_engagements() {
    772     global $wp_query;
    773 
    774     // Assume false
    775     $retval = false;
     806
     807    // Assume false
     808    $retval = false;
     809
     810    // Get the main query global
     811    $wp_query = bbp_get_wp_query();
    776812
    777813    // Check query
     
    793829 */
    794830function bbp_is_single_view() {
    795     global $wp_query;
    796 
    797     // Assume false
    798     $retval = false;
     831
     832    // Assume false
     833    $retval = false;
     834
     835    // Get the main query global
     836    $wp_query = bbp_get_wp_query();
    799837
    800838    // Check query
     
    821859 */
    822860function bbp_is_search() {
    823     global $wp_query;
    824861
    825862    // Bail if search is disabled
     
    831868    $retval = false;
    832869
     870    // Get the main query global
     871    $wp_query = bbp_get_wp_query();
     872
    833873    // Check query
    834874    if ( ! empty( $wp_query->bbp_is_search ) && ( true === $wp_query->bbp_is_search ) ) {
     
    859899 */
    860900function bbp_is_search_results() {
    861     global $wp_query;
    862901
    863902    // Bail if search is disabled
     
    869908    $retval = false;
    870909
     910    // Get the main query global
     911    $wp_query = bbp_get_wp_query();
     912
    871913    // Check query
    872914    if ( ! empty( $wp_query->bbp_search_terms ) ) {
     
    896938 */
    897939function bbp_is_edit() {
    898     global $wp_query;
    899 
    900     // Assume false
    901     $retval = false;
     940
     941    // Assume false
     942    $retval = false;
     943
     944    // Get the main query global
     945    $wp_query = bbp_get_wp_query();
    902946
    903947    // Check query
Note: See TracChangeset for help on using the changeset viewer.