[dev] [cvs] commit: turba/lib api.php

[Chuck Hagenbuch]

Quoting Jan Schneider <jan at horde.org>:

>> Maybe its a case of confusion as to the expected behaviour on my
>> part.   From Kronolith_FreeBusy::getUrl() if the user's
>> search_sources pref  is empty (as would be the case if it was never
>> explicitly set) then an  empty array is passed and only one address
>> book - the one that happens  to be first in the cfgSources array -
>> is searched.  I would have  expected that all your available address
>> books would be searched -  just like the behaviour of the
>> addressbooks pref in Turba (IIRC).
>
> I'm not sure if this is really a good idea. This could take a lot of
> time if you have many sources, and produce a lot of load on the
> backend servers. Admins should set the preference default value.

I agree with Jan - this is how the search api works (using all  
addressbooks is done on the IMP side, not the turba side), and I think  
this should be reverted.

-chuck