[Freeipa-devel] environment in Param(s)

[Adam Young]

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.