Could also be done using sections module etc, but adding a whole extra module for this seems silly.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xurizaemon’s picture

patch

xurizaemon’s picture

Status: Active » Needs review

i really really really have to write a greasemonkey that doesn't let me add a patch without reminding me to set "needs review" :P

fago’s picture

Status: Needs review » Needs work

Makes sense, but shouldn't that be arg(2) == 'access' ? I wonder whether this patch really works?

xurizaemon’s picture

totally true, this is what it oughta be

admin.css may be redundant too?

fago’s picture

Sry for not noting before, the comment misses the ending point "." - probably the rest of the module doesn't follow this yet, but let's do it at least for new code.

Also an @see system_init() would be good. The .css should be fine as the system module does it too, though I don't know what for.

xurizaemon’s picture

Status: Needs work » Needs review
FileSize
1.27 KB
fago’s picture

Status: Needs review » Fixed

thanks, committed.

Status: Fixed » Closed (fixed)

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