Make WordPress Core

Changeset 57633


Ignore:
Timestamp:
02/14/2024 07:25:56 PM (5 months ago)
Author:
SergeyBiryukov
Message:

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.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/class-wp-text-diff-renderer-inline.php

    r56547 r57633  
    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    }
Note: See TracChangeset for help on using the changeset viewer.