WordPress.org
Get WordPress

Make WordPress Core

Ticket #48170: 48170.3.diff

File 48170.3.diff, 12.9 KB (added by audrasjb, 4 years ago)

Coding standards fixes

  • src/wp-includes/widgets/class-wp-nav-menu-widget.php 

    diff --git a/src/wp-includes/widgets/class-wp-nav-menu-widget.php b/src/wp-includes/widgets/class-wp-nav-menu-widget.php
index 1edaeac464..9972e2a2c7 100644
    a b class WP_Nav_Menu_Widget extends WP_Widget { 
    5757                        echo $args['before_title'] . $title . $args['after_title'];
    5858                }
    5959
    60                 $nav_menu_args = array(
    61                         'fallback_cb' => '',
    62                         'menu'        => $nav_menu,
    63                 );
     60                $format = current_theme_supports( 'html5', 'navigation' ) ? 'html5' : 'xhtml';
     61
     62                /**
     63                 * Filters the HTML format of widgets with navigation links.
     64                 *
     65                 * @since 5.5.0
     66                 *
     67                 * @param string $format The type of markup to use in widgets with navigation links.
     68                 *                       Accepts 'html5', 'xhtml'.
     69                 */
     70                $format = apply_filters( 'navigation_widgets_format', $format );
     71
     72                if ( 'html5' === $format ) {
     73                        $nav_menu_args = array(
     74                                'fallback_cb' => '',
     75                                'menu'        => $nav_menu,
     76                                'items_wrap'  => '<nav role="navigation"><ul id="%1$s" class="%2$s">%3$s</ul></nav>',
     77                        );
     78                } else {
     79                        $nav_menu_args = array(
     80                                'fallback_cb' => '',
     81                                'menu'        => $nav_menu,
     82                        );
     83                }
    6484
    6585                /**
    6686                 * Filters the arguments for the Navigation Menu widget.

  • src/wp-includes/widgets/class-wp-widget-archives.php 

    diff --git a/src/wp-includes/widgets/class-wp-widget-archives.php b/src/wp-includes/widgets/class-wp-widget-archives.php
index 2dfd60c7a5..fa2911564b 100644
    a b class WP_Widget_Archives extends WP_Widget { 
    121121/* ]]> */
    122122</script>
    123123
    124                 <?php } else { ?>
    125                 <ul>
    126                         <?php
    127                         wp_get_archives(
    128                                 /**
    129                                  * Filters the arguments for the Archives widget.
    130                                  *
    131                                  * @since 2.8.0
    132                                  * @since 4.9.0 Added the `$instance` parameter.
    133                                  *
    134                                  * @see wp_get_archives()
    135                                  *
    136                                  * @param array $args     An array of Archives option arguments.
    137                                  * @param array $instance Array of settings for the current widget.
    138                                  */
    139                                 apply_filters(
    140                                         'widget_archives_args',
    141                                         array(
    142                                                 'type'            => 'monthly',
    143                                                 'show_post_count' => $count,
    144                                         ),
    145                                         $instance
    146                                 )
    147                         );
     124                <?php } else {
     125                        $format = current_theme_supports( 'html5', 'navigation' ) ? 'html5' : 'xhtml';
     126
     127                        /**
     128                         * Filters the HTML format of widgets with navigation links.
     129                         *
     130                         * @since 5.5.0
     131                         *
     132                         * @param string $format The type of markup to use in widgets with navigation links.
     133                         *                       Accepts 'html5', 'xhtml'.
     134                         */
     135                        $format = apply_filters( 'navigation_widgets_format', $format );
    148136                        ?>
    149                 </ul>
    150                         <?php
    151                 }
    152137
     138                        <?php if ( 'html5' === $format ) : ?>
     139                                <nav role="navigation">
     140                        <?php endif; ?>
     141
     142                        <ul>
     143                                <?php
     144                                wp_get_archives(
     145                                        /**
     146                                         * Filters the arguments for the Archives widget.
     147                                         *
     148                                         * @since 2.8.0
     149                                         * @since 4.9.0 Added the `$instance` parameter.
     150                                         *
     151                                         * @see wp_get_archives()
     152                                         *
     153                                         * @param array $args     An array of Archives option arguments.
     154                                         * @param array $instance Array of settings for the current widget.
     155                                         */
     156                                        apply_filters(
     157                                                'widget_archives_args',
     158                                                array(
     159                                                        'type'            => 'monthly',
     160                                                        'show_post_count' => $count,
     161                                                ),
     162                                                $instance
     163                                        )
     164                                );
     165                                ?>
     166                        </ul>
     167                        <?php if ( 'html5' === $format ) : ?>
     168                                </nav>
     169                        <?php endif; ?>
     170
     171                <?php
    153172                echo $args['after_widget'];
    154173        }
    155174

  • src/wp-includes/widgets/class-wp-widget-categories.php 

    diff --git a/src/wp-includes/widgets/class-wp-widget-categories.php b/src/wp-includes/widgets/class-wp-widget-categories.php
