[imp] folder acl want to protect cyrus and owner

[LALOT Dominique]

2011/6/7 Michael M Slusarz <slusarz at horde.org>

> Quoting LALOT Dominique <dom.lalot at gmail.com>:
>
>  2011/6/7 LALOT Dominique <dom.lalot at gmail.com>
>>
>>  Hello,
>>>
>>> my backend is a cyrus imap 2.3.x server
>>> I wanted to protect the owner of a mailbox and cyrus account from being
>>> removed. So I tried a quick patch on imp/lib/Prefs/Ui.php line 606
>>>
>>>                /* Create table of each ACL option for each user granted
>>>                 * permissions; enabled indicates the right has been given
>>> to
>>>                 * the user. */
>>> $login=$GLOBALS['registry']->getAuth();
>>>                if ($index=='cyrus'|| $index==$login) $disable=1;
>>>                else $disable=0;
>>>                $rightsmbox = $acl->getRightsMbox($folder, $index);
>>>                foreach (array_keys($rightslist) as $val) {
>>>                    $entry['rule'][] = array(
>>>                        'disable' => !$canEdit || !$rightsmbox[$val] ||
>>> $disable,
>>>
>>> On the interface, that's perfect, cyrus and owner appears in grey, so I
>>> can't change. The bad news is: if I click on modify, cyrus and owner get
>>> removed. I am not sure I can easily improve my patch. Looks like a bug
>>> anyway as a disabled valued should not be removed.
>>>
>>> Dom
>>>
>>>
>>>
>>>  Trying some other tests showed me something wrong about ACL:
>>
>> You select a subfolder fred, you can see "current access to fred", the acl
>> are from that folder, but the folder icon followed by the name of the
>> folder
>> is coming back to INBOX. So when you click modify it's modifying INBOX.
>>
>
> This was already fixed.
>
> michael
>
>
Hello Michael

But not in the last release, may be in git? What should I do?

Dom

-- 
Dominique LALOT
Ingénieur Systèmes et Réseaux
http://annuaire.univmed.fr/showuser.php?uid=lalot