Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#3114 closed enhancement (no action required)

Action in bp_core_screen_general_settings for validating email

Reported by: dimitryz's profile dimitryz Owned by:
Milestone: 1.5 Priority: normal
Severity: normal Version: 1.2.8
Component: Settings Keywords:
Cc:

Description

The current buddypress version (1.2.8) does not include email validation for the settings profile page. A user can enter any value and it will be saved.

Attached is a patch that fixes this problem. It enables the declaration of an action function to validate the POSTed email address. If the email address is not valid, a function may modify $email_error and set it to 'true'. Example:

function test_settings_email() {
    global $email_error;
    if ( !is_email($_POST['email']) ) {
        $email_error = true;
    }
}
add_action('bp_core_screen_general_settings_before_save', 'test_settings_email', 1);

Attachments (1)

bp-core-settings.patch (5.3 KB) - added by dimitryz 12 years ago.
Patch adding a new action and email validation to bp_core_screen_general_settings

Download all attachments as: .zip

Change History (4)

@dimitryz
12 years ago

Patch adding a new action and email validation to bp_core_screen_general_settings

#1 @cnorris23
12 years ago

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

This has been fixed in trunk [3524]. Highly unlikely it will be fixed in the 1.2 branch.

#2 @johnjamesjacoby
11 years ago

  • Component changed from Core to Settings
  • Keywords email settings validation removed
  • Milestone changed from Awaiting Review to 1.5
  • Severity set to normal

Moving closed ticket out of Awaiting Review.

#3 @r-a-y
11 years ago

  • Version set to 1.2.8

Just ran across this on a BP 1.2.x install. If there is going to be a BP 1.2.11 release, this should be added.

Note: See TracTickets for help on using tickets.