WordPress.org

Make WordPress Core

Ticket #42315: 42315.diff

File 42315.diff, 2.2 KB (added by susiyanti, 3 years ago)

My first time contributing. Changes I made:Redirect to the home page rather than the admin dashboard Not show the inaccessible links in the admin toolbar

  • src/wp-includes/admin-bar.php

     
    222222
    223223        if ( current_user_can( 'read' ) ) {
    224224                $profile_url = get_edit_profile_url( $user_id );
    225         } elseif ( is_multisite() ) {
     225        } elseif ( is_multisite() && current_user_can( 'read' ) ) {
    226226                $profile_url = get_dashboard_url( $user_id, 'profile.php' );
    227227        } else {
    228228                $profile_url = false;
     
    263263
    264264        if ( current_user_can( 'read' ) ) {
    265265                $profile_url = get_edit_profile_url( $user_id );
    266         } elseif ( is_multisite() ) {
     266        } elseif ( is_multisite() && current_user_can( 'read' )) {
    267267                $profile_url = get_dashboard_url( $user_id, 'profile.php' );
    268268        } else {
    269269                $profile_url = false;
     
    465465                $my_sites_url = admin_url( 'my-sites.php' );
    466466        }
    467467
    468         $wp_admin_bar->add_menu(
    469                 array(
    470                         'id'    => 'my-sites',
    471                         'title' => __( 'My Sites' ),
    472                         'href'  => $my_sites_url,
    473                 )
    474         );
     468        if ( current_user_can( 'read' ) ) {
     469                $wp_admin_bar->add_menu(
     470                        array(
     471                                'id'    => 'my-sites',
     472                                'title' => __( 'My Sites' ),
     473                                'href'  => $my_sites_url,
     474                        )
     475                );
     476        }
    475477
    476478        if ( current_user_can( 'manage_network' ) ) {
    477479                $wp_admin_bar->add_group(
     
    597599                                        'href'   => admin_url(),
    598600                                )
    599601                        );
    600                 } else {
    601                         $wp_admin_bar->add_menu(
    602                                 array(
    603                                         'parent' => 'my-sites-list',
    604                                         'id'     => $menu_id,
    605                                         'title'  => $blavatar . $blogname,
    606                                         'href'   => home_url(),
    607                                 )
    608                         );
    609602                }
    610603
    611604                if ( current_user_can( get_post_type_object( 'post' )->cap->create_posts ) ) {
  • src/wp-login.php

     
    996996                                if ( is_multisite() && ! get_active_blog_for_user( $user->ID ) && ! is_super_admin( $user->ID ) ) {
    997997                                        $redirect_to = user_admin_url();
    998998                                } elseif ( is_multisite() && ! $user->has_cap( 'read' ) ) {
    999                                         $redirect_to = get_dashboard_url( $user->ID );
     999                                        $redirect_to = home_url();
    10001000                                } elseif ( ! $user->has_cap( 'edit_posts' ) ) {
    10011001                                        $redirect_to = $user->has_cap( 'read' ) ? admin_url( 'profile.php' ) : home_url();
    10021002                                }