event hooks for account settings

What about:

plugin_hook('user', 'password:update', $u)

On the user details that I'm porting from units to mod (I didn't commit it yet) I added:

plugin_hook('profile','republish',$u)

And it would notify any change on the user details (we lack of this even hook too)

The trouble is that an identifier will need to get added, since account settings get stored in separate fields and even tables, unlike the profile account settings. My first suggestion did break a convention, but what about something like:

plugin_hook('account:password','republish',$u)

i like the profile republish, but for password -i think- the plugin hook must be before password get updated successful, right?

In both cases the event would be triggered after the event happened. I suggest to add both hooks:

plugin_hook('account:password','republish',$u)

plugin_hook('profile','republish',$u)

What do you think?

Replying to rho:

i like the profile republish, but for password -i think- the plugin hook must be before password get updated successful, right?

Yes, the hook must be before the password gets updated and md5'd

Replying to dramirez:

In both cases the event would be triggered after the event happened. I suggest to add both hooks: {{{ plugin_hook('account:password','republish',$u) }}} {{{ plugin_hook('profile','republish',$u) }}} What do you think?

Profile looks good to me, I'm still unsure about the account:password construction. Ideally we would have a standard object for this, e.g.

$u->name  = 'password';
$u->value = 'unencrypted_password';

So you could have plugin_hook('account','republish',$u). But then again, password is stored in the user table, whereas e.g. setting email nofications is stored in user_flags. Or should we pass it as a user_flag object and let the event handler deal with it?:

$u->flag  = 'password';
$u->value = 'unencrypted_password';

Add hooks to user detail editing

Adds hooks to email, name and password change

Add hooks to user detail editing

Uses simply userdetails now, plus fixed typo.

Hooks for change in name, email or password implemented in r1329