[wp-trac] [WordPress Trac] #60328: Deprecated parameter default value of unregister_setting() is not correct type

WordPress Trac noreply at wordpress.org
Sat Jan 27 10:45:51 UTC 2024


#60328: Deprecated parameter default value of unregister_setting() is not correct
type
-------------------------------------------------+-------------------------
 Reporter:  crstauf                              |       Owner:  audrasjb
     Type:  defect (bug)                         |      Status:  reviewing
 Priority:  normal                               |   Milestone:  Awaiting
                                                 |  Review
Component:  General                              |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch changes-requested needs-   |     Focuses:  docs
  unit-tests                                     |
-------------------------------------------------+-------------------------
Changes (by audrasjb):

 * keywords:  has-patch => has-patch changes-requested needs-unit-tests


Comment:

 Hi there, thanks for the patch.
 @crstauf @gerardreches the pull request triggers some PHPUnit issues.
 I think the condition `if ( '' !== $deprecated ) ` needs to be updated to
 check for `null`. The related PHPUnit tests should probably be updated,
 too.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/60328#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list