[wp-trac] [WordPress Trac] #61890: Handle WP_Term known, named dynamic properties for PHP 8.2
WordPress Trac
noreply at wordpress.org
Fri Aug 23 18:35:00 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: 4.4
Severity: minor | Resolution:
Keywords: php82 has-patch has- | Focuses: coding-standards, php-
unit-tests needs-testing needs- | compatibility
dev-note has-testing-info |
-------------------------------------+-------------------------------------
Comment (by hellofromTonya):
== Test Result for WooCommerce
WooCommerce type casts to an array, instead of using
`WP_Term::to_array()`.
Patch tested: https://github.com/WordPress/wordpress-develop/pull/7231
=== Environment
- OS: macOS
- Web Server:
- For after: Nginx (with wp docker)
- For before: Nginx via Local
- PHP: 8.3.8
- WordPress: 6.6.1 (for before) and `trunk` (for after)
- Browser: Firefox 129.0.1
- Theme: See testing instructions
- Active Plugins:
- WooCommerce
=== Testing Instructions
1. Add a new product.
* Configure its Inventory, Shipping, etc.
* Add Attributes such as Brand, Material, Color.
* Add a Product category.
* Add a Product tag.
2. View the product in the frontend.
* Navigate through its tabs.
* Click on its product category.
* Click on its tag(s).
3. View the Shop page. Navigate to the product.
4. View the server logs or `debug.log`, noting any errors (especially for
terms).
=== Actual Results
- ✅ No differences between WP 6.6.1 without the patch (before) and
`trunk` with the patch (after).
- ✅ No errors in server logs or `debug.log`.
- ✅ Each of the expectations worked as expected.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/61890#comment:16>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list