This had confused me for a while and led me to believe that I was experiencing the following problems:
http://drupal.org/node/256432
http://drupal.org/node/291665
Apparently, I am used to having access to everything as the primary admin.
If the admin itself is not in the roles that are allowed access via role_delegation to any particular vocabulary, then the admin user cannot access them.
This is can be misleading as most people (or so I assume) assume that the admin user can see/do everything on the system.
Perhaps you could either make a comment about this in the documentation or add a check for admin user in the permissions handler?
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 427616-taxonomy_delegate-admin_access-1.3.patch | 2.3 KB | pearcec |
| #4 | 427616-taxonomy_delegate-admin_access-1.3.patch | 2.18 KB | dkingofpa |
| #3 | 427616-taxonomy_delegate-admin-access.patch | 630 bytes | mrfelton |
Comments
Comment #1
Leeteq commentedNot sure if this should be regarded as a bug. I think it is generally preferable if modules require role access even for uid1, when such a feature is in place.
Personally, I never use admin1 for other than system update(.php) tasks, in particular to be sure at all times that I quickly spot whenever the role-based security settings does not work well.
If working with uid1, you may be fooled to believe that all is well, and potentially waste time if finding out "too late" that something is wrong with the security.
Comment #2
cyberwolf commentedIMHO it would be good to allow users with the permission "administer taxonomy", including admin users, to be able to see the menu item.
Quick fix in _taxonomy_delegate_access():
/**
* Menu access checking function.
*/
function _taxonomy_delegate_access() {
global $user;
return count(_taxonomy_delegate_my_vocabularies()) > 0 || user_access('administer taxonomy');
}
This also allows admin to add new menu items pointing to category_admin. Otherwise, he gets the message: "The path 'category_admin' is either invalid or you do not have access to it.".
Comment #3
mrfelton commentedI agree. Admin should be able to access the page. Patch attached.
Comment #4
dkingofpa commentedPatch for currently released version (1.3). I can't use a moving target like dev in my environment. Moved the user_access('administer taxonomy') call to _taxonomy_delegate_my_vocabularies() so admin can see all delegated vocabularies as well as the menu link.
Comment #5
pearcec commentedRerolled this from a CVS tagged checkout of Drupal 6.x-1.3 so it works from a drush make.
Comment #6
nancydruSimilar idea, different code. There is now a setting to show the menu to admins.
Committed to 6.x-1.x-dev.