Seems there is a missing argument for hook_views_admin_links_alter() via drupal_alter() in theme.inc.

Very simple 1 line patch attached that fixes the issue by adding the $view to the drupal_alter() call.

CommentFileSizeAuthor
#3 views-drupal_alter-missing_argument.patch930 bytesdawehner
views-drupal_alter-missing_argument.patch643 bytespsynaptic
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

psynaptic’s picture

psynaptic CreditAttribution: psynaptic commented
Status: Active » Needs review
dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Needs work

this cannot work there is no $views there, just $view

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs work » Needs review
FileSize
views-drupal_alter-missing_argument.patch930 bytes

here is a updated patch

i think there should be some docs about using it in the help folder

psynaptic’s picture

psynaptic CreditAttribution: psynaptic commented

You are correct. Not sure how that got in there! Thanks for rerolling :)

psynaptic’s picture

psynaptic CreditAttribution: psynaptic commented

Please can someone commit this? It's a single line change. Without it we don't have any context in the drupal_alter call.

The Block Edit module will soon depend on this patch.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Needs review » Fixed

Patience. I've not had tons of free time lately. A week is actually pretty quick for me, nowadays.

psynaptic’s picture

psynaptic CreditAttribution: psynaptic commented

Sorry for harassing and thanks for committing!

Status: Fixed » Closed (fixed)

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