Skip to:
Content
Pages
Categories
Search
Top
Bottom

Suggestion: Pass $user_id to `bp_core_general_settings_after_save`

  • @alexmangini

    Participant

    Hi everybody, I am integrating BuddyPress into a plugin of mine which also happens to sync user data to a third party forum software.

    Thanks to BP’s hooks I have been able to:

    1) register the user to the forum software with same credentials from BP register page
    2) Update user email address when changed from BP > Settings > General

    The last thing I need to do is: 3) update user password when changed from BP > Settings > General

    I thought this would be straightforward by using the bp_core_general_settings_after_save hook (/bp-settings/actions/general.php line 237) but this hook doesn’t pass the WP user ID like the similar hook used to change the email address, bp_settings_email_changed (same file, line 237).

    My request is to pass the $user_id variable to the bp_core_general_settings_after_save hook so developer’s can easily refer to the WP user in custom requests. Hopefully simple enough.

    Thank you, and apologies for not making a pull request — I am not so familiar Git.

Viewing 3 replies - 1 through 3 (of 3 total)
  • @johnjamesjacoby

    Keymaster

    This sounds like a totally reasonable suggestion. Thanks for making it.

    cc @imath 💪

    @imath

    Moderator

    Sure!

    Thanks for the ping @johnjamesjacoby. I’ve open a ticket with a patch about it. @alexmangini don’t hesitate to test it. By the way we don’t use Git but SVN for BuddyPress code. If you want to dive into BuddyPress code contribution, I advise you to read this tutorial to get started quickly 😉

    @imath

    Moderator

    Hi!

    This will be available in 11.0, I’ve just committed the patch on the ticket num 8631.

Viewing 3 replies - 1 through 3 (of 3 total)
  • You must be logged in to reply to this topic.
Skip to toolbar