Pass the user ID inside the `bp_core_general_settings_after_save` hook

[imath]

This is a suggestion/request made from this support topic : ​https://buddypress.org/support/topic/suggestion-pass-user_id-to-bp_core_general_settings_after_save/

I see no reason not to do it as we're including this parameter into the bp_core_general_settings_after_save hook.

The attached patch also include some code formatting improvement.

[oztaser]

I was thinking bp_displayed_user() function could be use for this requirement but also I don't see any blocker about adding parameters in to the action hook.

Your patch looks great to me. I just think we can also add since information about new $redirect_to parameter. Is there any rule about it?

[imath]

Hi @oztaser

You're totally right, that's probably why I've improved a bit some other part of the code 😉. As we're passing this param into the other hook, let's say we're improving consistency

[alexmangini]

Thank you, the patch works great and adds some nice cleanup to the function.

[espellcaste]

Curious: Is it a good idea to add a hook (filter) for the $redirect_to URL?

[imath]

I think it is 😉 A good reason for this is the hard work I had to achieve to regenerate URLs using the WP Rewrite API for the BP Rewrites plugin 😅

[imath]

In 13292:

Pass the user ID inside the hook fired after settings are saved

This is a reply to a request made in the BuddyPress support forum. See
[​https://buddypress.org/support/topic/suggestion-pass-user_id-to-bp_core_general_settings_after_save/
this topic] for more information about it.

Props alexmangini, oztaser, espellcaste.

Fixes #8631

--Cette ligne, et les suivantes

ci-dessous, seront ignorées--

M src/bp-settings/actions/general.php