WordPress.org
Get WordPress

Make WordPress Core

Changeset 48764


Ignore:
Timestamp:
08/08/2020 09:02:18 PM (6 years ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Fix WPCS issues in wp-admin/includes/plugin.php.

Includes minor code layout fixes for better readability.

Props rnaby for initial patch.
See #50767, #43848.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/includes/plugin.php

    r48762 r48764  
    652652        }
    653653
    654         if ( ( $network_wide && ! isset( $current[ $plugin ] ) ) || ( ! $network_wide && ! in_array( $plugin, $current, true ) ) ) {
     654        if ( $network_wide && ! isset( $current[ $plugin ] )
     655                || ! $network_wide && ! in_array( $plugin, $current, true )
     656        ) {
    655657                if ( ! empty( $redirect ) ) {
    656658                        // We'll override this later if the plugin can be included without fatal error.
    … …  
    765767                }
    766768
    767                 $network_deactivating = false !== $network_wide && is_plugin_active_for_network( $plugin );
     769                $network_deactivating = ( false !== $network_wide ) && is_plugin_active_for_network( $plugin );
    768770
    769771                if ( ! $silent ) {
    … …  
    979981                // If plugin is in its own directory, recursively delete the directory.
    980982                // Base check on if plugin includes directory separator AND that it's not the root plugin folder.
    981                 if ( strpos( $plugin_file, '/' ) && $this_plugin_dir != $plugins_dir ) {
     983                if ( strpos( $plugin_file, '/' ) && $this_plugin_dir !== $plugins_dir ) {
    982984                        $deleted = $wp_filesystem->delete( $this_plugin_dir, true );
    983985                } else {
    … …  
    11511153        $compatible_php = is_php_version_compatible( $requirements['requires_php'] );
    11521154
    1153         /* translators: %s: URL to Update PHP page. */
    11541155        $php_update_message = '</p><p>' . sprintf(
     1156                /* translators: %s: URL to Update PHP page. */
    11551157                __( '<a href="%s">Learn more about updating PHP</a>.' ),
    11561158                esc_url( wp_get_update_php_url() )
    … …  
    14151417         * this case, don't automatically add a link back to avoid duplication.
    14161418         */
    1417         if ( ! isset( $submenu[ $parent_slug ] ) && $menu_slug != $parent_slug ) {
     1419        if ( ! isset( $submenu[ $parent_slug ] ) && $menu_slug !== $parent_slug ) {
    14181420                foreach ( (array) $menu as $parent_menu ) {
    1419                         if ( $parent_menu[2] == $parent_slug && current_user_can( $parent_menu[1] ) ) {
     1421                        if ( $parent_menu[2] === $parent_slug && current_user_can( $parent_menu[1] ) ) {
    14201422                                $submenu[ $parent_slug ][] = array_slice( $parent_menu, 0, 4 );
    14211423                        }
    … …  
    17681770
    17691771        foreach ( $menu as $i => $item ) {
    1770                 if ( $menu_slug == $item[2] ) {
     1772                if ( $menu_slug === $item[2] ) {
    17711773                        unset( $menu[ $i ] );
    17721774                        return $item;
    … …  
    17961798
    17971799        foreach ( $submenu[ $menu_slug ] as $i => $item ) {
    1798                 if ( $submenu_slug == $item[2] ) {
     1800                if ( $submenu_slug === $item[2] ) {
    17991801                        unset( $submenu[ $menu_slug ][ $i ] );
    18001802                        return $item;
    … …  
    18231825        if ( isset( $_parent_pages[ $menu_slug ] ) ) {
    18241826                $parent_slug = $_parent_pages[ $menu_slug ];
     1827
    18251828                if ( $parent_slug && ! isset( $_parent_pages[ $parent_slug ] ) ) {
    18261829                        $url = admin_url( add_query_arg( 'page', $menu_slug, $parent_slug ) );
    … …  
    18711874                        $parent = $_wp_real_parent_file[ $parent ];
    18721875                }
     1876
    18731877                return $parent;
    18741878        }
    … …  
    18761880        if ( 'admin.php' === $pagenow && isset( $plugin_page ) ) {
    18771881                foreach ( (array) $menu as $parent_menu ) {
    1878                         if ( $parent_menu[2] == $plugin_page ) {
     1882                        if ( $parent_menu[2] === $plugin_page ) {
    18791883                                $parent_file = $plugin_page;
     1884
    18801885                                if ( isset( $_wp_real_parent_file[ $parent_file ] ) ) {
    18811886                                        $parent_file = $_wp_real_parent_file[ $parent_file ];
    18821887                                }
     1888
    18831889                                return $parent_file;
    18841890                        }
    … …  
    18861892                if ( isset( $_wp_menu_nopriv[ $plugin_page ] ) ) {
    18871893                        $parent_file = $plugin_page;
     1894
    18881895                        if ( isset( $_wp_real_parent_file[ $parent_file ] ) ) {
    18891896                                        $parent_file = $_wp_real_parent_file[ $parent_file ];
    18901897                        }
     1898
    18911899                        return $parent_file;
    18921900                }
    … …  
    18951903        if ( isset( $plugin_page ) && isset( $_wp_submenu_nopriv[ $pagenow ][ $plugin_page ] ) ) {
    18961904                $parent_file = $pagenow;
     1905
    18971906                if ( isset( $_wp_real_parent_file[ $parent_file ] ) ) {
    18981907                        $parent_file = $_wp_real_parent_file[ $parent_file ];
    18991908                }
     1909
    19001910                return $parent_file;
    19011911        }
    … …  
    19061916                                $parent = $_wp_real_parent_file[ $parent ];
    19071917                        }
    1908                         if ( ! empty( $typenow ) && ( "$pagenow?post_type=$typenow" === $submenu_array[2] ) ) {
     1918
     1919                        if ( ! empty( $typenow ) && "$pagenow?post_type=$typenow" === $submenu_array[2] ) {
    19091920                                $parent_file = $parent;
    19101921                                return $parent;
    1911                         } elseif ( $submenu_array[2] == $pagenow && empty( $typenow ) && ( empty( $parent_file ) || false === strpos( $parent_file, '?' ) ) ) {
     1922                        } elseif ( empty( $typenow ) && $pagenow === $submenu_array[2]
     1923                                && ( empty( $parent_file ) || false === strpos( $parent_file, '?' ) )
     1924                        ) {
    19121925                                $parent_file = $parent;
    19131926                                return $parent;
    1914                         } elseif ( isset( $plugin_page ) && ( $plugin_page == $submenu_array[2] ) ) {
     1927                        } elseif ( isset( $plugin_page ) && $plugin_page === $submenu_array[2] ) {
    19151928                                $parent_file = $parent;
    19161929                                return $parent;
    … …  
    19541967                foreach ( (array) $menu as $menu_array ) {
    19551968                        if ( isset( $menu_array[3] ) ) {
    1956                                 if ( $menu_array[2] == $pagenow ) {
     1969                                if ( $menu_array[2] === $pagenow ) {
    19571970                                        $title = $menu_array[3];
    19581971                                        return $menu_array[3];
    1959                                 } elseif ( isset( $plugin_page ) && ( $plugin_page == $menu_array[2] ) && ( $hook == $menu_array[5] ) ) {
     1972                                } elseif ( isset( $plugin_page ) && $plugin_page === $menu_array[2] && $hook === $menu_array[5] ) {
    19601973                                        $title = $menu_array[3];
    19611974                                        return $menu_array[3];
    … …  
    19691982                foreach ( array_keys( $submenu ) as $parent ) {
    19701983                        foreach ( $submenu[ $parent ] as $submenu_array ) {
    1971                                 if ( isset( $plugin_page ) &&
    1972                                         ( $plugin_page == $submenu_array[2] ) &&
    1973                                         (
    1974                                                 ( $parent == $pagenow ) ||
    1975                                                 ( $parent == $plugin_page ) ||
    1976                                                 ( $plugin_page == $hook ) ||
    1977                                                 ( 'admin.php' === $pagenow && $parent1 != $submenu_array[2] ) ||
    1978                                                 ( ! empty( $typenow ) && $parent == $pagenow . '?post_type=' . $typenow )
    1979                                         )
     1984                                if ( isset( $plugin_page )
     1985                                        && ( $plugin_page === $submenu_array[2] )
     1986                                        && ( $pagenow === $parent
     1987                                                || $plugin_page === $parent
     1988                                                || $plugin_page === $hook
     1989                                                || 'admin.php' === $pagenow && $parent1 !== $submenu_array[2]
     1990                                                || ! empty( $typenow ) && "$pagenow?post_type=$typenow" === $parent )
    19801991                                        ) {
    19811992                                                $title = $submenu_array[3];
    … …  
    19831994                                }
    19841995
    1985                                 if ( $submenu_array[2] != $pagenow || isset( $_GET['page'] ) ) { // Not the current page.
     1996                                if ( $submenu_array[2] !== $pagenow || isset( $_GET['page'] ) ) { // Not the current page.
    19861997                                        continue;
    19871998                                }
    … …  
    19982009                if ( empty( $title ) ) {
    19992010                        foreach ( $menu as $menu_array ) {
    2000                                 if ( isset( $plugin_page ) &&
    2001                                         ( $plugin_page == $menu_array[2] ) &&
    2002                                         ( 'admin.php' === $pagenow ) &&
    2003                                         ( $parent1 == $menu_array[2] ) ) {
     2011                                if ( isset( $plugin_page )
     2012                                        && $plugin_page === $menu_array[2]
     2013                                        && 'admin.php' === $pagenow
     2014                                        && $parent1 === $menu_array[2]
     2015                                ) {
    20042016                                                $title = $menu_array[3];
    20052017                                                return $menu_array[3];
    … …  
    21272139        }
    21282140
    2129         if ( isset( $plugin_page ) && ( $plugin_page == $parent ) && isset( $_wp_menu_nopriv[ $plugin_page ] ) ) {
     2141        if ( isset( $plugin_page ) && ( $plugin_page === $parent ) && isset( $_wp_menu_nopriv[ $plugin_page ] ) ) {
    21302142                return false;
    21312143        }
    … …  
    21332145        if ( isset( $submenu[ $parent ] ) ) {
    21342146                foreach ( $submenu[ $parent ] as $submenu_array ) {
    2135                         if ( isset( $plugin_page ) && ( $submenu_array[2] == $plugin_page ) ) {
     2147                        if ( isset( $plugin_page ) && ( $submenu_array[2] === $plugin_page ) ) {
    21362148                                return current_user_can( $submenu_array[1] );
    2137                         } elseif ( $submenu_array[2] == $pagenow ) {
     2149                        } elseif ( $submenu_array[2] === $pagenow ) {
    21382150                                return current_user_can( $submenu_array[1] );
    21392151                        }
    … …  
    21422154
    21432155        foreach ( $menu as $menu_array ) {
    2144                 if ( $menu_array[2] == $parent ) {
     2156                if ( $menu_array[2] === $parent ) {
    21452157                        return current_user_can( $menu_array[1] );
    21462158                }
Note: See TracChangeset for help on using the changeset viewer.