From 7b24083859c37d8ac75b9549f458bc389610eaef Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Wed, 14 Feb 2024 19:25:56 +0000 Subject: [PATCH] Coding Standards: Allow `$newlineEscape` parameter in `WP_Text_Diff_Renderer_inline::_splitOnWords()`. This resolves a WPCS warning: {{{ Variable "$newlineEscape" is not in valid snake_case format, try "$newline_escape" }}} The `WP_Text_Diff_Renderer_inline` class extends the `Text_Diff_Renderer_inline` class from the `Text_Diff` package and should have the same parameters as the parent class method, per the Task 1 section of ticket #51553. Follow-up to [7747], [55163]. See #59650. git-svn-id: https://develop.svn.wordpress.org/trunk@57633 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/class-wp-text-diff-renderer-inline.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/wp-includes/class-wp-text-diff-renderer-inline.php b/src/wp-includes/class-wp-text-diff-renderer-inline.php index f3e4f6b7b0..aef1738f1f 100644 --- a/src/wp-includes/class-wp-text-diff-renderer-inline.php +++ b/src/wp-includes/class-wp-text-diff-renderer-inline.php @@ -24,10 +24,10 @@ class WP_Text_Diff_Renderer_inline extends Text_Diff_Renderer_inline { * @param string $newlineEscape * @return string */ - public function _splitOnWords( $string, $newlineEscape = "\n" ) { // phpcs:ignore Universal.NamingConventions.NoReservedKeywordParameterNames.stringFound + public function _splitOnWords( $string, $newlineEscape = "\n" ) { // phpcs:ignore Universal.NamingConventions.NoReservedKeywordParameterNames.stringFound,WordPress.NamingConventions.ValidVariableName.VariableNotSnakeCase $string = str_replace( "\0", '', $string ); $words = preg_split( '/([^\w])/u', $string, -1, PREG_SPLIT_DELIM_CAPTURE ); - $words = str_replace( "\n", $newlineEscape, $words ); + $words = str_replace( "\n", $newlineEscape, $words ); // phpcs:ignore WordPress.NamingConventions.ValidVariableName.VariableNotSnakeCase return $words; } }