[Freeipa-devel] [PATCH] Fix Update function on details page.
Pavel Zuna
pzuna at redhat.com
Tue Aug 17 13:08:42 UTC 2010
This is a fix to the setattr/addattr problem discussed in the thread:
[Freeipa-devel] [PATCH] Change the behaviour of addattr/setattr parameters
The problem was that parameters with no values are automatically
set to None by the framework and it wasn't handled properly in
baseldap.py:get_attributes function. Also, there were two logical
bugs in details.js:
1) atttribute callbacks to update values were called for input elements
instead of dt elements
2) it was always trying to update the primary key
I've seen Adam's fix. It's another way to do it, but I think it's better not to
rely on things like element focus to make assumptions about data. There were
also some logical bugs in the update code as mentioned above, so this patch is a
little bit more complete.
By the way; please use Reply All when replying to freeipa-devel threads.
Pavel
More information about the Freeipa-devel
mailing list