[wp-trac] [WordPress Trac] #51423: Fix function argument type issues reported by PHPStan
WordPress Trac
noreply at wordpress.org
Thu Nov 12 12:47:58 UTC 2020
#51423: Fix function argument type issues reported by PHPStan
-------------------------------------------+-------------------------------
Reporter: SergeyBiryukov | Owner: SergeyBiryukov
Type: task (blessed) | Status: accepted
Priority: normal | Milestone: 5.6
Component: General | Version:
Severity: normal | Resolution:
Keywords: php8 has-patch has-unit-tests | Focuses: coding-standards
-------------------------------------------+-------------------------------
Comment (by hellofromTonya):
Replying to [comment:40 jrf]:
>At the request of @hellofromTonya I've reviewed
[https://github.com/WordPress/wordpress-develop/pull/736 PR #736] now and
left an extensive comment for you to consider:
https://github.com/WordPress/wordpress-
develop/pull/736#pullrequestreview-528734649
>
>The feedback I left there might also help inform how to handle some of
the other changes needed.
>
>Hope it helps!
Your thorough and extensive comment gave me an Aha Moment! I can now
clearly see both the goal of this ticket, the process, and next steps for
all of its patches/PRs.
The **goal of this ticket** is to match the data types of each function
parameter, i.e. as part of the PHP 8 compatibility.
We should seek:
- to retain the current behavior (i.e. to avoid breaking changes)
- to retain the triggering of notice/warning/error, which are clues and
indicators (i.e. rather than silently letting the problem propagate
downstream)
- to gain PHP 8 compatibility
--
Ticket URL: <https://core.trac.wordpress.org/ticket/51423#comment:42>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list