Skip to:
Content

BuddyPress.org

Ticket #8292: 8292.3.patch

File 8292.3.patch, 8.6 KB (added by imath, 4 years ago)

Improves the multiple select display (use all available width)

  • src/bp-members/admin/css/admin-rtl.css

    diff --git src/bp-members/admin/css/admin-rtl.css src/bp-members/admin/css/admin-rtl.css
    index 3b11ec412..d5f4175de 100644
    div#community-profile-page #profile-nav { 
    2121        margin-bottom: 1em;
    2222}
    2323
     24div#community-profile-page #bp-members-profile-member-type {
     25        width: 100%;
     26}
     27
    2428#bp_members_admin_user_stats ul {
    2529        margin-bottom: 0;
    2630}
  • src/bp-members/admin/css/admin.css

    diff --git src/bp-members/admin/css/admin.css src/bp-members/admin/css/admin.css
    index ff65e5aa3..01f38f31a 100644
    div#community-profile-page #profile-nav { 
    2121        margin-bottom: 1em;
    2222}
    2323
     24div#community-profile-page #bp-members-profile-member-type {
     25        width: 100%;
     26}
     27
    2428#bp_members_admin_user_stats ul {
    2529        margin-bottom: 0;
    2630}
  • src/bp-members/bp-members-functions.php

    diff --git src/bp-members/bp-members-functions.php src/bp-members/bp-members-functions.php
    index c55e942c6..88a81b2da 100644
    add_action( bp_get_member_type_tax_name() . '_add_form', 'bp_insert_member_types 
    29932993 * Set type for a member.
    29942994 *
    29952995 * @since 2.2.0
     2996 * @since 7.0.0 $member_type parameter also accepts an array of member type names.
    29962997 *
    2997  * @param int    $user_id     ID of the user.
    2998  * @param string $member_type Member type.
    2999  * @param bool   $append      Optional. True to append this to existing types for user,
    3000  *                            false to replace. Default: false.
     2998 * @param int          $user_id     ID of the user.
     2999 * @param string|array $member_type The member type name or an array of member type names.
     3000 * @param bool         $append      Optional. True to append this to existing types for user,
     3001 *                                  false to replace. Default: false.
    30013002 * @return false|array $retval See {@see bp_set_object_terms()}.
    30023003 */
    30033004function bp_set_member_type( $user_id, $member_type, $append = false ) {
    30043005        // Pass an empty $member_type to remove a user's type.
    3005         if ( ! empty( $member_type ) && ! bp_get_member_type_object( $member_type ) ) {
    3006                 return false;
     3006        if ( ! empty( $member_type ) ) {
     3007                $member_types = (array) $member_type;
     3008                $valid_types  = array_filter( array_map( 'bp_get_member_type_object', $member_types ) );
     3009
     3010                if ( $valid_types ) {
     3011                        $member_type = wp_list_pluck( $valid_types, 'name' );
     3012                } else {
     3013                        return false;
     3014                }
    30073015        }
    30083016
    30093017        $retval = bp_set_object_terms( $user_id, $member_type, bp_get_member_type_tax_name(), $append );
    function bp_set_member_type( $user_id, $member_type, $append = false ) { 
    30173025                 *
    30183026                 * @since 2.2.0
    30193027                 *
    3020                  * @param int    $user_id     ID of the user whose member type has been updated.
    3021                  * @param string $member_type Member type.
    3022                  * @param bool   $append      Whether the type is being appended to existing types.
     3028                 * @param int          $user_id     ID of the user whose member type has been updated.
     3029                 * @param string|array $member_type The member type name or an array of member type names.
     3030                 * @param bool         $append      Whether the type is being appended to existing types.
    30233031                 */
    30243032                do_action( 'bp_set_member_type', $user_id, $member_type, $append );
    30253033        }
    function bp_get_member_type( $user_id, $single = true, $use_db = true ) { 
    31223130         *
    31233131         * @since 2.2.0
    31243132         *
    3125          * @param string $type    Member type.
    3126          * @param int    $user_id ID of the user.
    3127          * @param bool   $single  Whether to return a single type string, or an array.
     3133         * @param string|array|bool $type    Member type.
     3134         * @param int               $user_id ID of the user.
     3135         * @param bool              $single  Whether to return a single type string, or an array.
    31283136         */
    31293137        return apply_filters( 'bp_get_member_type', $type, $user_id, $single );
    31303138}
  • src/bp-members/classes/class-bp-members-admin.php

    diff --git src/bp-members/classes/class-bp-members-admin.php src/bp-members/classes/class-bp-members-admin.php
    index d344b5d7c..ff0a60d3b 100644
    class BP_Members_Admin { 
    12861286                        return;
    12871287                }
    12881288
    1289                 $types = bp_get_member_types( array(), 'objects' );
    1290                 $current_type = bp_get_member_type( $user->ID );
     1289                $types        = bp_get_member_types( array(), 'objects' );
     1290                $current_type = bp_get_member_type( $user->ID, false );
    12911291                ?>
    12921292
    1293                 <label for="bp-members-profile-member-type" class="screen-reader-text"><?php
     1293                <label for="bp-members-profile-member-type" class="screen-reader-text">
     1294                        <?php
    12941295                        /* translators: accessibility text */
    12951296                        esc_html_e( 'Select member type', 'buddypress' );
    1296                 ?></label>
    1297                 <select name="bp-members-profile-member-type" id="bp-members-profile-member-type">
    1298                         <option value="" <?php selected( '', $current_type ); ?>><?php
    1299                                 /* translators: no option picked in select box */
    1300                                 esc_attr_e( '----', 'buddypress' );
    1301                         ?></option>
     1297                        ?>
     1298                </label>
     1299                <select name="bp-members-profile-member-type[]" id="bp-members-profile-member-type" multiple="multiple">
     1300                        <option value="" <?php selected( ! $current_type ); ?>>
     1301                                <?php
     1302                                        /* translators: no option picked in select box */
     1303                                        esc_attr_e( '----', 'buddypress' );
     1304                                ?>
     1305                        </option>
    13021306                        <?php foreach ( $types as $type ) : ?>
    1303                                 <option value="<?php echo esc_attr( $type->name ) ?>" <?php selected( $type->name, $current_type ) ?>><?php echo esc_html( $type->labels['singular_name'] ) ?></option>
     1307                                <option value="<?php echo esc_attr( $type->name ) ?>" <?php selected( in_array( $type->name, (array) $current_type, true ) ) ?>><?php echo esc_html( $type->labels['singular_name'] ) ?></option>
    13041308                        <?php endforeach; ?>
    13051309                </select>
    13061310
    13071311                <?php
    1308 
    13091312                wp_nonce_field( 'bp-member-type-change-' . $user->ID, 'bp-member-type-nonce' );
    13101313        }
    13111314
    class BP_Members_Admin { 
    13281331                        return;
    13291332                }
    13301333
    1331                 // Member type string must either reference a valid member type, or be empty.
    1332                 $member_type = stripslashes( $_POST['bp-members-profile-member-type'] );
    1333                 if ( ! empty( $member_type ) && ! bp_get_member_type_object( $member_type ) ) {
    1334                         return;
    1335                 }
     1334                // Member type [string] must either reference a valid member type, or be empty.
     1335                $member_type = wp_parse_slug_list( wp_unslash( $_POST['bp-members-profile-member-type'] ) );
     1336                $member_type = array_filter( $member_type );
    13361337
    13371338                /*
    13381339                 * If an invalid member type is passed, someone's doing something
    class BP_Members_Admin { 
    23872388                        foreach ( (array) $_REQUEST['users'] as $user_id ) {
    23882389                                $user_id = (int) $user_id;
    23892390
    2390                                 // Get the old member type to check against.
    2391                                 $member_type = bp_get_member_type( $user_id );
     2391                                // Get the old member types to check against.
     2392                                $current_types = bp_get_member_type( $user_id, false );
    23922393
    2393                                 if ( 'remove_member_type' === $new_type ) {
    2394                                         // Remove the current member type, if there's one to remove.
    2395                                         if ( $member_type ) {
    2396                                                 $removed = bp_remove_member_type( $user_id, $member_type );
    2397                                                 if ( false === $removed || is_wp_error( $removed ) ) {
    2398                                                         $error = true;
    2399                                                 }
    2400                                         }
    2401                                 } else {
     2394                                if ( $current_types && 'remove_member_type' === $new_type ) {
     2395                                        $member_types = array();
     2396                                } elseif ( 1 !== count( $current_types ) || $new_type !== $current_types[0] ) {
    24022397                                        // Set the new member type.
    2403                                         if ( $new_type !== $member_type ) {
    2404                                                 $set = bp_set_member_type( $user_id, $new_type );
    2405                                                 if ( false === $set || is_wp_error( $set ) ) {
    2406                                                         $error = true;
    2407                                                 }
     2398                                        $member_types = array( $new_type );
     2399                                }
     2400
     2401                                if ( isset( $member_types ) ) {
     2402                                        $set = bp_set_member_type( $user_id, $member_types );
     2403                                        if ( false === $set || is_wp_error( $set ) ) {
     2404                                                $error = true;
    24082405                                        }
     2406                                        unset( $member_types );
    24092407                                }
    24102408                        }
    24112409                }
    class BP_Members_Admin { 
    24772475                }
    24782476
    24792477                // Get the member type.
    2480                 $type = bp_get_member_type( $user_id );
     2478                $member_type = bp_get_member_type( $user_id, false );
     2479
     2480                // Build the Output.
     2481                if ( $member_type ) {
     2482                        $member_types = array_filter( array_map( 'bp_get_member_type_object', $member_type ) );
     2483                        if ( ! $member_types ) {
     2484                                return $retval;
     2485                        }
     2486
     2487                        $type_links = array();
     2488                        foreach ( $member_types as $type ) {
     2489                                $url          = add_query_arg( array( 'bp-member-type' => urlencode( $type->name ) ) );
     2490                                $type_links[] = sprintf(
     2491                                        '<a href="%1$s">%2$s</a>',
     2492                                        esc_url( $url ),
     2493                                        esc_html( $type->labels['singular_name'] )
     2494                                );
     2495                        }
    24812496
    2482                 // Output the
    2483                 if ( $type_obj = bp_get_member_type_object( $type ) ) {
    2484                         $url = add_query_arg( array( 'bp-member-type' => urlencode( $type ) ) );
    2485                         $retval = '<a href="' . esc_url( $url ) . '">' . esc_html( $type_obj->labels['singular_name'] ) . '</a>';
     2497                        $retval = implode( ', ', $type_links );
    24862498                }
    24872499
    24882500                return $retval;