Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

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

Download all attachments as: .zip

Change History (4)

@techguytom
13 years ago

#1 @boonebgorges
13 years ago

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

Confirmed.

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