Skip to:
Content

BuddyPress.org

Ticket #7529: 7529-groups-component.01.diff

File 7529-groups-component.01.diff, 15.1 KB (added by dcavins, 8 years ago)

Use bp_parse_args() instead of wp_parse_args().

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

    diff --git src/bp-groups/bp-groups-activity.php src/bp-groups/bp-groups-activity.php
index cb7c69c..92f8f1e 100644
    function groups_record_activity( $args = '' ) { 
    362362                }
    363363        }
    364364
    365         $r = wp_parse_args( $args, array(
     365        $r = bp_parse_args( $args, array(
    366366                'id'                => false,
    367367                'user_id'           => bp_loggedin_user_id(),
    368368                'action'            => '',
    … … function groups_record_activity( $args = '' ) { 
    375375                'recorded_time'     => bp_core_current_time(),
    376376                'hide_sitewide'     => $hide_sitewide,
    377377                'error_type'        => 'bool'
    378         ) );
     378        ), 'groups_record_activity' );
    379379
    380380        return bp_activity_add( $r );
    381381}

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

    diff --git src/bp-groups/bp-groups-functions.php src/bp-groups/bp-groups-functions.php
index 70dbdf1..0a5f1a8 100644
    function groups_get_group( $group_id ) { 
    4848         * Old-style arguments take the form of an array or a query string.
    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'];
    5858        }
    … … function groups_get_group( $group_id ) { 
    9898 */
    9999function groups_create_group( $args = '' ) {
    100100
    101         $defaults = array(
     101        $args = bp_parse_args( $args, array(
    102102                'group_id'     => 0,
    103103                'creator_id'   => 0,
    104104                'name'         => '',
    … … function groups_create_group( $args = '' ) { 
    108108                'parent_id'    => 0,
    109109                'enable_forum' => 0,
    110110                'date_created' => bp_core_current_time()
    111         );
    112 
    113         $args = wp_parse_args( $args, $defaults );
     111        ), 'groups_create_group' );
    114112        extract( $args, EXTR_SKIP );
    115113
    116114        // Pass an existing group ID.
    … … function groups_edit_base_group_details( $args = array() ) { 
    245243                $args = bp_core_parse_args_array( $old_args_keys, func_get_args() );
    246244        }
    247245
    248         $r = wp_parse_args( $args, array(
     246        $r = bp_parse_args( $args, array(
    249247                'group_id'       => bp_get_current_group_id(),
    250248                'name'           => null,
    251249                'slug'           => null,
    252250                'description'    => null,
    253251                'notify_members' => false,
    254         ) );
     252        ), 'groups_edit_base_group_details' );
    255253
    256254        if ( ! $r['group_id'] ) {
    257255                return false;
    … … function groups_get_group_members( $args = array() ) { 
    679677                $args = bp_core_parse_args_array( $old_args_keys, func_get_args() );
    680678        }
    681679
    682         $r = wp_parse_args( $args, array(
     680        $r = bp_parse_args( $args, array(
    683681                'group_id'            => bp_get_current_group_id(),
    684682                'per_page'            => false,
    685683                'page'                => false,
    … … function groups_get_group_members( $args = array() ) { 
    689687                'group_role'          => array(),
    690688                'search_terms'        => false,
    691689                'type'                => 'last_joined',
    692         ) );
     690        ), 'groups_get_group_members' );
    693691
    694692        // For legacy users. Use of BP_Groups_Member::get_all_for_group() is deprecated.
    695693        if ( apply_filters( 'bp_use_legacy_group_member_query', false, __FUNCTION__, func_get_args() ) ) {
    … … function groups_post_update( $args = '' ) { 
    12611259
    12621260        $bp = buddypress();
    12631261
    1264         $defaults = array(
     1262        $r = bp_parse_args( $args, array(
    12651263                'content'    => false,
    12661264                'user_id'    => bp_loggedin_user_id(),
    12671265                'group_id'   => 0,
    12681266                'error_type' => 'bool'
    1269         );
    1270 
    1271         $r = wp_parse_args( $args, $defaults );
     1267        ), 'groups_post_update' );
    12721268        extract( $r, EXTR_SKIP );
    12731269
    12741270        if ( empty( $group_id ) && !empty( $bp->groups->current_group->id ) )
    … … function groups_get_invite_count_for_user( $user_id = 0 ) { 
    13901386 */
    13911387function groups_invite_user( $args = '' ) {
    13921388
    1393         $defaults = array(
     1389        $args = bp_parse_args( $args, array(
    13941390                'user_id'       => false,
    13951391                'group_id'      => false,
    13961392                'inviter_id'    => bp_loggedin_user_id(),
    13971393                'date_modified' => bp_core_current_time(),
    13981394                'is_confirmed'  => 0
    1399         );
    1400 
    1401         $args = wp_parse_args( $args, $defaults );
     1395        ), 'groups_invite_user' );
    14021396        extract( $args, EXTR_SKIP );
    14031397
    14041398        if ( ! $user_id || ! $group_id || ! $inviter_id ) {

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

    diff --git src/bp-groups/bp-groups-template.php src/bp-groups/bp-groups-template.php
index ac0d315..a80a475 100644
    function bp_group_last_active( $group = false, $args = array() ) { 
    925925                        $group =& $groups_template->group;
    926926                }
    927927
    928                 $r = wp_parse_args( $args, array(
     928                $r = bp_parse_args( $args, array(
    929929                        'relative' => true,
    930                 ) );
     930                ), 'group_last_active' );
    931931
    932932                $last_active = $group->last_activity;
    933933                if ( ! $last_active ) {
    … … function bp_group_date_created( $group = false, $args = array() ) { 
    13531353        function bp_get_group_date_created( $group = false, $args = array() ) {
    13541354                global $groups_template;
    13551355
    1356                 $r = wp_parse_args( $args, array(
     1356                $r = bp_parse_args( $args, array(
    13571357                        'relative' => true,
    1358                 ) );
     1358                ), 'group_date_created' );
    13591359
    13601360                if ( empty( $group ) ) {
    13611361                        $group =& $groups_template->group;
    … … function bp_group_creator_avatar( $group = false, $args = array() ) { 
    15611561                        $group =& $groups_template->group;
    15621562                }
    15631563
    1564                 $defaults = array(
     1564                $r = bp_parse_args( $args, array(
    15651565                        'type'   => 'full',
    15661566                        'width'  => false,
    15671567                        'height' => false,
    15681568                        'class'  => 'avatar',
    15691569                        'id'     => false,
    15701570                        'alt'    => sprintf( __( 'Group creator profile photo of %s', 'buddypress' ),  bp_core_get_user_displayname( $group->creator_id ) )
    1571                 );
    1572 
    1573                 $r = wp_parse_args( $args, $defaults );
     1571                ), 'group_creator_avatar' );
    15741572                extract( $r, EXTR_SKIP );
    15751573
    15761574                $avatar = bp_core_fetch_avatar( array( 'item_id' => $group->creator_id, 'type' => $type, 'css_id' => $id, 'class' => $class, 'width' => $width, 'height' => $height, 'alt' => $alt ) );
    … … function bp_group_forum_topic_count( $args = '' ) { 
    20892087        function bp_get_group_forum_topic_count( $args = '' ) {
    20902088                global $groups_template;
    20912089
    2092                 $defaults = array(
     2090                $r = bp_parse_args( $args, array(
    20932091                        'showtext' => false
    2094                 );
    2095 
    2096                 $r = wp_parse_args( $args, $defaults );
     2092                ), 'group_forum_topic_count' );
    20972093                extract( $r, EXTR_SKIP );
    20982094
    20992095                if ( !$forum_id = groups_get_groupmeta( $groups_template->group->id, 'forum_id' ) ) {
    … … function bp_group_forum_post_count( $args = '' ) { 
    21552151        function bp_get_group_forum_post_count( $args = '' ) {
    21562152                global $groups_template;
    21572153
    2158                 $defaults = array(
     2154                $r = bp_parse_args( $args, array(
    21592155                        'showtext' => false
    2160                 );
    2161 
    2162                 $r = wp_parse_args( $args, $defaults );
     2156                ), 'group_forum_post_count' );
    21632157                extract( $r, EXTR_SKIP );
    21642158
    21652159                if ( !$forum_id = groups_get_groupmeta( $groups_template->group->id, 'forum_id' ) ) {
    … … function bp_group_member_promote_mod_link( $args = '' ) { 
    26052599        function bp_get_group_member_promote_mod_link( $args = '' ) {
    26062600                global $members_template, $groups_template;
    26072601
    2608                 $defaults = array(
     2602                $r = bp_parse_args( $args, array(
    26092603                        'user_id' => $members_template->member->user_id,
    26102604                        'group'   => &$groups_template->group
    2611                 );
    2612 
    2613                 $r = wp_parse_args( $args, $defaults );
     2605                ), 'group_member_promote_mod_link' );
    26142606                extract( $r, EXTR_SKIP );
    26152607
    26162608                /**
    … … function bp_group_member_promote_admin_link( $args = '' ) { 
    26482640        function bp_get_group_member_promote_admin_link( $args = '' ) {
    26492641                global $members_template, $groups_template;
    26502642
    2651                 $defaults = array(
     2643                $r = bp_parse_args( $args, array(
    26522644                        'user_id' => !empty( $members_template->member->user_id ) ? $members_template->member->user_id : false,
    26532645                        'group'   => &$groups_template->group
    2654                 );
    2655 
    2656                 $r = wp_parse_args( $args, $defaults );
     2646                ), 'group_member_promote_admin_link' );
    26572647                extract( $r, EXTR_SKIP );
    26582648
    26592649                /**
    … … function bp_group_has_members( $args = '' ) { 
    38883878                $search_terms_default = stripslashes( $_REQUEST[ $search_query_arg ] );
    38893879        }
    38903880
    3891         $r = wp_parse_args( $args, array(
     3881        $r = bp_parse_args( $args, array(
    38923882                'group_id'            => bp_get_current_group_id(),
    38933883                'page'                => 1,
    38943884                'per_page'            => 20,
    … … function bp_group_has_members( $args = '' ) { 
    38993889                'group_role'          => false,
    39003890                'search_terms'        => $search_terms_default,
    39013891                'type'                => 'last_joined',
    3902         ) );
     3892        ), 'group_has_members' );
    39033893
    39043894        /*
    39053895         * If an empty search_terms string has been passed,
    … … function bp_group_member_joined_since( $args = array() ) { 
    42744264        function bp_get_group_member_joined_since( $args = array() ) {
    42754265                global $members_template;
    42764266
    4277                 $r = wp_parse_args( $args, array(
     4267                $r = bp_parse_args( $args, array(
    42784268                        'relative' => true,
    4279                 ) );
     4269                ), 'group_member_joined_since' );
    42804270
    42814271                // We do not want relative time, so return now.
    42824272                // @todo Should the 'bp_get_group_member_joined_since' filter be applied here?
    … … function bp_new_group_invite_friend_list( $args = array() ) { 
    52135203                }
    52145204
    52155205                // Parse arguments.
    5216                 $r = wp_parse_args( $args, array(
     5206                $r = bp_parse_args( $args, array(
    52175207                        'user_id'   => bp_loggedin_user_id(),
    52185208                        'group_id'  => false,
    52195209                        'separator' => 'li'
    5220                 ) );
     5210                ), 'group_invite_friend_list' );
    52215211
    52225212                // No group passed, so look for new or current group ID's.
    52235213                if ( empty( $r['group_id'] ) ) {
    … … function bp_custom_group_fields() { 
    55655555function bp_group_has_membership_requests( $args = '' ) {
    55665556        global $requests_template;
    55675557
    5568         $defaults = array(
     5558        $r = bp_parse_args( $args, array(
    55695559                'group_id' => bp_get_current_group_id(),
    55705560                'per_page' => 10,
    55715561                'page'     => 1,
    55725562                'max'      => false
    5573         );
    5574 
    5575         $r = wp_parse_args( $args, $defaults );
     5563        ), 'group_has_membership_requests' );
    55765564
    55775565        $requests_template = new BP_Groups_Membership_Requests_Template( $r );
    55785566
    … … function bp_group_requests_pagination_count() { 
    58145802function bp_group_has_invites( $args = '' ) {
    58155803        global $invites_template, $group_id;
    58165804
    5817         $r = wp_parse_args( $args, array(
     5805        $r = bp_parse_args( $args, array(
    58185806                'group_id' => false,
    58195807                'user_id'  => bp_loggedin_user_id(),
    58205808                'per_page' => false,
    58215809                'page'     => 1,
    5822         ) );
     5810        ), 'group_has_invites' );
    58235811
    58245812        if ( empty( $r['group_id'] ) ) {
    58255813                if ( groups_get_current_group() ) {

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

    diff --git src/bp-groups/classes/class-bp-group-member-query.php src/bp-groups/classes/class-bp-group-member-query.php
index cec4a7c..7faa980 100644
    class BP_Group_Member_Query extends BP_User_Query { 
    8888                // We loop through to make sure that defaults are set (though
    8989                // values passed to the constructor will, as usual, override
    9090                // these defaults).
    91                 $this->query_vars = wp_parse_args( $this->query_vars, array(
     91                $this->query_vars = bp_parse_args( $this->query_vars, array(
    9292                        'group_id'     => 0,
    9393                        'group_role'   => array( 'member' ),
    9494                        'is_confirmed' => true,
    9595                        'invite_sent'  => null,
    9696                        'inviter_id'   => null,
    9797                        'type'         => 'last_joined',
    98                 ) );
     98                ), 'group_member_ids' );
    9999
    100100                $group_member_ids = $this->get_group_member_ids();
    101101

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

    diff --git src/bp-groups/classes/class-bp-groups-group-members-template.php src/bp-groups/classes/class-bp-groups-group-members-template.php
index 8b25d75..78df766 100644
    class BP_Groups_Group_Members_Template { 
    111111                        $args = bp_core_parse_args_array( $old_args_keys, func_get_args() );
    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,
    117117                        'per_page'            => 20,
    … … class BP_Groups_Group_Members_Template { 
    123123                        'group_role'          => false,
    124124                        'search_terms'        => false,
    125125                        'type'                => 'last_joined',
    126                 ) );
     126                ), 'group_members_template' );
    127127
    128128                $this->pag_arg  = sanitize_key( $r['page_arg'] );
    129129                $this->pag_page = bp_sanitize_pagination_arg( $this->pag_arg, $r['page']     );

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

    diff --git src/bp-groups/classes/class-bp-groups-group.php src/bp-groups/classes/class-bp-groups-group.php
index 1c243bd..61f94f3 100644
    class BP_Groups_Group { 
    10781078                        'status'             => array()
    10791079                );
    10801080
    1081                 $r = wp_parse_args( $args, $defaults );
     1081                $r = bp_parse_args( $args, $defaults, 'bp_groups_group_get' );
    10821082
    10831083                $bp = buddypress();
    10841084

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

    diff --git src/bp-groups/classes/class-bp-groups-invite-template.php src/bp-groups/classes/class-bp-groups-invite-template.php
index 6498cd4..0d16db8 100644
    class BP_Groups_Invite_Template { 
    9191                        $args = bp_core_parse_args_array( $old_args_keys, func_get_args() );
    9292                }
    9393
    94                 $r = wp_parse_args( $args, array(
     94                $r = bp_parse_args( $args, array(
    9595                        'page'     => 1,
    9696                        'per_page' => 10,
    9797                        'page_arg' => 'invitepage',
    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'] );
    103103                $this->pag_page = bp_sanitize_pagination_arg( $this->pag_arg, $r['page']     );

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

    diff --git src/bp-groups/classes/class-bp-groups-membership-requests-template.php src/bp-groups/classes/class-bp-groups-membership-requests-template.php
index b0a011a..3a9c2f6 100644
    class BP_Groups_Membership_Requests_Template { 
    9999                        $args = bp_core_parse_args_array( $old_args_keys, func_get_args() );
    100100                }
    101101
    102                 $r = wp_parse_args( $args, array(
     102                $r = bp_parse_args( $args, array(
    103103                        'page'     => 1,
    104104                        'per_page' => 10,
    105105                        'page_arg' => 'mrpage',
    106106                        'max'      => false,
    107107                        'type'     => 'first_joined',
    108108                        'group_id' => bp_get_current_group_id(),
    109                 ) );
     109                ), 'group_membership_requests_template' );
    110110
    111111                $this->pag_arg  = sanitize_key( $r['page_arg'] );
    112112                $this->pag_page = bp_sanitize_pagination_arg( $this->pag_arg, $r['page']     );

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

    diff --git src/bp-groups/classes/class-bp-groups-template.php src/bp-groups/classes/class-bp-groups-template.php
index ba5e412..f6483cd 100644
    class BP_Groups_Template { 
    175175                        'update_admin_cache' => false,
    176176                );
    177177
    178                 $r = wp_parse_args( $args, $defaults );
     178                $r = bp_parse_args( $args, $defaults, 'bp_groups_template' );
    179179                extract( $r );
    180180
    181181                $this->pag_arg  = sanitize_key( $r['page_arg'] );

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

    diff --git src/bp-groups/classes/class-bp-groups-widget.php src/bp-groups/classes/class-bp-groups-widget.php
index 049ab31..d18301f 100644
    class BP_Groups_Widget extends WP_Widget { 
    206206                        'group_default' => 'active',
    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'] );
    212212                $max_groups    = strip_tags( $instance['max_groups'] );