[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