[horde] sql share driver: groups uid and integer cast

Chuck Hagenbuch chuck at horde.org
Thu Dec 4 17:09:21 UTC 2008


Quoting Dominique LALOT <dom.lalot at gmail.com>:

> #                     
> $share['perm']['groups'][*(int)*$row['group_uid']] =  
> (int)$row['perm'];
>
> That instruction was the problem converting my groups uid to 0 (no  
> warning on cast..) which does not exists

> I know that's not an Horde bug properly speaking. But in sql  
> database, the group_uid is varchar(255). So why should we cast?
> I also suppose there's another cast somewhere else as one of my  
> group_uid saved is 0.
> Is it reasonable to change that line?

Yes, if that still exists in CVS it should be removed. Please open a  
bug on bugs.horde.org, with a patch if possible.

Thanks,
-chuck


More information about the horde mailing list