Make WordPress Core

Changeset 48764


Ignore:
Timestamp:
08/08/2020 09:02:18 PM (5 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.