index 60bf6ef819..b5b675ae1d 100644
    a b class WP_Widget_Categories extends WP_Widget { 
    111111
    112112                        <?php
    113113                } else {
    114                         ?>
    115                 <ul>
    116                         <?php
    117                         $cat_args['title_li'] = '';
     114                        $format = current_theme_supports( 'html5', 'navigation' ) ? 'html5' : 'xhtml';
    118115
    119116                        /**
    120                          * Filters the arguments for the Categories widget.
     117                         * Filters the HTML format of widgets with navigation links.
    121118                         *
    122                          * @since 2.8.0
    123                          * @since 4.9.0 Added the `$instance` parameter.
     119                         * @since 5.5.0
    124120                         *
    125                          * @param array $cat_args An array of Categories widget options.
    126                          * @param array $instance Array of settings for the current widget.
     121                         * @param string $format The type of markup to use in widgets with navigation links.
     122                         *                       Accepts 'html5', 'xhtml'.
    127123                         */
    128                         wp_list_categories( apply_filters( 'widget_categories_args', $cat_args, $instance ) );
     124                        $format = apply_filters( 'navigation_widgets_format', $format );
    129125                        ?>
    130                 </ul>
     126
     127                        <?php if ( 'html5' === $format ) : ?>
     128                                <nav role="navigation">
     129                        <?php endif; ?>
     130
     131                        <ul>
     132                                <?php
     133                                $cat_args['title_li'] = '';
     134
     135                                /**
     136                                 * Filters the arguments for the Categories widget.
     137                                 *
     138                                 * @since 2.8.0
     139                                 * @since 4.9.0 Added the `$instance` parameter.
     140                                 *
     141                                 * @param array $cat_args An array of Categories widget options.
     142                                 * @param array $instance Array of settings for the current widget.
     143                                 */
     144                                wp_list_categories( apply_filters( 'widget_categories_args', $cat_args, $instance ) );
     145                                ?>
     146                        </ul>
     147
     148                        <?php if ( 'html5' === $format ) : ?>
     149                                </nav>
     150                        <?php endif; ?>
     151
    131152                        <?php
    132153                }
    133154

  • src/wp-includes/widgets/class-wp-widget-meta.php 

    diff --git a/src/wp-includes/widgets/class-wp-widget-meta.php b/src/wp-includes/widgets/class-wp-widget-meta.php
