[dev] Problem with compose_html logic (HEAD)

[Jan Schneider]

Zitat von Jan Kuipers <jrkuipers at lauwerscollege.nl>:

> Citeren Jan Schneider <jan at horde.org>:
>
> > > compose_html
> > > locked  value
> > >   1       1      html, locked, no pref shown, switchlink in
> composescreen
> >
> > This doesn't make sense. If the pref is locked, it is locked, so there
> > shouldn't be the switchlink for this case. Of course it doesn't make
> much
> > sense to lock the pref to allow html composition only, but as the admin
> > decides, so be it.
>
> I agree, but the head code responds this way. It seems if you can use
> html
> compose , the switchlink is always present. For me, this is not a
> problem. But
> with the present code the switchlink is also present with locked=1 and
> value=0,
> which I use here and this shouldn't be like this.

Correct. Can you re-submit your patch with the switch link *always* disabled
if the pref is locked?

Jan.

--
http://www.horde.org - The Horde Project
http://www.ammma.de - discover your knowledge
http://www.tip4all.de - Deine private Tippgemeinschaft