Re: [commits] Horde branch master updated. 17438425c7069b16ebe28fc1ecdf38039d76804d

Michael J Rubinsky <[email protected]>
Newsgroups gmane.comp.horde.devel
Message-ID <20150422110753.Horde.pE8R6ZOQo9EscuPE9cO7Ljp@h4.theupstairsroom.com>
Quoting Michael J Rubinsky <[email protected]>:

> Quoting Jan Schneider <[email protected]>:
>
>>
>> commit 22516d5b4b07fcc49d370e699b4a339cb21c4d7a
>> Author: Jan Schneider <[email protected]>
>> 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

-- 
dev mailing list
Frequently Asked Questions: http://wiki.horde.org/FAQ
To unsubscribe, mail: [email protected]
smime.p7s (application/pkcs7-signature, 5.7 KB) - not displayed
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.