Changeset 43571 for trunk/src/wp-includes/theme.php

Timestamp:

08/17/2018 01:50:26 AM (6 years ago)

Author:

pento

Message:

Coding Standards: Upgrade WPCS to 1.0.0

WPCS 1.0.0 includes a bunch of new auto-fixers, which drops the number of coding standards issues across WordPress significantly. Prior to running the auto-fixers, there were 15,312 issues detected. With this commit, we now drop to 4,769 issues.

This change includes three notable additions:

• Multiline function calls must now put each parameter on a new line.
• Auto-formatting files is now part of the grunt precommit script.
• Auto-fixable coding standards issues will now cause Travis failures.

Fixes #44600.

File:

• trunk/src/wp-includes/theme.php (modified) (7 diffs)

trunk/src/wp-includes/theme.php

@@ -726 +726 @@
     if ( is_array( $_sidebars_widgets ) ) {
         set_theme_mod(
-            'sidebars_widgets', array(
+            'sidebars_widgets',
+            array(
                 'time' => time(),
                 'data' => $_sidebars_widgets,
@@ -1678 +1679 @@
         $style .= $image . $position . $size . $repeat . $attachment;
     }
-?>
+    ?>
 <style type="text/css" id="custom-background-css">
 body.custom-background { <?php echo trim( $style ); ?> }
 </style>
-<?php
+    <?php
 }
 
@@ -1693 +1694 @@
     $styles = wp_get_custom_css();
     if ( $styles || is_customize_preview() ) :
-    ?>
+        ?>
         <style type="text/css" id="wp-custom-css">
             <?php echo strip_tags( $styles ); // Note that esc_html() cannot be used because `div &gt; span` is not interpreted properly. ?>
         </style>
-    <?php
+        <?php
     endif;
 }
@@ -1803 +1804 @@
 function wp_update_custom_css_post( $css, $args = array() ) {
     $args = wp_parse_args(
-        $args, array(
+        $args,
+        array(
             'preprocessed' => '',
             'stylesheet'   => get_stylesheet(),
@@ -2022 +2024 @@
                     'title'  => _x( 'Find Us', 'Theme starter content' ),
                     'text'   => join(
-                        '', array(
+                        '',
+                        array(
                             '<strong>' . _x( 'Address', 'Theme starter content' ) . "</strong>\n",
                             _x( '123 Main Street', 'Theme starter content' ) . "\n" . _x( 'New York, NY 10001', 'Theme starter content' ) . "\n\n",
@@ -2540 +2543 @@
                 /* translators: 1: title-tag, 2: wp_loaded */
                 _doing_it_wrong(
-                    "add_theme_support( 'title-tag' )", sprintf(
+                    "add_theme_support( 'title-tag' )",
+                    sprintf(
                         __( 'Theme support for %1$s should be registered before the %2$s hook.' ),
-                        '<code>title-tag</code>', '<code>wp_loaded</code>'
-                    ), '4.1.0'
+                        '<code>title-tag</code>',
+                        '<code>wp_loaded</code>'
+                    ),
+                    '4.1.0'
                 );
 
@@ -2614 +2620 @@
             }
         </style>
-    <?php
+        <?php
     }
 }