The token browser is automatically added to the configuration form of certain system actions, but only during the "add" operation and not on "edit". This makes editing form actions harder than it needs to be and is easily fixed by implementing hook_form_BASE_FORM_ID_alter instead of hook_form_FORM_ID_alter.

CommentFileSizeAuthor
#2 token-token_browser_on_action_edit-2868244-2.patch694 bytesckaotik
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ckaotik created an issue. See original summary.

ckaotik’s picture

ckaotik
Primary language German
Location Berlin
CreditAttribution: ckaotik at werk21 commented
Assigned: ckaotik » Unassigned
Status: Active » Needs review
FileSize
token-token_browser_on_action_edit-2868244-2.patch694 bytes

See the attached patch.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Fixed

Thanks, committed.

  • Berdir committed fbaeaf6 on 8.x-1.x authored by ckaotik 
    Issue #2868244 by ckaotik: Token Browser does not appear on system...

Status: Fixed » Closed (fixed)

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

LpSolit’s picture

LpSolit CreditAttribution: LpSolit as a volunteer commented

The switch block should be enclosed in

if (isset($form['plugin'])) {
...
}

else the filebrowser module throws:

Undefined index: plugin in token_form_action_form_alter() (line 343 of modules/token/token.module).

With this patch backed out, the filebrowser module doesn't throw such messages, so this problem is new in Token 1.0.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented

Please create a new issue, with a patch if possible, thanks!