Skip to content

Commit

Permalink
Fix "It is recommended not to use reserved keyword "string" as functi…
Browse files Browse the repository at this point in the history
…on parameter name. Found: $string".
  • Loading branch information
remcotolsma committed May 15, 2024
1 parent 52a8a30 commit a903388
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions tests/src/ParserTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -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 );
}

/**
Expand Down

0 comments on commit a903388

Please sign in to comment.