Skip to:
Content

BuddyPress.org

Changeset 13302


Ignore:
Timestamp:
07/22/2022 11:44:14 AM (2 years ago)
Author:
imath
Message:

Introduce a function to only get the "Join/Leave Group" button args

bp_groups_get_group_join_button_args() eases the process of getting these arguments and removes the need for BP Nouveau's workaround.
Deprecate the bp_nouveau_groups_catch_button_args() as no more needed.

Props hnla

See #7126
See #8722

Location:
trunk/src
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-core/deprecated/11.0.php

    r13301 r13302  
    3434    return array();
    3535}
     36
     37/**
     38 * Catch the arguments for buttons
     39 *
     40 * @since 3.0.0
     41 * @deprecated 11.0.0
     42 *
     43 * @param array $button The arguments of the button that BuddyPress is about to create.
     44 *
     45 * @return array An empty array to stop the button creation process.
     46 */
     47function bp_nouveau_groups_catch_button_args( $button = array() ) {
     48    _deprecated_function( __FUNCTION__, '11.0.0' );
     49
     50    /**
     51     * Globalize the arguments so that we can use it
     52     * in bp_nouveau_get_groups_buttons().
     53     */
     54    bp_nouveau()->groups->button_args = $button;
     55
     56    // return an empty array to stop the button creation process
     57    return array();
     58}

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

    r13296 r13302  
    35643564    echo bp_get_group_join_button( $group );
    35653565}
    3566     /**
    3567      * Return button to join a group.
    3568      *
    3569      * @since 1.0.0
    3570      *
    3571      * @param object|bool $group Single group object.
    3572      * @return false|string
    3573      */
    3574     function bp_get_group_join_button( $group = false ) {
    3575         global $groups_template;
    3576 
    3577         // Set group to current loop group if none passed.
    3578         if ( empty( $group ) ) {
    3579             $group =& $groups_template->group;
     3566
     3567    /**
     3568     * Get the arguments for the Join button group
     3569     *
     3570     * @since 11.0.0
     3571     *
     3572     * @param BP_Groups_Group $group The group object.
     3573     * @return Array The arguments for the Join button group
     3574     */
     3575    function bp_groups_get_group_join_button_args( $group = null ) {
     3576        $button_args = array();
     3577
     3578        if ( empty( $group->id ) ) {
     3579            return $button_args;
    35803580        }
    35813581
    35823582        // Don't show button if not logged in or previously banned.
    35833583        if ( ! is_user_logged_in() || bp_group_is_user_banned( $group ) ) {
    3584             return false;
     3584            return $button_args;
    35853585        }
    35863586
    35873587        // Group creation was not completed or status is unknown.
    35883588        if ( empty( $group->status ) ) {
    3589             return false;
     3589            return $button_args;
    35903590        }
    35913591
    … …  
    35963596            $group_admins = groups_get_group_admins( $group->id );
    35973597            if ( ( 1 == count( $group_admins ) ) && ( bp_loggedin_user_id() === (int) $group_admins[0]->user_id ) ) {
    3598                 return false;
     3598                return $button_args;
    35993599            }
    36003600
    36013601            // Setup button attributes.
    3602             $button = array(
     3602            $button_args = array(
    36033603                'id'                => 'leave_group',
    36043604                'component'         => 'groups',
    … …  
    36093609                'link_href'         => wp_nonce_url( trailingslashit( bp_get_group_permalink( $group ) . 'leave-group' ), 'groups_leave_group' ),
    36103610                'link_text'         => __( 'Leave Group', 'buddypress' ),
     3611                'link_title'        => __( 'Leave Group', 'buddypress' ),
    36113612                'link_class'        => 'group-button leave-group',
    36123613            );
    … …  
    36183619            switch ( $group->status ) {
    36193620                case 'hidden' :
    3620                     return false;
     3621                    return $button_args;
    36213622
    36223623                case 'public':
    3623                     $button = array(
     3624                    $button_args = array(
    36243625                        'id'                => 'join_group',
    36253626                        'component'         => 'groups',
    … …  
    36303631                        'link_href'         => wp_nonce_url( trailingslashit( bp_get_group_permalink( $group ) . 'join' ), 'groups_join_group' ),
    36313632                        'link_text'         => __( 'Join Group', 'buddypress' ),
     3633                        'link_title'        => __( 'Join Group', 'buddypress' ),
    36323634                        'link_class'        => 'group-button join-group',
    36333635                    );
    … …  
    36393641                    // show an "Accept Invitation" button.
    36403642                    if ( $group->is_invited ) {
    3641                         $button = array(
     3643                        $button_args = array(
    36423644                            'id'                => 'accept_invite',
    36433645                            'component'         => 'groups',
    … …  
    36483650                            'link_href'         => add_query_arg( 'redirect_to', bp_get_group_permalink( $group ), bp_get_group_accept_invite_link( $group ) ),
    36493651                            'link_text'         => __( 'Accept Invitation', 'buddypress' ),
     3652                            'link_title'        => __( 'Accept Invitation', 'buddypress' ),
    36503653                            'link_class'        => 'group-button accept-invite',
    36513654                        );
    … …  
    36543657                    // show a "Request Sent" button.
    36553658                    } elseif ( $group->is_pending ) {
    3656                         $button = array(
     3659                        $button_args = array(
    36573660                            'id'                => 'membership_requested',
    36583661                            'component'         => 'groups',
    … …  
    36633666                            'link_href'         => bp_get_group_permalink( $group ),
    36643667                            'link_text'         => __( 'Request Sent', 'buddypress' ),
     3668                            'link_title'        => __( 'Request Sent', 'buddypress' ),
    36653669                            'link_class'        => 'group-button pending membership-requested',
    36663670                        );
    … …  
    36693673                    // show a "Request Membership" button.
    36703674                    } else {
    3671                         $button = array(
     3675                        $button_args = array(
    36723676                            'id'                => 'request_membership',
    36733677                            'component'         => 'groups',
    … …  
    36783682                            'link_href'         => wp_nonce_url( trailingslashit( bp_get_group_permalink( $group ) . 'request-membership' ), 'groups_request_membership' ),
    36793683                            'link_text'         => __( 'Request Membership', 'buddypress' ),
     3684                            'link_title'        => __( 'Request Membership', 'buddypress' ),
    36803685                            'link_class'        => 'group-button request-membership',
    36813686                        );
    … …  
    36873692
    36883693        /**
    3689          * Filters the HTML button for joining a group.
     3694         * Filters the arguments of the button for joining a group.
    36903695         *
    36913696         * @since 1.2.6
    36923697         * @since 2.4.0 Added $group parameter to filter args.
    36933698         *
    3694          * @param string $button HTML button for joining a group.
    3695          * @param object $group BuddyPress group object
    3696          */
    3697         return bp_get_button( apply_filters( 'bp_get_group_join_button', $button, $group ) );
     3699         * @param array  $button_args The arguments for the button.
     3700         * @param object $group       BuddyPress group object
     3701         */
     3702        return apply_filters( 'bp_get_group_join_button', $button_args, $group );
     3703    }
     3704    /**
     3705     * Return button to join a group.
     3706     *
     3707     * @since 1.0.0
     3708     * @since 11.0.0 uses `bp_groups_get_group_join_button_args()`.
     3709     *
     3710     * @param object|bool $group Single group object.
     3711     * @return false|string
     3712     */
     3713    function bp_get_group_join_button( $group = false ) {
     3714        global $groups_template;
     3715
     3716        // Set group to current loop group if none passed.
     3717        if ( empty( $group ) ) {
     3718            $group =& $groups_template->group;
     3719        }
     3720
     3721        $button_args = bp_groups_get_group_join_button_args( $group );
     3722
     3723        if ( ! array_filter( $button_args ) ) {
     3724            return false;
     3725        }
     3726
     3727        return bp_get_button( $button_args );
    36983728    }
    36993729

  • trunk/src/bp-templates/bp-nouveau/includes/groups/functions.php

    r13192 r13302  
    661661
    662662/**
    663  * Catch the arguments for buttons
    664  *
    665  * @since 3.0.0
    666  *
    667  * @param array $button The arguments of the button that BuddyPress is about to create.
    668  *
    669  * @return array An empty array to stop the button creation process.
    670  */
    671 function bp_nouveau_groups_catch_button_args( $button = array() ) {
    672     /**
    673      * Globalize the arguments so that we can use it
    674      * in bp_nouveau_get_groups_buttons().
    675      */
    676     bp_nouveau()->groups->button_args = $button;
    677 
    678     // return an empty array to stop the button creation process
    679     return array();
    680 }
    681 
    682 /**
    683663 * Catch the content hooked to the 'bp_group_header_meta' action
    684664 *

  • trunk/src/bp-templates/bp-nouveau/includes/groups/template-tags.php

    r13193 r13302  
    997997        // Membership button on groups loop or single group's header
    998998        } else {
    999             /*
    1000              * This filter workaround is waiting for a core adaptation
    1001              * so that we can directly get the groups button arguments
    1002              * instead of the button.
    1003              *
    1004              * See https://buddypress.trac.wordpress.org/ticket/7126
    1005              */
    1006             add_filter( 'bp_get_group_join_button', 'bp_nouveau_groups_catch_button_args', 100, 1 );
    1007 
    1008             bp_get_group_join_button( $group );
    1009 
    1010             remove_filter( 'bp_get_group_join_button', 'bp_nouveau_groups_catch_button_args', 100, 1 );
    1011 
    1012             if ( isset( bp_nouveau()->groups->button_args ) && bp_nouveau()->groups->button_args ) {
    1013                 $button_args = bp_nouveau()->groups->button_args;
    1014 
    1015                 // If we pass through parent classes merge those into the existing ones
    1016                 if ( $parent_class ) {
    1017                     $parent_class .= ' ' . $button_args['wrapper_class'];
    1018                 }
    1019 
    1020                 // The join or leave group header button should default to 'button'
    1021                 // Reverse the earler button var to set default as 'button' not 'a'
    1022                 if ( empty( $args['button_element'] ) ) {
    1023                     $button_element = 'button';
    1024                 }
    1025 
    1026                 $buttons['group_membership'] = array(
    1027                     'id'                => 'group_membership',
    1028                     'position'          => 5,
    1029                     'component'         => $button_args['component'],
    1030                     'must_be_logged_in' => $button_args['must_be_logged_in'],
    1031                     'block_self'        => $button_args['block_self'],
    1032                     'parent_element'    => $parent_element,
    1033                     'button_element'    => $button_element,
    1034                     'link_text'         => $button_args['link_text'],
    1035                     'parent_attr'       => array(
    1036                             'id'    => $button_args['wrapper_id'],
    1037                             'class' => $parent_class,
    1038                     ),
    1039                     'button_attr'       => array(
    1040                         'id'    => ! empty( $button_args['link_id'] ) ? $button_args['link_id'] : '',
    1041                         'class' => $button_args['link_class'] . ' button',
    1042                         'rel'   => ! empty( $button_args['link_rel'] ) ? $button_args['link_rel'] : '',
    1043                         'title' => '',
    1044                     ),
    1045                 );
     999            $button_args = bp_groups_get_group_join_button_args( $group );
     1000
     1001            // If we pass through parent classes merge those into the existing ones
     1002            if ( $parent_class ) {
     1003                $parent_class .= ' ' . $button_args['wrapper_class'];
     1004            }
     1005
     1006            // The join or leave group header button should default to 'button'
     1007            // Reverse the earler button var to set default as 'button' not 'a'
     1008            if ( empty( $args['button_element'] ) ) {
     1009                $button_element = 'button';
     1010            }
     1011
     1012            $buttons['group_membership'] = array(
     1013                'id'                => 'group_membership',
     1014                'position'          => 5,
     1015                'component'         => $button_args['component'],
     1016                'must_be_logged_in' => $button_args['must_be_logged_in'],
     1017                'block_self'        => $button_args['block_self'],
     1018                'parent_element'    => $parent_element,
     1019                'button_element'    => $button_element,
     1020                'link_text'         => $button_args['link_text'],
     1021                'link_title'        => $button_args['link_title'],
     1022                'parent_attr'       => array(
     1023                        'id'    => $button_args['wrapper_id'],
     1024                        'class' => $parent_class,
     1025                ),
     1026                'button_attr'       => array(
     1027                    'id'    => ! empty( $button_args['link_id'] ) ? $button_args['link_id'] : '',
     1028                    'class' => $button_args['link_class'] . ' button',
     1029                    'rel'   => ! empty( $button_args['link_rel'] ) ? $button_args['link_rel'] : '',
     1030                    'title' => '',
     1031                ),
     1032            );
    10461033
    10471034            // If button element set add nonce 'href' link to data-attr attr.
    … …  
    10511038            // Else this is an anchor so use an 'href' attr.
    10521039                $buttons['group_membership']['button_attr']['href'] = $button_args['link_href'];
    1053             }
    1054 
    1055                 unset( bp_nouveau()->groups->button_args );
    10561040            }
    10571041        }
Note: See TracChangeset for help on using the changeset viewer.