[dev] Problem with compose_html logic (HEAD)

[Jan Schneider]

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

> Regarding HEAD:
>
> There is a small problem with the compose_html logic in imp/compose.php
> imho it should work this way:
>
> 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.

>   1       0      plain, locked, no pref shown, _no_ switchlink in
> composescreen
>   0       1      html, not locked, pref shown, switchlink in
> composescreen
>   0       0      plain, not locked, pref shown, switchlink in
> composescreen
>
> With the existing logic it doesn't work this way.
> I attached a small patch which accomplishes this behaviour.

Jan.

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