Skip to content

Commit 7b24083

Browse files
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
1 parent e2aeb78 commit 7b24083

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/wp-includes/class-wp-text-diff-renderer-inline.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,10 @@ class WP_Text_Diff_Renderer_inline extends Text_Diff_Renderer_inline {
2424
* @param string $newlineEscape
2525
* @return string
2626
*/
27-
public function _splitOnWords( $string, $newlineEscape = "\n" ) { // phpcs:ignore Universal.NamingConventions.NoReservedKeywordParameterNames.stringFound
27+
public function _splitOnWords( $string, $newlineEscape = "\n" ) { // phpcs:ignore Universal.NamingConventions.NoReservedKeywordParameterNames.stringFound,WordPress.NamingConventions.ValidVariableName.VariableNotSnakeCase
2828
$string = str_replace( "\0", '', $string );
2929
$words = preg_split( '/([^\w])/u', $string, -1, PREG_SPLIT_DELIM_CAPTURE );
30-
$words = str_replace( "\n", $newlineEscape, $words );
30+
$words = str_replace( "\n", $newlineEscape, $words ); // phpcs:ignore WordPress.NamingConventions.ValidVariableName.VariableNotSnakeCase
3131
return $words;
3232
}
3333
}

0 commit comments

Comments
 (0)