[dev] [commits] Horde branch master updated. 17438425c7069b16ebe28fc1ecdf38039d76804d

Michael J Rubinsky mrubinsk at horde.org
Wed Apr 22 15:07:53 UTC 2015


Quoting Michael J Rubinsky <mrubinsk at horde.org>:

> Quoting Jan Schneider <jan at horde.org>:
>
>>
>> commit 22516d5b4b07fcc49d370e699b4a339cb21c4d7a
>> Author: Jan Schneider <jan at horde.org>
>> Date:   Wed Apr 22 12:26:34 2015 +0200
>>
>>    Revert "Must include system calendars in _allCalendars."
>>
>>    This reverts commit 44877fa8e8494a42d48d18384ca6be7f51c6cf1d.
>>
>>    listSystemCalendars() does *not* check permissions. Also, system  
>> calendars are already returned by listInternalCalendars().
>
> The system shares were NOT returned in listInternalCalendars() for  
> me, though perhaps your other commit about fixing permissions on the  
> sqlng driver fixes that issue - I'll check when I'm back in front of  
> my dev system.


I made other comments via GitHub, but thinking about some of the  
issues you were "fixing", it leads me to believe that maybe some  
caching on your system is causing  
$injector->getInstance('Kronolith_Shares') to continue to return a  
Horde_Share_Base object instead of the new Kronolith_Shares object  
that was introduced along with the other recent Resource changes...

-- 
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/20150422/7685416c/attachment.bin>


More information about the dev mailing list