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

[Sumit Bose]

On Tue, Feb 19, 2013 at 02:01:24PM +0100, Petr Vobornik wrote:
> 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.
I agree with both :-). Since it is not straightforward I guess it would
be better not to add this with this ticket but open a new one. Do you
agree?

bye,
Sumit

> -- 
> Petr Vobornik