[wp-trac] [WordPress Trac] #33654: wp_new_user_notification breaking change.

[WordPress Trac]

#33654: wp_new_user_notification breaking change.
-------------------------------+------------------------------
 Reporter:  alexander.rohmann  |       Owner:  adamsilverstein
     Type:  defect (bug)       |      Status:  assigned
 Priority:  normal             |   Milestone:  4.3.1
Component:  Users              |     Version:  4.3
 Severity:  normal             |  Resolution:
 Keywords:  has-patch          |     Focuses:
-------------------------------+------------------------------
Changes (by ocean90):

 * keywords:  dev-feedback has-patch => has-patch


Comment:

 Replying to [comment:16 adamsilverstein]:
 > In [attachment:33654.4.diff]:
 > * Add an additional test verifying calling function with second argument
 thrown the deprecated error as expected

 `@expectedDeprecated` takes care of the deprecation notice,
 `$this->deprecated_called` isn't needed. Please have also in mind, that
 unit tests have to pass on PHP 5.2 too, so no closures in tests.

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