[Freeipa-devel] [PATCH] 908 make some fields required

[Rob Crittenden]

Endi Sukma Dewata wrote:
> On 11/28/2011 12:09 PM, Rob Crittenden wrote:
>> Some attributes in the framework were not marked as required even though
>> they are in the schema. These are typically computed values and I think
>> the intention was to not prompt for them. This has the side-effect of
>> them showing as not required in the UI even though they are.
>>
>> Since they all have default values set they won't be prompted for on the
>> CLI so there won't be any practical changes.
>
> This patch fixes the problem with required attributes in DNS Zones and
> cn, uidNumber, and gidNumber in Users. The UI now shows the required
> indicators for these attributes. So this patch is ACKed.
>
> Some problems mentioned in ticket #2015 are still present:
>
> 1. Removing the homeDirectory from a user fails because it's required by
> posixAccount.

Ok, I can probably fix this one.

> 2. Removing the gidNumber from a group fails because it's required by
> posixGroup.

Well, its hard to make gidNumber required since you can have non-posix 
groups. I think we'll have to just live with this one.

>
> 3. Removing config attributes listed in the ticket generates internal
> error. I think at least the server should return a proper error message.
> The required indicator can be hard-coded in the UI if necessary.
>

The config problems are a separate issue. To make them required would 
mean that the user would need to provide a value with a -mod request.

rob