[Freeipa-devel] [PATCH] allow RDN change in UI

Karl MacMillan kmacmill at redhat.com
Thu Dec 6 22:24:52 UTC 2007


On Thu, 2007-12-06 at 14:49 -0500, Rob Crittenden wrote:
> Make uid an editable field in the Edit UI so we can do RDN changes. I 
> pop up a javascript warning when I detect a change here to let the 
> updater know that the password will need to be reset. It isn't perfect 
> but should hit most of the time.
> 
> Fix group RDN changes and tweaked the UI to display it better. Rather 
> than making cn a required field it is only required if edit protected 
> fields is checked. This way it won't show as pink.
> 
> Remove a copy/paste error in the group UI update that caused 2 updates!? 
> update_group() got called twice which in itself isn't a huge problem but 
> when changing a group RDN it is.
> 
> Fix variable name so groups don't get user objectclasses. Oops.
> 
> Remove color CSS for field backgrounds as they override disabled field 
> display. The background was a baby blue but the disabled fields should 
> dispaly as grey. This must be handled automagically by the browser as I 
> could find no CSS control for detecting a javascript disabled field.
> 

Pushed.

Only comment is that the warning dialog might say password "reset"
rather than "change". Change, to me, implies something that the user
could do with their old password. Reset, on the other hand, is an admin
action. Could just be me though.

Karl




More information about the Freeipa-devel mailing list