Skip to:
Content

BuddyPress.org

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#3671 closed defect (bug) (fixed)

General Settings update is not checking for invalid user email at error handling

Reported by: techguytom Owned by:
Milestone: 1.5.1 Priority: normal
Severity: normal Version: 1.5
Component: Settings Keywords: dev-feedback has-patch
Cc:

Description

http://buddypress.trac.wordpress.org/browser/trunk/bp-settings/bp-settings-actions.php#L95

a check is made to see if there are errors in the password fields but the email field is ignored. If the email address is invalid then the messaging still reports changes saved even though they are not.

I've submitted a patch.

Attachments (1)

Added_missing_check_on_email_address_change_for_error_reporting_.patch (617 bytes) - added by techguytom 8 years ago.

Download all attachments as: .zip

Change History (4)

#1 @boonebgorges
8 years ago

  • Component changed from Core to Settings
  • Milestone changed from Awaiting Review to 1.5.1

Confirmed.

#2 @boonebgorges
8 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [5231]) Ensures that an error message is correctly delivered when a changed email address is not valid, in a user's Settings. Fixes #3671. Props techguytom

#3 @boonebgorges
8 years ago

(In [5232]) Ensures that an error message is correctly delivered when a changed email address is not valid, in a user's Settings. Fixes #3671. Props techguytom

Note: See TracTickets for help on using tickets.