[wp-trac] [WordPress Trac] #61890: Handle WP_Term known, named dynamic properties for PHP 8.2
WordPress Trac
noreply at wordpress.org
Wed Aug 21 21:04:31 UTC 2024
#61890: Handle WP_Term known, named dynamic properties for PHP 8.2
-------------------------------------+-------------------------------------
Reporter: hellofromTonya | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: 6.7
Component: Taxonomy | Version:
Severity: minor | Resolution:
Keywords: php82 has-patch has- | Focuses: coding-standards, php-
unit-tests | compatibility
-------------------------------------+-------------------------------------
Comment (by hellofromTonya):
In [changeset:"58920" 58920]:
{{{
#!CommitTicketReference repository="" revision="58920"
Tests: Remove 'errors' assertion when not WP_Error in
Tests_Term_WpInsertTerm.
Removes the assertion for 'errors' being empty when the instance is
`WP_Term` and not `WP_Error`. This property exists on `WP_Error`.
This assertion always passed because it was checking a dynamic property on
`WP_Term` that does not exist and is not added within Core. Thus, this
assertion is not needed and fails with dynamic property deprecations.
Follow-up to [51403], [34646], [29830].
See #61890, #61530.
}}}
--
Ticket URL: <https://core.trac.wordpress.org/ticket/61890#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list