[Kolab-devel] distributionlists testing

[Thomas Jarosch]

Hello Mike,

On Monday, 21. April 2008 13:51:20 Mike Gabriel wrote:
> --- b/turba/lib/Driver/kolab.php.orig   2008-04-20 18:49:17.000000000 +0000
> +++ b/turba/lib/Driver/kolab.php        2008-04-20 18:47:49.000000000 +0000
> @@ -898,7 +898,10 @@
>                   }
>                   foreach ($entries as $entry) {
>                       if (isset($entry[$field])) {
> -                        $pos = strpos($entry[$field], $value);
> +                       $pos = false;
> +                       if ($value) {
> +                            $pos = strpos($entry[$field], $value);
> +                       }
>                           if ($pos === false) {
>                               continue;
>                           }

I've fixed the same bug a while ago. Please see upstream bug,
it does a bit more cleanup: http://bugs.horde.org/ticket/6546
Dunno why it's not applied yet.

I'll open another bug since I'm not happy about the stripos() to strpos() 
change (stripos is PHP5 only). People would be quite upset
if google would be case sensitive...

Thomas