[Freeipa-devel] [PATCH] 137 Entitlement registration.
Adam Young
ayoung at redhat.com
Wed Apr 13 15:35:54 UTC 2011
On 04/13/2011 11:30 AM, Endi Sukma Dewata wrote:
> On 4/13/2011 10:13 AM, Adam Young wrote:
>>> There's no pattern defined in the metadata for the quantity. There is
>>> an 'int' type though. I think we can add a general purpose type
>>> checking in addition to pattern validation. This can be done in a
>>> separate patch.
>>
>> We can add a validation rule [1..9][0..9]* to the entitlement plugin,
>> since it should be a positive integer.
>
> It's kind of redundant since this pattern is implied by the int type
> and there are already min/max value params that limit the range to
> positive integers. I think it's better to validate the type and the
> min/max values in the widget because other parameters can benefit from
> it too.
Agreed. Still, the Jascript regex for int type can use that pattern.
More information about the Freeipa-devel
mailing list