[Freeipa-devel] [PATCH] 255 Added Web UI support for service PAC type option: NONE

[Petr Vobornik]

On 02/19/2013 01:40 PM, Sumit Bose wrote:
> On Thu, Feb 14, 2013 at 09:56:44AM -0600, Endi Sukma Dewata wrote:
>> On 2/14/2013 6:30 AM, Petr Vobornik wrote:
>>>> If they are mutually exclusive, they probably should be separated using
>>>> radio buttons like this:
>>>>
>>>>    PAC: ( ) None
>>>>         (o) Type:
>>>>             [x] MS-PAC
>>>>             [ ] PAD
>>>
>>> You missed one option: nothing selected. It can be solved by adding   '(
>>> ) Inherited' radio.
>>
>> I wouldn't have guessed that :) I agree we should add the
>> 'Inherited' option.
>>
>
> Would it be possible to print the inherited values here, i.e. the
> default values given in the ipakrbauthzdata attribute of ipaGuiConfig
> object? I think this would improve the user experience because you do
> not have to remember/look up the default values.
>
> bye,
> Sumit
>

It is possible, but not straightforward.

I see two options:
   1) Currently Web UI loads config at start so we can use it. The 
disadvantage is that the value might be changed and so the displayed 
value might be incorrect.
   2) Other option is to run config-show along with service-show in a 
batch to get the up-to-date value.

I'm not a big fan of #2 but it is probably better solution.
-- 
Petr Vobornik