Context Navigation

← Previous Change
Next Change →

Changeset 62719 for trunk/phpstan.neon.dist

Timestamp:

07/14/2026 04:35:26 AM (26 hours ago)

Author:

westonruter

Message:

Code Modernization: Add a polyfill for PHP 8.6's clamp().

Introduce a clamp() polyfill in compat.php so that core, plugins, and themes can call the function on every supported PHP version without a version check. It returns the value when it lies within the bounds and the nearest bound otherwise, and throws a ValueError when $min is greater than $max or when either bound is NAN. On PHP 7.x, an InvalidArgumentException is thrown instead, since the ValueError class does not yet exist.

Adopt the function at the two places in core that were clamping via a nested min( max( … ) ): the oEmbed response width in get_oembed_response_data(), and the minimum font size factor in wp_get_typography_font_size_value().

Additionally, fix PHPStan errors in code that touches the newly-introduced clamp() usage. Add types to wp_get_typography_value_and_unit() and refactor the parsing to allow static analysis to identify the shape of $matches and avoid issues with passing special regex characters in units.

Developed in https://github.com/WordPress/wordpress-develop/pull/11669.
Follow-up to r35436, r54260, r55947.

Props soean, mukesh27, westonruter.
See #64898.
Fixes #65143.

File:

: 1 edited

trunk/phpstan.neon.dist (modified) (1 diff)

Legend:

: Unmodified
: Added
: Removed

trunk/phpstan.neon.dist

-              r62618
+              r62719
                         path: src/wp-includes/canonical.php
                         count: 1
+                # Level 2:
+                # ValueError is PHP 8.0+; core throws it conditionally so the docblocks are correct for WP's 7.4+ range,
+                # but bleedingEdge's version-aware check treats the class as non-existent against the PHP 7.4 floor.
+                -
+                        message: '#^PHPDoc tag @throws with type InvalidArgumentException\|ValueError is not subtype of Throwable$#'
+                        path: src/wp-includes/compat.php
+                        reportUnmatched: false

Note: See TracChangeset for help on using the changeset viewer.

Make WordPress Core

Context Navigation

Changeset 62719 for trunk/phpstan.neon.dist

Legend:

trunk/phpstan.neon.dist

Download in other formats: