WordPress.org

Make WordPress Core

Opened 6 months ago

Last modified 7 weeks ago

#52464 reviewing defect (bug)

The value of argument passed to the update_option_new_admin_email() is not a valid email.

Reported by: tremidkhar Owned by: SergeyBiryukov
Milestone: 5.9 Priority: normal
Severity: normal Version:
Component: Networks and Sites Keywords: has-patch
Focuses: multisite Cc:

Description

While creating a patch for ticket #43706, I found out that the function update_option_new_admin_email() is responsible for the template. It does accept two parameters, however, the first parameter $old_value does not seem to get the correct value.

When checking the value, it holds the value new_admin_email instead of the old email address.

Thanks!

Change History (7)

#1 @sabernhardt
6 months ago

  • Component changed from General to Networks and Sites
  • Focuses multisite added

This ticket was mentioned in PR #1014 on WordPress/wordpress-develop by tremidkhar.


5 months ago

  • Keywords has-patch added

When changing the admin email address, the argument passed to update_option_new_admin_email() function is a string instead of an email address.

Trac ticket: https://core.trac.wordpress.org/ticket/52464

This ticket was mentioned in Slack in #core by sergey. View the logs.


5 months ago

#4 @SergeyBiryukov
5 months ago

  • Milestone changed from Awaiting Review to 5.8
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

This ticket was mentioned in Slack in #core by chaion07. View the logs.


8 weeks ago

#6 @chaion07
8 weeks ago

Thanks to @tremidkhar for reporting this. We reviewed this ticket during a recent [bug-scrub session]https://wordpress.slack.com/archives/C02RQBWTW/p1623098372373600. With Beta 1 coming in a day, checking with @SergeyBiryukov to kindly draw some attention to this. Thank you!

#7 @hellofromTonya
7 weeks ago

  • Milestone changed from 5.8 to 5.9

Today is 5.8 Beta 1. With no activity, punting to 5.9.

Note: See TracTickets for help on using tickets.