Add rest_sanitize_boolean() to unslashing sanitizing functions

[westonruter]

The rest_sanitize_boolean() function always returns a bool so it is essentially the same as boolval().

[GaryJones]

The test failure looks unrelated to the changes in this PR.

@jrfnl We pull in PHPCSUtils develop for the tests - maybe PHPCSStandards/PHPCSUtils#677 or similar upstream is related for the PHP 8.4 test failure on the public $notAllowed property?

[GaryJones]

https://developer.wordpress.org/reference/functions/rest_sanitize_boolean/
Change makes sense to me.

[jrfnl]

The test failure looks unrelated to the changes in this PR.

Correct.

@jrfnl We pull in PHPCSUtils develop for the tests - maybe PHPCSStandards/PHPCSUtils#677 or similar upstream is related for the PHP 8.4 test failure on the public $notAllowed property?

This is actually related to a change in PHP 8.4, which will be supported in PHPCS 4.x and which PHPCSUtils 1.1.0 will support for both PHPCS 3.x as well as 4.x: properties in interfaces are now allowed.
See: PHPCSStandards/PHPCSUtils@8471a1c

[jrfnl]

This is actually related to a change in PHP 8.4, which will be supported in PHPCS 4.x and which PHPCSUtils 1.1.0 will support for both PHPCS 3.x as well as 4.x: properties in interfaces are now allowed. See: PHPCSStandards/PHPCSUtils@8471a1c

FYI: I have a fix ready for this, but that fix needs PHPCSUtils 1.1.0 (for supporting PHPCS 3.x), so I'll pull it once that's been released (expected this week).

[jrfnl]

Thanks @westonruter ! LGTM.

Once PR #2532 has been merged, this can be rebased and merged.

[jrfnl]

@westonruter When you have time, would you please rebase this PR (without changes), so we can have a passing build before merging ?

[westonruter]

@jrfnl rebased!

[jrfnl]

Thanks @westonruter ! Merging now.