Minor issue on PHP 8.1

  • Resolved Eusebiu Oprinoiu

    (@eusebiuoprinoiu)


    3 years, 6 months ago

    Thank you for adding support for PHP 8.
    I did some tests, and I found a small issue on PHP 8.1.

    Deprecated: trim(): Passing null to parameter #1 ($string) of type string is deprecated in /wp-content/plugins/backwpup/inc/class-option.php on line 328

Viewing 3 replies - 1 through 3 (of 3 total)
  • happyAnt

    (@duongcuong96)

    3 years, 5 months ago

    Hello @eusebiuoprinoiu

    thank you for the report, we already reported this issue to devs. Will back to you when i have something new!

    Thread Starter Eusebiu Oprinoiu

    (@eusebiuoprinoiu)

    3 years, 5 months ago

    Thank you very much!
    I appreciate it…

    Marcel Pol

    (@mpol)

    3 years, 5 months ago

    Changing line 328 to this, using (string) to cast to a string, should be enough. The function trim() will cast it to an empty string anyway.
    You could also change the default parameter of the function to have an empty string, but that changes the signature of the function.

    $key = sanitize_key(trim( (string) $key ));

Viewing 3 replies - 1 through 3 (of 3 total)

The topic ‘Minor issue on PHP 8.1’ is closed to new replies.