WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/13/2024 12:25:43 PM (21 months ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Consistently escape URLs in wp-admin/themes.php.

Includes:

  • Wrapping long lines for better readability.
  • Bringing some consistency to the placement of href and aria-label attributes.
  • Adding missing aria-label attributes for Live Preview links.

Follow-up to [26726], [52020], [51083].

Props patelketan, sainathpoojary, SergeyBiryukov.
Fixes #62405.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/themes.php

    r58879 r59400  
    584584        $details_aria_label = sprintf( _x( 'View Theme Details for %s', 'theme' ), $theme['name'] );
    585585        ?>
    586         <button type="button" aria-label="<?php echo esc_attr( $details_aria_label ); ?>" class="more-details" id="<?php echo esc_attr( $aria_action ); ?>"><?php _e( 'Theme Details' ); ?></button>
     586        <button type="button" class="more-details" id="<?php echo esc_attr( $aria_action ); ?>"
     587                aria-label="<?php echo esc_attr( $details_aria_label ); ?>"
     588        ><?php _e( 'Theme Details' ); ?></button>
    587589        <div class="theme-author">
    588590                <?php
    … …  
    608610                                $customize_aria_label = sprintf( _x( 'Customize %s', 'theme' ), $theme['name'] );
    609611                                ?>
    610                                 <a aria-label="<?php echo esc_attr( $customize_aria_label ); ?>" class="button button-primary customize load-customize hide-if-no-customize" href="<?php echo $theme['actions']['customize']; ?>"><?php _e( 'Customize' ); ?></a>
     612                                <a class="button button-primary customize load-customize hide-if-no-customize"
     613                                        href="<?php echo esc_url( $theme['actions']['customize'] ); ?>"
     614                                        aria-label="<?php echo esc_attr( $customize_aria_label ); ?>"
     615                                ><?php _e( 'Customize' ); ?></a>
    611616                        <?php } ?>
    612617                <?php } elseif ( $theme['compatibleWP'] && $theme['compatiblePHP'] ) { ?>
    … …  
    615620                        $aria_label = sprintf( _x( 'Activate %s', 'theme' ), '{{ data.name }}' );
    616621                        ?>
    617                         <a class="button activate" href="<?php echo $theme['actions']['activate']; ?>" aria-label="<?php echo esc_attr( $aria_label ); ?>"><?php _e( 'Activate' ); ?></a>
     622                        <a class="button activate"
     623                                href="<?php echo esc_url( $theme['actions']['activate'] ); ?>"
     624                                aria-label="<?php echo esc_attr( $aria_label ); ?>"
     625                        ><?php _e( 'Activate' ); ?></a>
     626
    618627                        <?php
    619628                        // Only classic themes require the "customize" capability.
    … …  
    622631                                $live_preview_aria_label = sprintf( _x( 'Live Preview %s', 'theme' ), '{{ data.name }}' );
    623632                                ?>
    624                                 <a aria-label="<?php echo esc_attr( $live_preview_aria_label ); ?>" class="button button-primary load-customize hide-if-no-customize" href="<?php echo $theme['actions']['customize']; ?>"><?php _e( 'Live Preview' ); ?></a>
     633                                <a class="button button-primary load-customize hide-if-no-customize"
     634                                        href="<?php echo esc_url( $theme['actions']['customize'] ); ?>"
     635                                        aria-label="<?php echo esc_attr( $live_preview_aria_label ); ?>"
     636                                ><?php _e( 'Live Preview' ); ?></a>
    625637                        <?php } ?>
    626638                <?php } else { ?>
    … …  
    629641                        $aria_label = sprintf( _x( 'Cannot Activate %s', 'theme' ), '{{ data.name }}' );
    630642                        ?>
    631                         <a class="button disabled" aria-label="<?php echo esc_attr( $aria_label ); ?>"><?php _ex( 'Cannot Activate', 'theme' ); ?></a>
    632                         <?php if ( ! $theme['blockTheme'] && current_user_can( 'edit_theme_options' ) && current_user_can( 'customize' ) ) { ?>
    633                                 <a class="button button-primary hide-if-no-customize disabled"><?php _e( 'Live Preview' ); ?></a>
     643                        <a class="button disabled"
     644                                aria-label="<?php echo esc_attr( $aria_label ); ?>"
     645                        ><?php _ex( 'Cannot Activate', 'theme' ); ?></a>
     646
     647                        <?php
     648                        if ( ! $theme['blockTheme'] && current_user_can( 'edit_theme_options' ) && current_user_can( 'customize' ) ) {
     649                                /* translators: %s: Theme name. */
     650                                $live_preview_aria_label = sprintf( _x( 'Live Preview %s', 'theme' ), '{{ data.name }}' );
     651                                ?>
     652                                <a class="button button-primary hide-if-no-customize disabled"
     653                                        aria-label="<?php echo esc_attr( $live_preview_aria_label ); ?>"
     654                                ><?php _e( 'Live Preview' ); ?></a>
    634655                        <?php } ?>
    635656                <?php } ?>
    … …  
    955976        $details_aria_label = sprintf( _x( 'View Theme Details for %s', 'theme' ), '{{ data.name }}' );
    956977        ?>
    957         <button type="button" aria-label="<?php echo esc_attr( $details_aria_label ); ?>" class="more-details" id="{{ data.id }}-action"><?php _e( 'Theme Details' ); ?></button>
     978        <button type="button" class="more-details" id="{{ data.id }}-action"
     979                aria-label="<?php echo esc_attr( $details_aria_label ); ?>"
     980        ><?php _e( 'Theme Details' ); ?></button>
    958981        <div class="theme-author">
    959982                <?php
    … …  
    9791002                                        $customize_aria_label = sprintf( _x( 'Customize %s', 'theme' ), '{{ data.name }}' );
    9801003                                        ?>
    981                                         <a aria-label="<?php echo esc_attr( $customize_aria_label ); ?>" class="button button-primary customize load-customize hide-if-no-customize" href="{{{ data.actions.customize }}}"><?php _e( 'Customize' ); ?></a>
     1004                                        <a class="button button-primary customize load-customize hide-if-no-customize"
     1005                                                href="{{{ data.actions.customize }}}"
     1006                                                aria-label="<?php echo esc_attr( $customize_aria_label ); ?>"
     1007                                        ><?php _e( 'Customize' ); ?></a>
    9821008                                <# } #>
    9831009                        <# } else { #>
    … …  
    9871013                                        $aria_label = sprintf( _x( 'Activate %s', 'theme' ), '{{ data.name }}' );
    9881014                                        ?>
    989                                         <a class="button activate" href="{{{ data.actions.activate }}}" aria-label="<?php echo esc_attr( $aria_label ); ?>"><?php _e( 'Activate' ); ?></a>
     1015                                        <a class="button activate"
     1016                                                href="{{{ data.actions.activate }}}"
     1017                                                aria-label="<?php echo esc_attr( $aria_label ); ?>"
     1018                                        ><?php _e( 'Activate' ); ?></a>
     1019
    9901020                                        <?php
    9911021                                        /* translators: %s: Theme name. */
    9921022                                        $live_preview_aria_label = sprintf( _x( 'Live Preview %s', 'theme' ), '{{ data.name }}' );
    9931023                                        ?>
    994                                         <a aria-label="<?php echo esc_attr( $live_preview_aria_label ); ?>" class="button button-primary load-customize hide-if-no-customize" href="{{{ data.actions.customize }}}"><?php _e( 'Live Preview' ); ?></a>
     1024                                        <a class="button button-primary load-customize hide-if-no-customize"
     1025                                                href="{{{ data.actions.customize }}}"
     1026                                                aria-label="<?php echo esc_attr( $live_preview_aria_label ); ?>"
     1027                                        ><?php _e( 'Live Preview' ); ?></a>
    9951028                                <# } else { #>
    9961029                                        <?php
    … …  
    9981031                                        $aria_label = sprintf( _x( 'Cannot Activate %s', 'theme' ), '{{ data.name }}' );
    9991032                                        ?>
    1000                                         <a class="button disabled" aria-label="<?php echo esc_attr( $aria_label ); ?>"><?php _ex( 'Cannot Activate', 'theme' ); ?></a>
     1033                                        <a class="button disabled"
     1034                                                aria-label="<?php echo esc_attr( $aria_label ); ?>"
     1035                                        ><?php _ex( 'Cannot Activate', 'theme' ); ?></a>
     1036
    10011037                                        <# if ( ! data.blockTheme ) { #>
    1002                                                 <a class="button button-primary hide-if-no-customize disabled"><?php _e( 'Live Preview' ); ?></a>
     1038                                                <?php
     1039                                                /* translators: %s: Theme name. */
     1040                                                $live_preview_aria_label = sprintf( _x( 'Live Preview %s', 'theme' ), '{{ data.name }}' );
     1041                                                ?>
     1042                                                <a class="button button-primary hide-if-no-customize disabled"
     1043                                                        aria-label="<?php echo esc_attr( $live_preview_aria_label ); ?>"
     1044                                                ><?php _e( 'Live Preview' ); ?></a>
    10031045                                        <# } #>
    10041046                                <# } #>
    … …  
    12121254                <div class="theme-actions">
    12131255                        <div class="active-theme">
    1214                                 <a href="{{{ data.actions.customize }}}" class="button button-primary customize load-customize hide-if-no-customize"><?php _e( 'Customize' ); ?></a>
     1256                                <a class="button button-primary customize load-customize hide-if-no-customize"
     1257                                        href="{{{ data.actions.customize }}}"
     1258                                ><?php _e( 'Customize' ); ?></a>
    12151259                                <?php echo implode( ' ', $current_theme_actions ); ?>
    12161260                        </div>
     1261
    12171262                        <div class="inactive-theme">
    12181263                                <# if ( data.compatibleWP && data.compatiblePHP ) { #>
    1219                                         <?php
    1220                                         /* translators: %s: Theme name. */
    1221                                         $aria_label = sprintf( _x( 'Activate %s', 'theme' ), '{{ data.name }}' );
    1222                                         ?>
    12231264                                        <# if ( ! data.blockTheme ) { #>
    1224                                                 <a href="{{{ data.actions.customize }}}" class="button button-primary load-customize hide-if-no-customize"><?php _e( 'Live Preview' ); ?></a>
     1265                                                <?php
     1266                                                /* translators: %s: Theme name. */
     1267                                                $live_preview_aria_label = sprintf( _x( 'Live Preview %s', 'theme' ), '{{ data.name }}' );
     1268                                                ?>
     1269                                                <a class="button button-primary load-customize hide-if-no-customize"
     1270                                                        href="{{{ data.actions.customize }}}"
     1271                                                        aria-label="<?php echo esc_attr( $live_preview_aria_label ); ?>"
     1272                                                ><?php _e( 'Live Preview' ); ?></a>
    12251273                                        <# } #>
     1274
    12261275                                        <# if ( data.actions.activate ) { #>
    1227                                                 <a href="{{{ data.actions.activate }}}" class="button activate" aria-label="<?php echo esc_attr( $aria_label ); ?>"><?php _e( 'Activate' ); ?></a>
     1276                                                <?php
     1277                                                /* translators: %s: Theme name. */
     1278                                                $aria_label = sprintf( _x( 'Activate %s', 'theme' ), '{{ data.name }}' );
     1279                                                ?>
     1280                                                <a class="button activate"
     1281                                                        href="{{{ data.actions.activate }}}"
     1282                                                        aria-label="<?php echo esc_attr( $aria_label ); ?>"
     1283                                                ><?php _e( 'Activate' ); ?></a>
    12281284                                        <# } #>
    12291285                                <# } else { #>
    1230                                         <?php
    1231                                         /* translators: %s: Theme name. */
    1232                                         $aria_label = sprintf( _x( 'Cannot Activate %s', 'theme' ), '{{ data.name }}' );
    1233                                         ?>
    12341286                                        <# if ( ! data.blockTheme ) { #>
    1235                                                 <a class="button button-primary hide-if-no-customize disabled"><?php _e( 'Live Preview' ); ?></a>
     1287                                                <?php
     1288                                                /* translators: %s: Theme name. */
     1289                                                $live_preview_aria_label = sprintf( _x( 'Live Preview %s', 'theme' ), '{{ data.name }}' );
     1290                                                ?>
     1291                                                <a class="button button-primary hide-if-no-customize disabled"
     1292                                                        aria-label="<?php echo esc_attr( $live_preview_aria_label ); ?>"
     1293                                                ><?php _e( 'Live Preview' ); ?></a>
    12361294                                        <# } #>
     1295
    12371296                                        <# if ( data.actions.activate ) { #>
    1238                                                 <a class="button disabled" aria-label="<?php echo esc_attr( $aria_label ); ?>"><?php _ex( 'Cannot Activate', 'theme' ); ?></a>
     1297                                                <?php
     1298                                                /* translators: %s: Theme name. */
     1299                                                $aria_label = sprintf( _x( 'Cannot Activate %s', 'theme' ), '{{ data.name }}' );
     1300                                                ?>
     1301                                                <a class="button disabled"
     1302                                                        aria-label="<?php echo esc_attr( $aria_label ); ?>"
     1303                                                ><?php _ex( 'Cannot Activate', 'theme' ); ?></a>
    12391304                                        <# } #>
    12401305                                <# } #>
    … …  
    12461311                                $aria_label = sprintf( _x( 'Delete %s', 'theme' ), '{{ data.name }}' );
    12471312                                ?>
    1248                                 <a href="{{{ data.actions['delete'] }}}" class="button delete-theme" aria-label="<?php echo esc_attr( $aria_label ); ?>"><?php _e( 'Delete' ); ?></a>
     1313                                <a class="button delete-theme"
     1314                                        href="{{{ data.actions['delete'] }}}"
     1315                                        aria-label="<?php echo esc_attr( $aria_label ); ?>"
     1316                                ><?php _e( 'Delete' ); ?></a>
    12491317                        <# } #>
    12501318                </div>
Note: See TracChangeset for help on using the changeset viewer.