index 33742b368d..4abd5d68fd 100644
    a b class WP_Widget_Meta extends WP_Widget { 
    5252                if ( $title ) {
    5353                        echo $args['before_title'] . $title . $args['after_title'];
    5454                }
     55
     56                $format = current_theme_supports( 'html5', 'navigation' ) ? 'html5' : 'xhtml';
     57
     58                /**
     59                 * Filters the HTML format of widgets with navigation links.
     60                 *
     61                 * @since 5.5.0
     62                 *
     63                 * @param string $format The type of markup to use in widgets with navigation links.
     64                 *                       Accepts 'html5', 'xhtml'.
     65                 */
     66        $format = apply_filters( 'navigation_widgets_format', $format );
     67
     68                <?php if ( 'html5' === $format ) : ?>
     69                        <nav role="navigation">
     70                <?php endif; ?>
     71
     72                <ul>
     73
     74                <?php wp_register(); ?>
     75                <li><?php wp_loginout(); ?></li>
     76                <li><a href="<?php echo esc_url( get_bloginfo( 'rss2_url' ) ); ?>"><?php _e( 'Entries feed' ); ?></a></li>
     77                <li><a href="<?php echo esc_url( get_bloginfo( 'comments_rss2_url' ) ); ?>"><?php _e( 'Comments feed' ); ?></a></li>
     78
     79                <?php
     80                /**
     81                 * Filters the "WordPress.org" list item HTML in the Meta widget.
     82                 *
     83                 * @since 3.6.0
     84                 * @since 4.9.0 Added the `$instance` parameter.
     85                 *
     86                 * @param string $html     Default HTML for the WordPress.org list item.
     87                 * @param array  $instance Array of settings for the current widget.
     88                 */
     89                echo apply_filters(
     90                        'widget_meta_poweredby',
     91                        sprintf(
     92                                '<li><a href="%1$s">%2$s</a></li>',
     93                                esc_url( __( 'https://wordpress.org/' ) ),
     94                                __( 'WordPress.org' )
     95                        ),
     96                        $instance
     97                );
     98
     99                wp_meta();
    55100                ?>
    56                         <ul>
    57                         <?php wp_register(); ?>
    58                         <li><?php wp_loginout(); ?></li>
    59                         <li><a href="<?php echo esc_url( get_bloginfo( 'rss2_url' ) ); ?>"><?php _e( 'Entries feed' ); ?></a></li>
    60                         <li><a href="<?php echo esc_url( get_bloginfo( 'comments_rss2_url' ) ); ?>"><?php _e( 'Comments feed' ); ?></a></li>
    61                         <?php
    62                         /**
    63                          * Filters the "WordPress.org" list item HTML in the Meta widget.
    64                          *
    65                          * @since 3.6.0
    66                          * @since 4.9.0 Added the `$instance` parameter.
    67                          *
    68                          * @param string $html     Default HTML for the WordPress.org list item.
    69                          * @param array  $instance Array of settings for the current widget.
    70                          */
    71                         echo apply_filters(
    72                                 'widget_meta_poweredby',
    73                                 sprintf(
    74                                         '<li><a href="%1$s">%2$s</a></li>',
    75                                         esc_url( __( 'https://wordpress.org/' ) ),
    76                                         __( 'WordPress.org' )
    77                                 ),
    78                                 $instance
    79                         );
    80 
    81                         wp_meta();
    82                         ?>
    83                         </ul>
    84                         <?php
    85 
    86                         echo $args['after_widget'];
     101
     102                </ul>
     103
     104                <?php if ( 'html5' === $format ) : ?>
     105                        </nav>
     106                <?php endif; ?>
     107
     108                <?php
     109                echo $args['after_widget'];
    87110        }
    88111
    89112        /**

  • src/wp-includes/widgets/class-wp-widget-pages.php 

    diff --git a/src/wp-includes/widgets/class-wp-widget-pages.php b/src/wp-includes/widgets/class-wp-widget-pages.php
index 94af11d658..fa0daeffe2 100644
    a b class WP_Widget_Pages extends WP_Widget { 
    8989                        if ( $title ) {
    9090                                echo $args['before_title'] . $title . $args['after_title'];
    9191                        }
     92
     93                        $format = current_theme_supports( 'html5', 'navigation' ) ? 'html5' : 'xhtml';
     94
     95                        /**
     96                         * Filters the HTML format of widgets with navigation links.
     97                         *
     98                         * @since 5.5.0
     99                         *
     100                         * @param string $format The type of markup to use in widgets with navigation links.
     101                         *                       Accepts 'html5', 'xhtml'.
     102                         */
     103                        $format = apply_filters( 'navigation_widgets_format', $format );
    92104                        ?>
    93                 <ul>
    94                         <?php echo $out; ?>
    95                 </ul>
     105
     106                        <?php if ( 'html5' === $format ) : ?>
     107                                <nav role="navigation">
     108                        <?php endif; ?>
     109
     110                        <ul>
     111                                <?php echo $out; ?>
     112                        </ul>
     113
     114                        <?php if ( 'html5' === $format ) : ?>
     115                                </nav>
     116                        <?php endif; ?>
     117
    96118                        <?php
    97119                        echo $args['after_widget'];
    98120                }

  • src/wp-includes/widgets/class-wp-widget-recent-comments.php 

    diff --git a/src/wp-includes/widgets/class-wp-widget-recent-comments.php b/src/wp-includes/widgets/class-wp-widget-recent-comments.php
