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?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Leeteq’s picture

Not 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.

Cyberwolf’s picture

IMHO 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.".

mrfelton’s picture

Status: Active » Needs review
FileSize
630 bytes

I agree. Admin should be able to access the page. Patch attached.

dkingofpa’s picture

Patch 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.

pearcec’s picture

Rerolled this from a CVS tagged checkout of Drupal 6.x-1.3 so it works from a drush make.

NancyDru’s picture

Status: Needs review » Fixed

Similar idea, different code. There is now a setting to show the menu to admins.

Committed to 6.x-1.x-dev.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.