Changeset 56752

Timestamp:

10/01/2023 12:20:53 AM (3 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Remove redundant ignore annotations, take 4.

This removes ignore annotations related to sniffs which are not used by WP Core (like sniffs in the WordPress-Extra ruleset).

Follow-up to [48072], [51003], [55204], [56714].

Props jrf.
See #59161.

Location:

trunk/src

Files:

• wp-admin/includes/file.php (modified) (1 diff)
• wp-admin/includes/post.php (modified) (1 diff)
• wp-includes/sitemaps/class-wp-sitemaps-renderer.php (modified) (2 diffs)
• wp-includes/sitemaps/class-wp-sitemaps-stylesheet.php (modified) (1 diff)
• wp-includes/template-canvas.php (modified) (1 diff)

trunk/src/wp-admin/includes/file.php

@@ -2765 +2765 @@
                                 '<code>wp_opcache_invalidate_directory()</code>'
                         );
-                        // phpcs:ignore WordPress.PHP.DevelopmentFunctions.error_log_trigger_error
                         trigger_error( $error_message );
                 }

trunk/src/wp-admin/includes/post.php

@@ -2012 +2012 @@
          * Ignoring sanitization to avoid altering meta. Ignoring the nonce check because
          * this is hooked on inner core hooks where a valid nonce was already checked.
-         *
-         * @phpcs:disable WordPress.Security
          */
         $posted_data = isset( $_POST['data']['wp_autosave'] ) ? $_POST['data']['wp_autosave'] : $_POST;
-        // phpcs:enable
 
         $post_type = get_post_type( $new_autosave['post_parent'] );

trunk/src/wp-includes/sitemaps/class-wp-sitemaps-renderer.php

@@ -132 +132 @@
                 if ( ! empty( $index_xml ) ) {
                         // All output is escaped within get_sitemap_index_xml().
-                        // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
                         echo $index_xml;
                 }
@@ -197 +196 @@
                 if ( ! empty( $sitemap_xml ) ) {
                         // All output is escaped within get_sitemap_xml().
-                        // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
                         echo $sitemap_xml;
                 }

trunk/src/wp-includes/sitemaps/class-wp-sitemaps-stylesheet.php

@@ -26 +26 @@
 
                 if ( 'sitemap' === $type ) {
-                        // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped -- All content escaped below.
+                        // All content is escaped below.
                         echo $this->get_sitemap_stylesheet();
                 }
 
                 if ( 'index' === $type ) {
-                        // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped -- All content escaped below.
+                        // All content is escaped below.
                         echo $this->get_sitemap_index_stylesheet();
                 }

trunk/src/wp-includes/template-canvas.php

@@ -21 +21 @@
 <?php wp_body_open(); ?>
 
-<?php echo $template_html; // phpcs:ignore WordPress.Security.EscapeOutput ?>
+<?php echo $template_html; ?>
 
 <?php wp_footer(); ?>