index e9da5c1bdc..b906d586f4 100644
    a b class WP_Widget_Recent_Comments extends WP_Widget { 
    125125                $recent_comments_id = ( $first_instance ) ? 'recentcomments' : "recentcomments-{$this->number}";
    126126                $first_instance     = false;
    127127
     128                $format = current_theme_supports( 'html5', 'navigation' ) ? 'html5' : 'xhtml';
     129
     130                /**
     131                 * Filters the HTML format of widgets with navigation links.
     132                 *
     133                 * @since 5.5.0
     134                 *
     135                 * @param string $format The type of markup to use in widgets with navigation links.
     136                 *                       Accepts 'html5', 'xhtml'.
     137                 */
     138                $format = apply_filters( 'navigation_widgets_format', $format );
     139
     140                if ( 'html5' === $format ) {
     141                        $output .= '<nav role="navigation">';
     142                }
     143
    128144                $output .= '<ul id="' . esc_attr( $recent_comments_id ) . '">';
    129145                if ( is_array( $comments ) && $comments ) {
    130146                        // Prime cache for associated posts. (Prime post term cache if we need it for permalinks.)
    … … class WP_Widget_Recent_Comments extends WP_Widget { 
    143159                        }
    144160                }
    145161                $output .= '</ul>';
     162
     163                if ( 'html5' === $format ) {
     164                        $output .= '</nav>';
     165                }
     166
    146167                $output .= $args['after_widget'];
    147168
    148169                echo $output;

  • src/wp-includes/widgets/class-wp-widget-recent-posts.php 

    diff --git a/src/wp-includes/widgets/class-wp-widget-recent-posts.php b/src/wp-includes/widgets/class-wp-widget-recent-posts.php
index 9767e3b4fc..672115c1ee 100644
    a b class WP_Widget_Recent_Posts extends WP_Widget { 
    8484                        return;
    8585                }
    8686                ?>
     87
    8788                <?php echo $args['before_widget']; ?>
     89
    8890                <?php
    8991                if ( $title ) {
    9092                        echo $args['before_title'] . $title . $args['after_title'];
    9193                }
     94
     95                $format = current_theme_supports( 'html5', 'navigation' ) ? 'html5' : 'xhtml';
     96
     97                /**
     98                 * Filters the HTML format of widgets with navigation links.
     99                 *
     100                 * @since 5.5.0
     101                 *
     102                 * @param string $format The type of markup to use in widgets with navigation links.
     103                 *                       Accepts 'html5', 'xhtml'.
     104                 */
     105                $format = apply_filters( 'navigation_widgets_format', $format );
    92106                ?>
     107
     108                <?php if ( 'html5' === $format ) : ?>
     109                        <nav role="navigation">
     110                <?php endif; ?>
     111
    93112                <ul>
    94113                        <?php foreach ( $r->posts as $recent_post ) : ?>
    95114                                <?php
    … … class WP_Widget_Recent_Posts extends WP_Widget { 
    109128                                </li>
    110129                        <?php endforeach; ?>
    111130                </ul>
     131                <?php if ( 'html5' === $format ) : ?>
     132                        </nav>
     133                <?php endif; ?>
     134
    112135                <?php
    113136                echo $args['after_widget'];
    114137        }

  • src/wp-includes/widgets/class-wp-widget-rss.php 

    diff --git a/src/wp-includes/widgets/class-wp-widget-rss.php b/src/wp-includes/widgets/class-wp-widget-rss.php
index 385bccfd7e..591b88f555 100644
    a b class WP_Widget_RSS extends WP_Widget { 
    9494                if ( $title ) {
    9595                        echo $args['before_title'] . $title . $args['after_title'];
    9696                }
     97
     98                $format = current_theme_supports( 'html5', 'navigation' ) ? 'html5' : 'xhtml';
     99
     100                /**
     101                 * Filters the HTML format of widgets with navigation links.
     102                 *
     103                 * @since 5.5.0
     104                 *
     105                 * @param string $format The type of markup to use in widgets with navigation links.
     106                 *                       Accepts 'html5', 'xhtml'.
     107                 */
     108                $format = apply_filters( 'navigation_widgets_format', $format );
     109
     110                if ( 'html5' === $format ) {
     111                        echo '<nav role="navigation">';
     112                }
     113
    97114                wp_widget_rss_output( $rss, $instance );
     115
     116                if ( 'html5' === $format ) {
     117                        echo '</nav>';
     118                }
     119
    98120                echo $args['after_widget'];
    99121
    100122                if ( ! is_wp_error( $rss ) ) {