[Kolab-devel] distributionlists testing

Thomas Jarosch thomas.jarosch at intra2net.com
Tue Apr 22 14:42:10 CEST 2008


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




More information about the devel mailing list