Skip to:
Content

BuddyPress.org

Changeset 11808


Ignore:
Timestamp:
01/10/2018 09:34:15 PM (7 years ago)
Author:
johnjamesjacoby
Message:

XProfile: use bp_get_admin_url() where appropriate.

This change updates all inline and querystring-based URL references to users.php to instead use the built-in supplied wrapper function. This comes with a few benefits:

  • URLs will be consistent across BuddyPress installation configurations (rather than relative to the current admin dashboard area)
  • Code is cleaner and easier to understand
  • Variable IDs are now cast to integers as needed (where they were not all previously)

Bonus: also updates a few bp_get_admin_url() calls in the about page to use the second parameter of add_query_arg() rather than wrapping it entirely, to match the expected usage format of all add_query_arg() calls.

Fixes #7536. Props JohnPBloch.

Location:
trunk/src
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-core/classes/class-bp-admin.php

    r11764 r11808  
    577577                                <ul>
    578578                                    <?php if ( bp_is_active( 'members' ) ) : ?>
    579                                         <li><?php printf( '<a href="%s" class="welcome-icon welcome-add-page">' . __( 'Add User Profile Fields', 'buddypress' ) . '</a>', esc_url( bp_get_admin_url( add_query_arg( array( 'page' => 'bp-profile-setup' ), 'users.php' ) ) ) ); ?></li>
     579                                        <li><?php printf( '<a href="%s" class="welcome-icon welcome-add-page">' . __( 'Add User Profile Fields', 'buddypress' ) . '</a>', esc_url( add_query_arg( array( 'page' => 'bp-profile-setup' ), bp_get_admin_url( 'users.php' ) ) ) ); ?></li>
    580580                                    <?php endif; ?>
    581                                     <li><?php printf( '<a href="%s" class="welcome-icon welcome-add-page">' . __( 'Manage User Signups', 'buddypress' ) . '</a>', esc_url( bp_get_admin_url( add_query_arg( array( 'page' => 'bp-signups' ), 'users.php' ) ) ) ); ?></li>
     581                                    <li><?php printf( '<a href="%s" class="welcome-icon welcome-add-page">' . __( 'Manage User Signups', 'buddypress' ) . '</a>', esc_url( add_query_arg( array( 'page' => 'bp-signups' ), bp_get_admin_url( 'users.php' ) ) ) ); ?></li>
    582582                                    <?php if ( bp_is_active( 'activity' ) ) : ?>
    583                                         <li><?php printf( '<a href="%s" class="welcome-icon welcome-add-page">' . __( 'Moderate Activity Streams', 'buddypress' ) . '</a>', esc_url( bp_get_admin_url( add_query_arg( array( 'page' => 'bp-activity' ), 'admin.php' ) ) ) ); ?></li>
     583                                        <li><?php printf( '<a href="%s" class="welcome-icon welcome-add-page">' . __( 'Moderate Activity Streams', 'buddypress' ) . '</a>', esc_url( add_query_arg( array( 'page' => 'bp-activity' ), bp_get_admin_url( 'admin.php' ) ) ) ); ?></li>
    584584                                    <?php endif; ?>
    585585                                    <?php if ( bp_is_active( 'groups' ) ) : ?>
    586                                         <li><?php printf( '<a href="%s" class="welcome-icon welcome-add-page">' . __( 'Manage Groups', 'buddypress' ) . '</a>', esc_url( bp_get_admin_url( add_query_arg( array( 'page' => 'bp-groups' ), 'admin.php' ) ) ) ); ?></li>
     586                                        <li><?php printf( '<a href="%s" class="welcome-icon welcome-add-page">' . __( 'Manage Groups', 'buddypress' ) . '</a>', esc_url( add_query_arg( array( 'page' => 'bp-groups' ), bp_get_admin_url( 'admin.php' ) ) ) ); ?></li>
    587587                                    <?php endif; ?>
    588                                     <li><?php printf( '<a href="%s" class="welcome-icon welcome-add-page">' . __( 'Repair Data', 'buddypress' ) . '</a>', esc_url( bp_get_admin_url( add_query_arg( array( 'page' => 'bp-tools' ), 'tools.php' ) ) ) ); ?>
     588                                    <li><?php printf( '<a href="%s" class="welcome-icon welcome-add-page">' . __( 'Repair Data', 'buddypress' ) . '</a>', esc_url( add_query_arg( array( 'page' => 'bp-tools' ), bp_get_admin_url( 'tools.php' ) ) ) ); ?>
    589589                                    </li>
    590590                                </ul>

  • trunk/src/bp-xprofile/bp-xprofile-admin.php

    r11741 r11808  
    125125function xprofile_admin_screen( $message = '', $type = 'error' ) {
    126126
     127    // Users admin URL
     128    $url = bp_get_admin_url( 'users.php' );
     129
     130    // Add Group
     131    $add_group_url = add_query_arg( array(
     132        'page' => 'bp-profile-setup',
     133        'mode' => 'add_group'
     134    ), $url );
     135
    127136    // Validate type.
    128137    $type = preg_replace( '|[^a-z]|i', '', $type );
    … …  
    138147            <h1 class="wp-heading-inline"><?php _ex( 'Profile Fields', 'Settings page header', 'buddypress'); ?></h1>
    139148
    140                 <a id="add_group" class="page-title-action" href="users.php?page=bp-profile-setup&amp;mode=add_group"><?php _e( 'Add New Field Group', 'buddypress' ); ?></a>
     149                <a id="add_group" class="page-title-action" href="<?php echo esc_url( $add_group_url ); ?>"><?php _e( 'Add New Field Group', 'buddypress' ); ?></a>
    141150
    142151            <hr class="wp-header-end">
    … …  
    146155            <h1>
    147156                <?php _ex( 'Profile Fields', 'Settings page header', 'buddypress'); ?>
    148                 <a id="add_group" class="add-new-h2" href="users.php?page=bp-profile-setup&amp;mode=add_group"><?php _e( 'Add New Field Group', 'buddypress' ); ?></a>
     157                <a id="add_group" class="add-new-h2" href="<?php echo esc_url( $add_group_url ); ?>"><?php _e( 'Add New Field Group', 'buddypress' ); ?></a>
    149158            </h1>
    150159
    … …  
    190199                </ul>
    191200
    192                 <?php if ( !empty( $groups ) ) : foreach ( $groups as $group ) : ?>
     201                <?php if ( !empty( $groups ) ) : foreach ( $groups as $group ) :
     202
     203                    // Add Field to Group URL
     204                    $add_field_url = add_query_arg( array(
     205                        'page'     => 'bp-profile-setup',
     206                        'mode'     => 'add_field',
     207                        'group_id' => (int) $group->id
     208                    ), $url );
     209
     210                    // Edit Group URL
     211                    $edit_group_url = add_query_arg( array(
     212                        'page'     => 'bp-profile-setup',
     213                        'mode'     => 'edit_group',
     214                        'group_id' => (int) $group->id
     215                    ), $url );
     216
     217                    // Delete Group URL
     218                    $delete_group_url = wp_nonce_url( add_query_arg( array(
     219                        'page'     => 'bp-profile-setup',
     220                        'mode'     => 'delete_group',
     221                        'group_id' => (int) $group->id
     222                    ), $url ), 'bp_xprofile_delete_group' ); ?>
    193223
    194224                    <noscript>
    … …  
    202232                        <div class="tab-toolbar">
    203233                            <div class="tab-toolbar-left">
    204                                 <a class="button-primary" href="users.php?page=bp-profile-setup&amp;group_id=<?php echo esc_attr( $group->id ); ?>&amp;mode=add_field"><?php _e( 'Add New Field', 'buddypress' ); ?></a>
    205                                 <a class="button edit" href="users.php?page=bp-profile-setup&amp;mode=edit_group&amp;group_id=<?php echo esc_attr( $group->id ); ?>"><?php _ex( 'Edit Group', 'Edit Profile Fields Group', 'buddypress' ); ?></a>
     234                                <a class="button-primary" href="<?php echo esc_url( $add_field_url ); ?>"><?php _e( 'Add New Field', 'buddypress' ); ?></a>
     235                                <a class="button edit" href="<?php echo esc_url( $edit_group_url ); ?>"><?php _ex( 'Edit Group', 'Edit Profile Fields Group', 'buddypress' ); ?></a>
    206236
    207237                                <?php if ( $group->can_delete ) : ?>
    208238
    209239                                    <div class="delete-button">
    210                                         <a class="confirm submitdelete deletion ajax-option-delete" href="<?php echo esc_url( wp_nonce_url( 'users.php?page=bp-profile-setup&amp;mode=delete_group&amp;group_id=' . intval( $group->id ), 'bp_xprofile_delete_group' ) ); ?>"><?php _ex( 'Delete Group', 'Delete Profile Fields Group', 'buddypress' ); ?></a>
     240                                        <a class="confirm submitdelete deletion ajax-option-delete" href="<?php echo esc_url( $delete_group_url ); ?>"><?php _ex( 'Delete Group', 'Delete Profile Fields Group', 'buddypress' ); ?></a>
    211241                                    </div>
    212242
    … …  
    284314
    285315                    <div id="message" class="error"><p><?php _ex( 'You have no groups.', 'You have no profile fields groups.', 'buddypress' ); ?></p></div>
    286                     <p><a href="users.php?page=bp-profile-setup&amp;mode=add_group"><?php _ex( 'Add New Group', 'Add New Profile Fields Group', 'buddypress' ); ?></a></p>
     316                    <p><a href="<?php echo esc_url( $add_group_url ); ?>"><?php _ex( 'Add New Group', 'Add New Profile Fields Group', 'buddypress' ); ?></a></p>
    287317
    288318                <?php endif; ?>
    … …  
    620650    $field = $admin_field;
    621651
    622     $field_edit_url = add_query_arg(
    623         array(
     652    // Users admin URL
     653    $url = bp_get_admin_url( 'users.php' );
     654
     655    // Edit
     656    $field_edit_url = add_query_arg( array(
     657        'page'     => 'bp-profile-setup',
     658        'mode'     => 'edit_field',
     659        'group_id' => (int) $field->group_id,
     660        'field_id' => (int) $field->id
     661    ), $url );
     662
     663    // Delete
     664    if ( $field->can_delete ) {
     665        $field_delete_url = add_query_arg( array(
    624666            'page'     => 'bp-profile-setup',
    625             'group_id' => (int) $field->group_id,
    626             'field_id' => (int) $field->id,
    627             'mode'     => 'edit_field'
    628         ),
    629         bp_get_admin_url( 'users.php' )
    630     );
    631 
    632     if ( $field->can_delete ) {
    633         $field_delete_url = add_query_arg(
    634             array(
    635                 'page'     => 'bp-profile-setup',
    636                 'field_id' => (int) $field->id,
    637                 'mode'     => 'delete_field'
    638             ),
    639             bp_get_admin_url( 'users.php' ) . '#tabs-' . (int) $field->group_id
    640         );
    641     }
    642     ?>
     667            'mode'     => 'delete_field',
     668            'field_id' => (int) $field->id
     669        ), $url . '#tabs-' . (int) $field->group_id );
     670    } ?>
    643671
    644672    <fieldset id="draggable_field_<?php echo esc_attr( $field->id ); ?>" class="sortable<?php echo ' ' . $field->type; if ( !empty( $class ) ) echo ' ' . $class; ?>">
    … …  
    647675                <?php bp_the_profile_field_name(); ?>
    648676
    649                 <?php if ( empty( $field->can_delete )                                    ) : ?><?php esc_html_e( '(Primary)',  'buddypress' ); endif; ?>
     677                <?php if ( empty( $field->can_delete )                                    ) : ?><?php esc_html_e( '(Primary)', 'buddypress' ); endif; ?>
    650678                <?php bp_the_profile_field_required_label(); ?>
    651                 <?php if ( bp_xprofile_get_meta( $field->id, 'field', 'signup_position' ) ) : ?><?php esc_html_e( '(Sign-up)',  'buddypress' ); endif; ?>
     679                <?php if ( bp_xprofile_get_meta( $field->id, 'field', 'signup_position' ) ) : ?><?php esc_html_e( '(Sign-up)', 'buddypress' ); endif; ?>
    652680                <?php if ( bp_get_member_types() ) : echo $field->get_member_type_label(); endif; ?>
    653681

  • trunk/src/bp-xprofile/classes/class-bp-xprofile-field.php

    r11700 r11808  
    11641164     */
    11651165    public function render_admin_form( $message = '' ) {
     1166
     1167        // Users Admin URL
     1168        $users_url = bp_get_admin_url( 'users.php' );
     1169
     1170        // Add New
    11661171        if ( empty( $this->id ) ) {
    11671172            $title  = __( 'Add New Field', 'buddypress' );
    1168             $action = "users.php?page=bp-profile-setup&amp;group_id=" . $this->group_id . "&amp;mode=add_field#tabs-" . $this->group_id;
    1169             $button = __( 'Save', 'buddypress' );
     1173            $button = __( 'Save',          'buddypress' );
     1174            $action = add_query_arg( array(
     1175                'page'     => 'bp-profile-setup',
     1176                'mode'     => 'add_field',
     1177                'group_id' => (int) $this->group_id
     1178            ), $users_url . '#tabs-' . (int) $this->group_id );
    11701179
    11711180            if ( !empty( $_POST['saveField'] ) ) {
    … …  
    11801189                }
    11811190            }
     1191
     1192        // Edit
    11821193        } else {
    11831194            $title  = __( 'Edit Field', 'buddypress' );
    1184             $action = "users.php?page=bp-profile-setup&amp;mode=edit_field&amp;group_id=" . $this->group_id . "&amp;field_id=" . $this->id . "#tabs-" . $this->group_id;
    1185             $button = __( 'Update', 'buddypress' );
     1195            $button = __( 'Update',     'buddypress' );
     1196            $action = add_query_arg( array(
     1197                'page'     => 'bp-profile-setup',
     1198                'mode'     => 'edit_field',
     1199                'group_id' => (int) $this->group_id,
     1200                'field_id' => (int) $this->id
     1201            ), $users_url . '#tabs-' . (int) $this->group_id );
    11861202        } ?>
    11871203
    … …  
    12871303    private function submit_metabox( $button_text = '' ) {
    12881304
     1305        // Setup the URL for deleting
     1306        $users_url  = bp_get_admin_url( 'users.php' );
     1307        $cancel_url = add_query_arg( array(
     1308            'page' => 'bp-profile-setup'
     1309        ), $users_url );
     1310
    12891311        /**
    12901312         * Fires before XProfile Field submit metabox.
    … …  
    13241346
    13251347                        <div id="delete-action">
    1326                             <a href="users.php?page=bp-profile-setup" class="deletion"><?php esc_html_e( 'Cancel', 'buddypress' ); ?></a>
     1348                            <a href="<?php echo esc_url( $cancel_url ); ?>" class="deletion"><?php esc_html_e( 'Cancel', 'buddypress' ); ?></a>
    13271349                        </div>
    13281350

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

    r11468 r11808  
    717717        global $message;
    718718
     719        // Users Admin URL
     720        $users_url = bp_get_admin_url( 'users.php' );
     721
     722        // URL to cancel to
     723        $cancel_url = add_query_arg( array(
     724            'page' => 'bp-profile-setup'
     725        ), $users_url );
     726
    719727        // New field group.
    720728        if ( empty( $this->id ) ) {
    721729            $title  = __( 'Add New Field Group', 'buddypress' );
    722             $action = add_query_arg( array( 'page' => 'bp-profile-setup', 'mode' => 'add_group' ), 'users.php' );
    723             $button = __( 'Save', 'buddypress' );
     730            $button = __( 'Save',                'buddypress' );
     731            $action = add_query_arg( array(
     732                'page' => 'bp-profile-setup',
     733                'mode' => 'add_group'
     734            ), $users_url );
    724735
    725736        // Existing field group.
    726737        } else {
    727738            $title  = __( 'Edit Field Group', 'buddypress' );
    728             $action = add_query_arg( array( 'page' => 'bp-profile-setup', 'mode' => 'edit_group', 'group_id' => $this->id ), 'users.php' );
    729             $button = __( 'Update', 'buddypress' );
     739            $button = __( 'Update',           'buddypress' );
     740            $action = add_query_arg( array(
     741                'page'     => 'bp-profile-setup',
     742                'mode'     => 'edit_group',
     743                'group_id' => (int) $this->id
     744            ), $users_url );
    730745        } ?>
    731746
    … …  
    814829                                            </div>
    815830                                            <div id="delete-action">
    816                                                 <a href="users.php?page=bp-profile-setup" class="deletion"><?php _e( 'Cancel', 'buddypress' ); ?></a>
     831                                                <a href="<?php echo esc_url( $cancel_url ); ?>" class="deletion"><?php _e( 'Cancel', 'buddypress' ); ?></a>
    817832                                            </div>
    818833                                            <div class="clear"></div>
Note: See TracChangeset for help on using the changeset viewer.