Closed (won't fix)
Project:
Nodequeue
Version:
4.7.x-1.1
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 Sep 2006 at 10:26 UTC
Updated:
28 May 2008 at 17:57 UTC
Hi,
I don't see the need to have the node queue tab cluter the interface if the node type cannot be added to a queue. This can be done with a simple check in nodequeue_menu(), line 66 of // $Id: nodequeue.module,v 1.36 2006/08/03 16:28:59 merlinofchaos Exp $
if (nodequeue_node_access($node->type)) {
$items[] = array(
'path' => 'node/' . $arg1 . '/nodequeue',
'title' => t('nodequeue'),
'access' => $access,
'callback' => 'nodequeue_page',
'callback arguments' => array($arg1),
'type' => MENU_LOCAL_TASK,
'weight' => 5
);
}
Do you think this makes sense ?
Comments
Comment #1
sethcohn commentedThis code was added but commented out in the current 4.7 release.
Any good reason why it's commented out? If not, please uncomment it, as it makes sense.
Comment #2
merlinofchaos commentedThis seems to be back in 1.2
Comment #3
merlinofchaos commentedThe nodequeue 1.x line is no longer available.