[dev] [commits] Horde branch master updated. a61da56f8c575c7da9c2c018a350119e4301a472

[Michael J Rubinsky]

Quoting Jan Schneider <jan at horde.org>:

> Zitat von Michael J Rubinsky <mrubinsk at horde.org>:
>
>> Quoting Jan Schneider <jan at horde.org>:
>>
>>>> commit a61da56f8c575c7da9c2c018a350119e4301a472
>>>> Author: Michael J Rubinsky <mrubinsk at horde.org>
>>>> Date:   Fri Apr 24 10:32:55 2015 -0400
>>>>
>>>>  Ensure admins see all system shares.
>>>>
>>>>  This is why my system shares were not included in listShares.
>>>>  If they are created, but don't explicitly have permissions set for
>>>>  at least PERMS_VIEW, they will not be returned.
>>>>
>>>> kronolith/lib/Shares.php |   10 +++++++++-
>>>> 1 files changed, 9 insertions(+), 1 deletions(-)
>>>>
>>>> http://github.com/horde/horde/commit/a61da56f8c575c7da9c2c018a350119e4301a472
>>>
>>> This is intended. System share should for admins only show up in  
>>> the system shares section, not in the regular section. For  
>>> non-admin users, the system shares should show up in the shared  
>>> calendars section, if they have sufficient perms.
>>
>> This didn't have anything to do with what section they show up in.  
>> If you create a system share, but do not explicitly set any  
>> permissions on it, it will not be viewable _at_all_, to anyone,  
>> ever. This commit fixes them not showing up in the system shares  
>> section for admins only.
>
> But since listSystemShares() doesn't do permission checking, they  
> should still show up in the system calendars section, no matter if  
> permissions have been set or not.

That's correct, but you removed the use of listSystemShares for  
listing the shares in the sidebar in a previous commit stating that  
listInternalShares() already returns systemShares - which it does, but  
only if permissions have been explicitly set.

See  
https://github.com/horde/horde/commit/22516d5b4b07fcc49d370e699b4a339cb21c4d7a

-- 
mike
The Horde Project
http://www.horde.org
https://www.facebook.com/hordeproject
https://www.twitter.com/hordeproject
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 5869 bytes
Desc: S/MIME Signature
URL: <http://lists.horde.org/archives/dev/attachments/20150427/a3ac249f/attachment.bin>