Profile Updated Handler

This forum is for questions about contributing to the project or proposing features that you would like to develop for the project. 

This thread is closed to new posts. You must sign in to post in the forums.
7/8/2012 11:58:53 AM
Gravatar
Total Posts 18

Profile Updated Handler

I have implemented one of these for the project I am currently working on, are there any plans to implement such a thing in the base code ?

Also, as well as hooking the onupdated event in the userprofile page, are there any other places I should hook it into to catch profile updates ?

7/8/2012 12:41:30 PM
Gravatar
Total Posts 18444

Re: Profile Updated Handler

Hi Huw,

Its best to avoid forking the code so you will be able to upgrade to new releases.

If you'd like to send me a signed contributor agreement and the files for the profile update handler provider I'll review it and consider integrating it. You can download the pdf of the contributor agreement on the contributor page and send the files to me at joe dot audette at g mail dotcom

Depending on what your code looks like I may implement it myself to keep it consistent with our other handler providers since it is a fairly easy thing to implement. In addition to the /Secure/UserProfile.aspx page it would probably also need to be wired into /Admin/ManageUsers.aspx since profiles can be updated from there, though we may want an event argument indicating the update was done by the admin in case that changes any decisions for the custom handler.

I'm also looking at adding a provider plugin system for user pre-delete handler so that its possible for custom code to do any cleanup of custom data if a user is deleted.

Best,

Joe

7/8/2012 1:46:07 PM
Gravatar
Total Posts 18

Re: Profile Updated Handler

Hi Joe,

Thanks for your response, I will send you a contributor agreement and files, basically mirrored how you did the userregistered handler, and then plugged the call into userprofile, so should be simple to add an extra argument to decide if it was an admin update etc. I did also think it would need to go in manage users also, but not looked at that file as yet.

I will sort out the files today/tomorrow for you, would certainly be handy not to have to keep the changes to core in sync, even though it is just a few lines.

7/10/2012 1:51:34 PM
Gravatar
Total Posts 18444

Re: Profile Updated Handler

Hi Huw,

I've integrated your changes but it will be a bit longer before I merge the changes to the default branch because I have some other unfinished work in my developer branch. Only change I made was the folder name for the config files I changed to userprofileupdatedhandlers just to make it sort close to the other user* stuff.

Thanks!

Joe

7/10/2012 3:18:50 PM
Gravatar
Total Posts 18

Re: Profile Updated Handler

excellent thanks Joe, I'll rename my folder too so it doesn't cause any problems when I update the code.

7/13/2012 8:20:19 AM
Gravatar
Total Posts 18

Re: Profile Updated Handler

Just pulled down the latest code, Update handler works great many thanks for allowing my small contribution.

You must sign in to post in the forums. This thread is closed to new posts.