Changeset 42249 for trunk/src/wp-includes/kses.php

Timestamp:

11/28/2017 04:23:11 AM (7 years ago)

Author:

pento

Message:

General: Add inline PHPCS options to leave regex indentation.

We have a handful of super long regexen that are written over multiple lines, as a collection of strings concatenated together. Each string is indented appropriately for the regex, but PHPCS doesn't recognised this, so defaults to removing the extra whitespace.

Disabling the Squiz.Strings.ConcatenationSpacing.PaddingFound rule for these blocks stops the extra whitespace from being removed.

See #41057.

File:

• trunk/src/wp-includes/kses.php (modified) (2 diffs)

trunk/src/wp-includes/kses.php

@@ -1094 +1094 @@
     }
 
+    // phpcs:disable Squiz.Strings.ConcatenationSpacing.PaddingFound -- don't remove regex indentation
     $regex =
     '(?:'
@@ -1114 +1115 @@
     . ')'
     . '\s*';              // Trailing space is optional except as mentioned above.
+    // phpcs:enable
 
     // Although it is possible to reduce this procedure to a single regexp,