[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