Make WordPress Core

Opened 20 months ago

Last modified 20 months ago

#56607 new defect (bug)

Improve doc for WP_Screen class

Reported by: chouby's profile Chouby Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 3.3
Component: Administration Keywords: has-patch
Focuses: docs Cc:

Description

WooCommerce 6.9 introduced a conflict with several plugins by accessing to the property WP_Screen::id and expecting it to be a string, while conflicting plugins just set the current screen with a simple call to set_current_screen() without parameter. In this case, get_current_screen() returns a WP_Screen object with the property id set to null.

This pushed me to review the doc of the class and notice that several properties are not correctly documented for this case.

Attachments (1)

56607.patch (1.4 KB) - added by Chouby 20 months ago.

Download all attachments as: .zip

Change History (2)

@Chouby
20 months ago

#1 @Chouby
20 months ago

  • Keywords has-patch added

Just for reference, the issue opened in WooCommerce: https://github.com/woocommerce/woocommerce/issues/34725

Note: See TracTickets for help on using tickets.