WordPress.org
Get WordPress

Make WordPress Core

Changeset 47083


Ignore:
Timestamp:
01/18/2020 12:24:57 AM (5 years ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Remove unnecessary // phpcs:ignore instances in WP_Debug_Data.

WP_Theme::__get() accepts Name, Version, and Author theme properties in lower case too.

Additionally, remove one short ternary instance, not allowed per the coding standards.

See #49222.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/includes/class-wp-debug-data.php

    r46586 r47083  
    8888                'permalink'              => array(
    8989                    'label' => __( 'Permalink structure' ),
    90                     'value' => $permalink_structure ?: __( 'No permalink structure set' ),
     90                    'value' => $permalink_structure ? $permalink_structure : __( 'No permalink structure set' ),
    9191                    'debug' => $permalink_structure,
    9292                ),
    … …  
    880880        $theme_updates = get_theme_updates();
    881881
    882         // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
    883         $active_theme_version       = $active_theme->Version;
     882        $active_theme_version       = $active_theme->version;
    884883        $active_theme_version_debug = $active_theme_version;
    885884
    … …  
    892891        }
    893892
    894         $active_theme_author_uri = $active_theme->offsetGet( 'Author URI' );
     893        $active_theme_author_uri = $active_theme->display( 'AuthorURI' );
    895894
    896895        if ( $active_theme->parent_theme ) {
    … …  
    914913            'name'           => array(
    915914                'label' => __( 'Name' ),
    916                 // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
    917915                'value' => sprintf(
    918916                    /* translators: 1: Theme name. 2: Theme slug. */
    919917                    __( '%1$s (%2$s)' ),
    920                     // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
    921                     $active_theme->Name,
     918                    $active_theme->name,
    922919                    $active_theme->stylesheet
    923920                ),
    … …  
    930927            'author'         => array(
    931928                'label' => __( 'Author' ),
    932                 // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
    933                 'value' => wp_kses( $active_theme->Author, array() ),
     929                'value' => wp_kses( $active_theme->author, array() ),
    934930            ),
    935931            'author_website' => array(
    … …  
    956952
    957953        if ( $parent_theme ) {
    958             // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
    959             $parent_theme_version       = $parent_theme->Version;
     954            $parent_theme_version       = $parent_theme->version;
    960955            $parent_theme_version_debug = $parent_theme_version;
    961956
    … …  
    968963            }
    969964
    970             $parent_theme_author_uri = $parent_theme->offsetGet( 'Author URI' );
     965            $parent_theme_author_uri = $parent_theme->display( 'AuthorURI' );
    971966
    972967            $info['wp-parent-theme']['fields'] = array(
    973968                'name'           => array(
    974969                    'label' => __( 'Name' ),
    975                     // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
    976970                    'value' => sprintf(
    977971                        /* translators: 1: Theme name. 2: Theme slug. */
    978972                        __( '%1$s (%2$s)' ),
    979                         // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
    980                         $parent_theme->Name,
     973                        $parent_theme->name,
    981974                        $parent_theme->stylesheet
    982975                    ),
    … …  
    989982                'author'         => array(
    990983                    'label' => __( 'Author' ),
    991                     // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
    992                     'value' => wp_kses( $parent_theme->Author, array() ),
     984                    'value' => wp_kses( $parent_theme->author, array() ),
    993985                ),
    994986                'author_website' => array(
    … …  
    10181010            }
    10191011
    1020             // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
    1021             $theme_version = $theme->Version;
    1022             // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
    1023             $theme_author = $theme->Author;
     1012            $theme_version = $theme->version;
     1013            $theme_author  = $theme->author;
    10241014
    10251015            // Sanitize
    … …  
    10531043            }
    10541044
    1055             // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
    1056             $info['wp-themes-inactive']['fields'][ sanitize_text_field( $theme->Name ) ] = array(
     1045            $info['wp-themes-inactive']['fields'][ sanitize_text_field( $theme->name ) ] = array(
    10571046                'label' => sprintf(
    10581047                    /* translators: 1: Theme name. 2: Theme slug. */
    10591048                    __( '%1$s (%2$s)' ),
    1060                     // phpcs:ignore WordPress.NamingConventions.ValidVariableName.UsedPropertyNotSnakeCase
    1061                     $theme->Name,
     1049                    $theme->name,
    10621050                    $theme_slug
    10631051                ),
Note: See TracChangeset for help on using the changeset viewer.