Removes some notice spooge from menu.inc. Although notices don't show up on a production server, the sheer number of these causes browser memory usage to explode if you're debugging your site and make it impossible to find out if your own code is producing any notice output.

CommentFileSizeAuthor
#2 40849_0.patch1.49 KBRichard Archer
#1 40849.patch1.38 KBWesley Tanaka
menu.spooge.patch608 bytesWesley Tanaka
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Wesley Tanaka’s picture

Wesley Tanaka CreditAttribution: Wesley Tanaka commented
Status: Reviewed & tested by the community » Needs work
FileSize
40849.patch1.38 KB

more of the same...

Richard Archer’s picture

Richard Archer CreditAttribution: Richard Archer commented
Status: Needs work » Needs review
FileSize
40849_0.patch1.49 KB

I have re-rolled this patch using the ternary operator.

Wesley, could you please make sure this patch still works as expected and change the issue status to "ready to commit" if so?

Wesley Tanaka’s picture

Wesley Tanaka CreditAttribution: Wesley Tanaka commented
Status: Needs review » Reviewed & tested by the community

40849_0.patch applied and worked for me.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)