Skip to:
Content

BuddyPress.org

Opened 10 years ago

Closed 10 years ago

Last modified 10 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 10 years ago.

Download all attachments as: .zip

Change History (4)

#1 @boonebgorges
10 years ago

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

Confirmed.

#2 @boonebgorges
10 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
10 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.