I still don't understand how this results in not getting the right 
values for read-only share access, but I've committed it for now.

With PHP 5 objects aren't copied unless you clone them, so none of 
this junk is necessary...

btw: PHP5 supports references in foreach, but that's out of the 
question right now for Horde. Using PHP5 it can be written like this:



foreach ($shares as $key => &$share) {

$share->get('name')

}

Here's the root of the problem: Let's say you have an array of 
references to Horde_Share objects named $shares. When you now process 
the array via



foreach ($shares as $key => $share) {

$share->get('name')

}



it creates a copy of the share object if you access it like this.



Here's the proper fix:



foreach(array_keys($shares) as $id) {

$shares[$id]->get('name');

}



That fixed the share renaming (among other reference fixes) in mnemo, 
which are going to be submitted soon. It's also the way Turba accesses 
the shares.

The problem still exists for me but I still need to investigate the 
correct source for the error and try to construct an example. I'll 
reopen it once I know more.

But this loop doesn't even use references.

The issue that I had matched what is described here:



http://www.php.net/manual/en/control-structures.foreach.php#66000



and changing the way the iteration works in that template fixed the 
problem for me. But I don't know if that is dependant on the PHP 
version. I am using PHP-5.1.6 on the development machine.



I can do further testing with the PHP version and also compare to the 
datatree driver if that is required.

This doesn't make a lot of sense to me. Even if it's an array of 
references, they're not changed in either loop, and you should be able 
to loop through it more than once. Is this a difference between the 
kolab and datatree drivers, or a PHP version thing?

s/the driver would/the share driver would/

The list of shares that is used within the tasklists.inc template is a 
list of references in my case (kolab share driver).



The use of the two foreach loops within the templates mixes up these 
refernces and once I edit a share I end up with one shares being 
listed twice and other shares missing.



In case the share array should really be an array of references the 
attached patch would fix the problematic foreach loops. In case the 
share list should not provide object references, the driver would 
probably need to be modified accordingly.