On 01.08.2011 17:15, Adam Young wrote: > 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. Should we check on modification as well?
-- / Alexander Bokovoy _______________________________________________ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel