From a90338889b20992b35898675456e033629e6236d Mon Sep 17 00:00:00 2001 From: Remco Tolsma <869674+remcotolsma@users.noreply.github.com> Date: Wed, 15 May 2024 15:29:40 +0200 Subject: [PATCH] Fix "It is recommended not to use reserved keyword "string" as function parameter name. Found: $string". --- tests/src/ParserTest.php | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/src/ParserTest.php b/tests/src/ParserTest.php index 7c33226..d8f5daf 100644 --- a/tests/src/ParserTest.php +++ b/tests/src/ParserTest.php @@ -47,20 +47,20 @@ public function set_up() { * * @param string $thousands_sep Thousands seperator. * @param string $decimal_sep Decimal seperator. - * @param string $string String value to convert. + * @param string $value String value to convert. * @param float $expected Expected float value. */ - public function test_string_to_amount( $thousands_sep, $decimal_sep, $string, $expected ) { + public function test_string_to_amount( $thousands_sep, $decimal_sep, $value, $expected ) { global $wp_locale; $wp_locale->number_format['thousands_sep'] = $thousands_sep; $wp_locale->number_format['decimal_point'] = $decimal_sep; - $number = $this->parser->parse( $string ); + $number = $this->parser->parse( $value ); - $value = $number->get_value(); + $result = $number->get_value(); - $this->assertSame( $expected, $value ); + $this->assertSame( $expected, $result ); } /**