On 08/01/2011 10:01 AM, Alexander Bokovoy wrote:
On 01.08.2011 17:00, Adam Young wrote:
I don't really have the context to ACK it, but it looks a lot better to
me than the previous solution.
Thanks.

Here is what I get without and with the patch:
----------------------------------
$ ./ipa automountkey-add pune
Map:
ipa: ERROR: 'automountmapautomountmapname' is required
$ patch -p1<  ~/params-env.patch
patching file ipalib/parameters.py
$ ./ipa automountkey-add pune
Map:
ipa: ERROR: 'automountmap' is required
-----------------------------------

On the Web UI side you seem to check required field before submitting
data to the server so you shouldn't get to this problem at all.
We check on add, but not on mod.

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to