Needs work
Project:
Taxonomy Term Permissions
Version:
6.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
27 Dec 2009 at 18:24 UTC
Updated:
28 Jun 2010 at 15:14 UTC
Jump to comment: Most recent file
Comments
Comment #1
elliotttt commentedI'm doing some cleanup on this module- will address this.
Comment #2
elliotttt commentedDoing a significant proposed overhaul of this module, so this patch actually removes some functionality, but addresses the aforementioned bug.
Things this patch removes:
Everything related to roles and term permissions. You've been warned! :)
Things this patch fixes/alters:
-The term count (as mentioned above)
-Moves the permissions control out of the taxonomy term page, and into the user/edit pages
-Checks to make sure the user making edits has the 'administer users' permission
-Changed the way the node/add form checks against the permissions
I figured I'd post this milestone, I plan on adding back in the roles functionality.
Comment #3
deviantintegral commentedThanks for the patch. It's much more difficult to test and review all of the changes when they are combined in a single patch, so it would be appreciated if it could be split up where possible.
Comment #4
elliotttt commentedupdated, this patch adds back in the ability to manage user's access to term permissions by role in addition to by user.
The role permissions have been moved into the normal system permissions page, since that's generally where role based permissions get administered.
I believe this change also makes it so we no longer need the second db cross ref table for managing role based permissions.
Comment #5
elliotttt commentedoops, realized I should have changed the status to needs review
Comment #6
elliotttt commentedfound a problem where the roles fix only works some of the time, looking into it
Comment #7
stephenrobinson commentedHi,
Multiple selects don't have a please select option, using this line addresses the multiple select issue:
//$total_terms = count($vocabulary['#options']) - 1;
($vocabulary['#multiple']==1) ? $total_terms = count($vocabulary['#options']) : $total_terms = count($vocabulary['#options']) - 1;