[#6619] Shares: Optionally show all groups
Summary Shares: Optionally show all groups
Queue Horde Base
Queue Version HEAD
Type Enhancement
State Resolved
Priority 1. Low
Owners Chuck Hagenbuch <chuck (at) horde (dot) org>
Requester thomas (dot) jarosch (at) intra2net (dot) com
Created 04/17/2008 (78 days ago)
Due
Updated 04/22/2008 (73 days ago)
Assigned
Resolved 04/22/2008 (73 days ago)
Attachments horde-share-show-all-groups.patch Download
kronolith-share-show-all-groups.patch Download
Milestone
Patch No

History
04/22/2008 Chuck Hagenbuch Comment #4
State ⇒ Resolved
Assigned to Chuck Hagenbuch		 Reply to this comment
Committed, thanks!
04/22/2008 thomas (dot) jarosch (at) intra2net (dot) com Comment #3
New Attachment: kronolith-share-show-all-groups.patch Download		 Reply to this comment
Here's the patch for kronolith. I've not tested it as I don't have 
kronolith installed right now. Anyway, the code in kronolith already 
used listGroups() which was inconsistent with the Horde base version. 
Should be fixed now.

Thomas

04/22/2008 Chuck Hagenbuch Comment #2
State ⇒ Feedback		 Reply to this comment
I think I'm fine with this, but you'd need to patch 
kronolith/perms.php too - I think that's the only other place.
04/17/2008 thomas (dot) jarosch (at) intra2net (dot) com Comment #1
New Attachment: horde-share-show-all-groups.patch Download
Patch ⇒
Milestone ⇒
Queue ⇒ Horde Base
Summary ⇒ Shares: Optionally show all groups
Type ⇒ Enhancement
Priority ⇒ 1. Low
State ⇒ New		 Reply to this comment
Hello together,

if you enable sharing of Horde's shares, users can grant access to 
groups of users. Normally the user can only see groups he/she is a 
member of. The attached patch allows you to optionally show all groups 
regardless of the membership.

This is useful if you want to grant access f.e. to the group "management"
even if you are not part of that group.

Cheers,
Thomas