Skip to:
Content

BuddyPress.org

Opened 15 years ago

Closed 15 years ago

Last modified 15 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:
Priority: normal Milestone: 1.5.1
Component: Settings Version: 1.5
Severity: normal 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 15 years ago.

Download all attachments as: .zip

Change History (4)

@techguytom
15 years ago

#1 @boonebgorges
15 years ago

  • Component Core → Settings
  • Milestone Awaiting Review → 1.5.1

Confirmed.

#2 @boonebgorges
15 years ago

  • Resolution → fixed
  • Status new → 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
15 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.