Menu item needs access arguments to be accessible

Garrett Albright - August 16, 2008 - 23:35
Project:Talk
Version:6.x-1.4
Component:Code
Category:bug report
Priority:critical
Assigned:cwgordon7
Status:closed
Description

I was quite surprised when I peeked at Talk's source code and saw that it had an admin page. It doesn't show up on the admin page menus because it didn't have an 'access arguments' element, which is required for all menu items after Drupal 6.2 (I think). Here's a patch which adds such an element and makes the admin page appear in the administration menu. Note that you'll need to rebuild your menu cache after applying this patch. (If you don't know how to do that, try going to your site's update.php page. Your caches will be dumped while going through the update process, even though nothing will really be updated.)

This is a pretty critical issue… Is Talk still being maintained?

AttachmentSize
access.patch450 bytes

#1

cwgordon7 - August 25, 2008 - 07:30
Assigned to:Anonymous» cwgordon7
Status:needs review» reviewed & tested by the community

Yes, yes, yes... this was not necessary before Drupal 6.2.

#2

bsherwood - August 27, 2008 - 06:25

Can we roll the patch in and create another stable version?

#3

cwgordon7 - September 17, 2008 - 20:56
Status:reviewed & tested by the community» fixed

Should be fixed in 6.x-1.5.

#4

Anonymous (not verified) - October 1, 2008 - 21:24
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.