Skip to:
Content

BuddyPress.org

Changeset 11805


Ignore:
Timestamp:
01/10/2018 12:49:30 PM (7 years ago)
Author:
djpaul
Message:

Groups: use bp_parse_args() in low-level parts of the codebase.

These replace calls to wp_parse_args(). Our version supports a before/after filter, giving flexibility to third-party developers.

Fixes #7529

Props dcavins, espellcaste.

Location:
trunk/src/bp-groups
Files:
10 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-groups/bp-groups-activity.php

    r11763 r11805  
    340340    }
    341341
    342     $r = wp_parse_args( $args, array(
     342    $r = bp_parse_args( $args, array(
    343343        'id'                => false,
    344344        'user_id'           => bp_loggedin_user_id(),
    … …  
    353353        'hide_sitewide'     => $hide_sitewide,
    354354        'error_type'        => 'bool'
    355     ) );
     355    ), 'groups_record_activity' );
    356356
    357357    return bp_activity_add( $r );

  • trunk/src/bp-groups/bp-groups-functions.php

    r11792 r11805  
    4949     */
    5050    if ( ! is_numeric( $group_id ) ) {
    51         $r = wp_parse_args( $group_id, array(
     51        $r = bp_parse_args( $group_id, array(
    5252            'group_id'        => false,
    5353            'load_users'      => false,
    5454            'populate_extras' => false,
    55         ) );
     55        ), 'groups_get_group' );
    5656
    5757        $group_id = $r['group_id'];
    … …  
    9898function groups_create_group( $args = '' ) {
    9999
    100     $defaults = array(
     100    $args = bp_parse_args( $args, array(
    101101        'group_id'     => 0,
    102102        'creator_id'   => 0,
    … …  
    108108        'enable_forum' => null,
    109109        'date_created' => null
    110     );
    111 
    112     $args = wp_parse_args( $args, $defaults );
     110    ), 'groups_create_group' );
     111
    113112    extract( $args, EXTR_SKIP );
    114113
    … …  
    256255    }
    257256
    258     $r = wp_parse_args( $args, array(
     257    $r = bp_parse_args( $args, array(
    259258        'group_id'       => bp_get_current_group_id(),
    260259        'name'           => null,
    … …  
    262261        'description'    => null,
    263262        'notify_members' => false,
    264     ) );
     263    ), 'groups_edit_base_group_details' );
    265264
    266265    if ( ! $r['group_id'] ) {
    … …  
    682681    }
    683682
    684     $r = wp_parse_args( $args, array(
     683    $r = bp_parse_args( $args, array(
    685684        'group_id'            => bp_get_current_group_id(),
    686685        'per_page'            => false,
    … …  
    692691        'search_terms'        => false,
    693692        'type'                => 'last_joined',
    694     ) );
     693    ), 'groups_get_group_members' );
    695694
    696695    // For legacy users. Use of BP_Groups_Member::get_all_for_group() is deprecated.
    … …  
    12661265    $bp = buddypress();
    12671266
    1268     $defaults = array(
     1267    $r = bp_parse_args( $args, array(
    12691268        'content'    => false,
    12701269        'user_id'    => bp_loggedin_user_id(),
    12711270        'group_id'   => 0,
    12721271        'error_type' => 'bool'
    1273     );
    1274 
    1275     $r = wp_parse_args( $args, $defaults );
     1272    ), 'groups_post_update' );
    12761273    extract( $r, EXTR_SKIP );
    12771274
    … …  
    13951392function groups_invite_user( $args = '' ) {
    13961393
    1397     $defaults = array(
     1394    $args = bp_parse_args( $args, array(
    13981395        'user_id'       => false,
    13991396        'group_id'      => false,
    … …  
    14011398        'date_modified' => bp_core_current_time(),
    14021399        'is_confirmed'  => 0
    1403     );
    1404 
    1405     $args = wp_parse_args( $args, $defaults );
     1400    ), 'groups_invite_user' );
    14061401    extract( $args, EXTR_SKIP );
    14071402

  • trunk/src/bp-groups/bp-groups-template.php

    r11782 r11805  
    922922        }
    923923
    924         $r = wp_parse_args( $args, array(
     924        $r = bp_parse_args( $args, array(
    925925            'relative' => true,
    926         ) );
     926        ), 'group_last_active' );
    927927
    928928        $last_active = $group->last_activity;
    … …  
    13501350        global $groups_template;
    13511351
    1352         $r = wp_parse_args( $args, array(
     1352        $r = bp_parse_args( $args, array(
    13531353            'relative' => true,
    1354         ) );
     1354        ), 'group_date_created' );
    13551355
    13561356        if ( empty( $group ) ) {
    … …  
    15581558        }
    15591559
    1560         $defaults = array(
     1560        $r = bp_parse_args( $args, array(
    15611561            'type'   => 'full',
    15621562            'width'  => false,
    … …  
    15651565            'id'     => false,
    15661566            'alt'    => sprintf( __( 'Group creator profile photo of %s', 'buddypress' ),  bp_core_get_user_displayname( $group->creator_id ) )
    1567         );
    1568 
    1569         $r = wp_parse_args( $args, $defaults );
     1567        ), 'group_creator_avatar' );
    15701568        extract( $r, EXTR_SKIP );
    15711569
    … …  
    24452443        global $members_template, $groups_template;
    24462444
    2447         $defaults = array(
     2445        $r = bp_parse_args( $args, array(
    24482446            'user_id' => $members_template->member->user_id,
    24492447            'group'   => &$groups_template->group
    2450         );
    2451 
    2452         $r = wp_parse_args( $args, $defaults );
     2448        ), 'group_member_promote_mod_link' );
    24532449        extract( $r, EXTR_SKIP );
    24542450
    … …  
    24882484        global $members_template, $groups_template;
    24892485
    2490         $defaults = array(
     2486        $r = bp_parse_args( $args, array(
    24912487            'user_id' => !empty( $members_template->member->user_id ) ? $members_template->member->user_id : false,
    24922488            'group'   => &$groups_template->group
    2493         );
    2494 
    2495         $r = wp_parse_args( $args, $defaults );
     2489        ), 'group_member_promote_admin_link' );
    24962490        extract( $r, EXTR_SKIP );
    24972491
    … …  
    36973691    }
    36983692
    3699     $r = wp_parse_args( $args, array(
     3693    $r = bp_parse_args( $args, array(
    37003694        'group_id'            => bp_get_current_group_id(),
    37013695        'page'                => 1,
    … …  
    37083702        'search_terms'        => $search_terms_default,
    37093703        'type'                => 'last_joined',
    3710     ) );
     3704    ), 'group_has_members' );
    37113705
    37123706    /*
    … …  
    40834077        global $members_template;
    40844078
    4085         $r = wp_parse_args( $args, array(
     4079        $r = bp_parse_args( $args, array(
    40864080            'relative' => true,
    4087         ) );
     4081        ), 'group_member_joined_since' );
    40884082
    40894083        // We do not want relative time, so return now.
    … …  
    50225016
    50235017        // Parse arguments.
    5024         $r = wp_parse_args( $args, array(
     5018        $r = bp_parse_args( $args, array(
    50255019            'user_id'   => bp_loggedin_user_id(),
    50265020            'group_id'  => false,
    50275021            'separator' => 'li'
    5028         ) );
     5022        ), 'group_invite_friend_list' );
    50295023
    50305024        // No group passed, so look for new or current group ID's.
    … …  
    53745368    global $requests_template;
    53755369
    5376     $defaults = array(
     5370    $r = bp_parse_args( $args, array(
    53775371        'group_id' => bp_get_current_group_id(),
    53785372        'per_page' => 10,
    53795373        'page'     => 1,
    53805374        'max'      => false
    5381     );
    5382 
    5383     $r = wp_parse_args( $args, $defaults );
     5375    ), 'group_has_membership_requests' );
    53845376
    53855377    $requests_template = new BP_Groups_Membership_Requests_Template( $r );
    … …  
    56235615    global $invites_template, $group_id;
    56245616
    5625     $r = wp_parse_args( $args, array(
     5617    $r = bp_parse_args( $args, array(
    56265618        'group_id' => false,
    56275619        'user_id'  => bp_loggedin_user_id(),
    56285620        'per_page' => false,
    56295621        'page'     => 1,
    5630     ) );
     5622    ), 'group_has_invites' );
    56315623
    56325624    if ( empty( $r['group_id'] ) ) {

  • trunk/src/bp-groups/classes/class-bp-group-member-query.php

    r11792 r11805  
    8888        // values passed to the constructor will, as usual, override
    8989        // these defaults).
    90         $this->query_vars = wp_parse_args( $this->query_vars, array(
     90        $this->query_vars = bp_parse_args( $this->query_vars, array(
    9191            'group_id'     => 0,
    9292            'group_role'   => array( 'member' ),
    … …  
    9595            'inviter_id'   => null,
    9696            'type'         => 'last_joined',
    97         ) );
     97        ), 'bp_group_member_query_get_include_ids' );
    9898
    9999        $group_member_ids = $this->get_group_member_ids();

  • trunk/src/bp-groups/classes/class-bp-groups-group-members-template.php

    r11363 r11805  
    112112        }
    113113
    114         $r = wp_parse_args( $args, array(
     114        $r = bp_parse_args( $args, array(
    115115            'group_id'            => bp_get_current_group_id(),
    116116            'page'                => 1,
    … …  
    124124            'search_terms'        => false,
    125125            'type'                => 'last_joined',
    126         ) );
     126        ), 'group_members_template' );
    127127
    128128        $this->pag_arg  = sanitize_key( $r['page_arg'] );

  • trunk/src/bp-groups/classes/class-bp-groups-group.php

    r11763 r11805  
    10851085        );
    10861086
    1087         $r = wp_parse_args( $args, $defaults );
     1087        $r = bp_parse_args( $args, $defaults, 'bp_groups_group_get' );
    10881088
    10891089        $bp = buddypress();

  • trunk/src/bp-groups/classes/class-bp-groups-invite-template.php

    r11606 r11805  
    9292        }
    9393
    94         $r = wp_parse_args( $args, array(
     94        $r = bp_parse_args( $args, array(
    9595            'page'     => 1,
    9696            'per_page' => 10,
    … …  
    9898            'user_id'  => bp_loggedin_user_id(),
    9999            'group_id' => bp_get_current_group_id(),
    100         ) );
     100        ), 'groups_invite_template' );
    101101
    102102        $this->pag_arg  = sanitize_key( $r['page_arg'] );

  • trunk/src/bp-groups/classes/class-bp-groups-membership-requests-template.php

    r11363 r11805  
    100100        }
    101101
    102         $r = wp_parse_args( $args, array(
     102        $r = bp_parse_args( $args, array(
    103103            'page'     => 1,
    104104            'per_page' => 10,
    … …  
    107107            'type'     => 'first_joined',
    108108            'group_id' => bp_get_current_group_id(),
    109         ) );
     109        ), 'groups_membership_requests_template' );
    110110
    111111        $this->pag_arg  = sanitize_key( $r['page_arg'] );

  • trunk/src/bp-groups/classes/class-bp-groups-template.php

    r11383 r11805  
    176176        );
    177177
    178         $r = wp_parse_args( $args, $defaults );
     178        $r = bp_parse_args( $args, $defaults, 'groups_template' );
    179179        extract( $r );
    180180

  • trunk/src/bp-groups/classes/class-bp-groups-widget.php

    r11564 r11805  
    207207            'link_title'    => false
    208208        );
    209         $instance = wp_parse_args( (array) $instance, $defaults );
     209        $instance = bp_parse_args( (array) $instance, $defaults, 'groups_widget_form' );
    210210
    211211        $title         = strip_tags( $instance['title'] );
Note: See TracChangeset for help on using the changeset viewer.