Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#3671 closed defect (bug) (fixed)

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

Reported by: techguytom's profile 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 12 years ago.

Download all attachments as: .zip

Change History (4)

@techguytom
12 years ago

#1 @boonebgorges
12 years ago

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

Confirmed.

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