I assume this was the bug you wanted me to close again, right? From 
the error description it sounds like this refers to the getId() 
problems I fixed in CVS recently. Please reopen if the problem persists.

Looks like the Kolab share handling in Turba, at least for me using 
PHP 5.2.5, is currently not working at all as Turba expects the key of 
the shares (=>$addressbooks) array to be the value of getName() 
instead of getId().



I wrote a small proof-of-concept patch to fix the issue, works like a 
charm now.

Would it make sense to use getId() for refering to the addressbook 
source? If yes, are the changes in the patchset to the History needed 
as well (I'm not familiar with Horde's History framework)?



Cheers,

Thomas

Hello together,



I did a fresh CVS checkout to test the current state

of the Kolab driver again. I had trouble getting the

list of addressbooks in Turba. Here's why:



Turba gets a list of addressbooks via



"$addressbooks = Turba::listShares(true);".



After that the names get sorted:

foreach ($addressbooks as $addressbook) {

     $sorted_addressbooks[$addressbook->getName()] = $addressbook->get('name');

}

asort($sorted_addressbooks);



Using the Kolab share driver the data was lay out like this:



$addressbook["INBOX%2FKontakte"]:

$addressbook->getName() = "INBOX/Kontakte";

$addressbook->get('name') = "Kontakte";



Because of the HTML escaping in the key of the array, Turba was unable

to find the addressbooks again when it tried to display the list.

Attached patch uses the real key of $addressbooks.



Cheers,

Thomas