Posted by mrfelton on October 4, 2012 at 12:31pm
6 followers
| Project: | Workbench |
| Version: | 7.x-1.2 |
| Component: | Code |
| Category: | support request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
I would have thought access to /admin/config/workbench should use the 'administer workbench' permission instead of 'access workbench'. The 'administer workbench' permission isn't actually used anywhere in workbench (or any of the workbench add on modules that we have enabled). As it is now, content editors (or anyone with the ability to access the workbench) can also acess the admin page at /admin/config/workbench - which just shows "You do not have any administrative items.", which is confusing.
Comments
#1
#2
The patch applied cleanly and fixed the configuration section using the wrong permission.
#3
Committed.
6857511..f40e723 7.x-1.x -> 7.x-1.x#4
ps, thanks for the (lack of) patch attribution! ;)
#5
Automatically closed -- issue fixed for 2 weeks with no activity.
#6
In workbench.module (7.x-1.2 version) I see this (as #1):
$items['admin/config/workbench'] = array('title' => 'Workbench',
'description' => 'Workbench',
'page callback' => 'system_admin_menu_block_page',
'access arguments' => array('administer workbench'),
[......]
but is not working for me, as administrator.
I dunno why this isn't working. Any idea?
#7
Have you cleared the cache / run update.php?
By "administrator" do you mean user 1?
#8
Yep, both cache cleared and I ran update.php.
And, yes, by "administrator" I mean user 1, of course :)
#9
Do you have any of the submodules enabled (Workbench Moderation, Workbench Access)? If not, this item won't appear because Workbench itself has no configuration.
#10
Oh sh**! Sorry :)
Anyway, there is a usability problem in this: if you give the ability to access to a configuration page, but the configuration page doesn't have any configurations is better to follow one of the next options:
1) the configuration page has no sense to exists;
2) if you need to have the configuration page, is better display a message like "There aren't configurations at the moment" instead of an access error message.
#11
Blame core's handling of configuration blocks. This works as designed.
#12
Automatically closed -- issue fixed for 2 weeks with no activity.