This patch came out of a discussion JonBob and I had on IRC. I saw an error under E_ALL, told 'im, he said "bugger!" and told me to edit this and that. So. Here ya go.

CommentFileSizeAuthor
jonbob_fix.patch873 bytesMorbus Iff
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TDobes’s picture

TDobes CreditAttribution: TDobes commented

+1

...but...

There's an extra set of parenthesis you can remove. Specifically,

+  while ($mid && (($menu['items'][$mid]['type'] & MENU_IS_LOCAL_TASK))) {

can become

+  while ($mid && ($menu['items'][$mid]['type'] & MENU_IS_LOCAL_TASK)) {
Dries’s picture

Dries CreditAttribution: Dries commented

Committed to HEAD. Thanks.

Anonymous’s picture

(not verified) CreditAttribution: commented