[Freeipa-devel] environment in Param(s)

[Alexander Bokovoy]

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