Cool that you could make this happen easily
Please feel free to share your result with the community (either as post-attachment, or upload as 3pd, pm nant for this), I'm pretty sure you will make quite a few happy, and probably get feedback and improvement proposals.
Just to understand right your proposal:
You suggesting to change:
[code:1] $_PLUGINS->trigger( 'onBeforeUserUpdate', array(&$row,&$rowExtras));[/code:1]
to:
[code:1] $_PLUGINS->trigger( 'onBeforeUserUpdate', array(&$row,&$rowExtras, &$oldRow, &$oldRowExtras));[/code:1]
($oldRow needing to be created) in comprofiler.php
and:
[code:1] $_PLUGINS->trigger( 'onBeforeUpdateUser', array(&$row,&$rowExtras));[/code:1]
to:
[code:1] $_PLUGINS->trigger( 'onBeforeUpdateUser', array(&$row,&$rowExtras, &$previousState ));[/code:1]
in admin.comprofiler.controller.php ?
The idea that this wasn't done initially, is that the old values could be gotten from database, but the above changes make sense and would be backwards-compatible.
If I didn't understand right, please just code-quote your proposed changes