From 9eb6dd69c85efb3a0017a736004ab202dd824ac7 Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Sat, 8 Aug 2020 20:22:34 +0000 Subject: [PATCH] Coding Standards: Simplify some parts of `user_can_access_admin_page()` for better readability. Props rnaby, valentinbora, khag7. Fixes #43848. git-svn-id: https://develop.svn.wordpress.org/trunk@48762 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-admin/includes/plugin.php | 20 +++++--------------- 1 file changed, 5 insertions(+), 15 deletions(-) diff --git a/src/wp-admin/includes/plugin.php b/src/wp-admin/includes/plugin.php index c217907436..07c9304a71 100644 --- a/src/wp-admin/includes/plugin.php +++ b/src/wp-admin/includes/plugin.php @@ -2113,6 +2113,7 @@ function user_can_access_admin_page() { if ( isset( $plugin_page ) && isset( $_wp_menu_nopriv[ $plugin_page ] ) ) { return false; } + foreach ( array_keys( $_wp_submenu_nopriv ) as $key ) { if ( isset( $_wp_submenu_nopriv[ $key ][ $pagenow ] ) ) { return false; @@ -2121,6 +2122,7 @@ function user_can_access_admin_page() { return false; } } + return true; } @@ -2131,28 +2133,16 @@ function user_can_access_admin_page() { if ( isset( $submenu[ $parent ] ) ) { foreach ( $submenu[ $parent ] as $submenu_array ) { if ( isset( $plugin_page ) && ( $submenu_array[2] == $plugin_page ) ) { - if ( current_user_can( $submenu_array[1] ) ) { - return true; - } else { - return false; - } + return current_user_can( $submenu_array[1] ); } elseif ( $submenu_array[2] == $pagenow ) { - if ( current_user_can( $submenu_array[1] ) ) { - return true; - } else { - return false; - } + return current_user_can( $submenu_array[1] ); } } } foreach ( $menu as $menu_array ) { if ( $menu_array[2] == $parent ) { - if ( current_user_can( $menu_array[1] ) ) { - return true; - } else { - return false; - } + return current_user_can( $menu_array[1] ); } }