[wp-trac] [WordPress Trac] #61574: [PHP] Remove PHP < 7.2 code
WordPress Trac
noreply at wordpress.org
Fri Jul 5 11:41:51 UTC 2024
#61574: [PHP] Remove PHP < 7.2 code
---------------------------------------------+-----------------------------
Reporter: ayeshrajans | Owner: SergeyBiryukov
Type: enhancement | Status: reviewing
Priority: normal | Milestone: 6.7
Component: General | Version: trunk
Severity: normal | Resolution:
Keywords: has-patch commit has-unit-tests | Focuses:
---------------------------------------------+-----------------------------
Comment (by jrf):
Replying to [comment:3 ayeshrajans]:
> Yay, thank you!
> I went through the compat files, `PHP_VERSION` constants, and
`function_exists` calls to look for snippets to remove. But I think you
are absolutely right, there has to be more things that we could remove.
>
> Happily offering an extra pair of hands if you have any suggestions
where to look for snippets to remove :)
I've rebased what I prepared earlier this week after the commit made by
@SergeyBiryukov (thanks!) and opened [https://github.com/WordPress
/wordpress-develop/pull/6975 PR 6975]. Definitely not claiming
completeness, but this should get us another step closer.
@ayeshrajans Would you like to review that PR ?
I came across these when searching for similar things, including
`ini_get()`, `@requires` (in the tests), `->markTestSkipped()` (tests),
`PHP_VERSION_ID`, `phpversion()`, `function_exists()`, `class_exists()`,
`extension_loaded()` and some more things (searches made in relation to
some changes which will be proposed for the PHPUnit workflow).
I've fixed things of which I felt sufficiently confident, but only skimmed
through the results of those searches, so there may well be more which can
be removed based on those searches alone.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/61574#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list