[cvs] commit: framework/Share/Share sql.php

Michael Rubinsky mike at theupstairsroom.com
Fri Jun 6 16:01:36 UTC 2008


Quoting "Jan Schneider" <jan at horde.org>:

> jan         2008-06-06 06:50:14 EDT
>
>   Modified files:
>     Share/Share          sql.php
>   Log:
>   $data['perm'] is not set here, so don't try to unset it.

I might be missing something here, but wouldn't it also be unnecessary  
to perform the isset($data['perm']['users']) and  
isset($data['perm']['groups']) checks in that block as well?  Want to  
make sure I'm thinking about that conditional correctly :)

-- 
Thanks
Mike

"Time just hates me, that's why it made me an adult." - Josh Joplin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: PGP Digital Signature
Url : http://lists.horde.org/archives/cvs/attachments/20080606/77e5801e/attachment.bin 


More information about the cvs mailing list