system_init() calls arg() but thats not available. simple patch.

CommentFileSizeAuthor
#4 patch_801.24 KBmoshe weitzman
patch_78683 bytesmoshe weitzman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

moshe weitzman’s picture

Priority: Normal » Critical
Status: Needs review » Needs work

even worse, the presence of system_init() function causes all 1600 lines of system.module to get loaded for every cached page view. the contents of that function should move to system_menu(!$may_cache). then this bug will be fixed, and bootstrap will be unburdened.

pwolanin’s picture

Is this true for for the 4.7 branch as well?

moshe weitzman’s picture

no, this went in with the new adminstration UI. will be an easy fix, fortunately.

moshe weitzman’s picture

Status: Needs work » Needs review
FileSize
1.24 KB
chx’s picture

Status: Needs review » Reviewed & tested by the community
Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)