Problem/Motivation

The dialog option for token browser is not used and is going to be removed as per #2640138: Use token_tree_link instead of token_tree to render links. There is only place where the token tree is called and is intended to show the browser inline. There is no real need for that and it would be better if it is a link. We should use the proper theme function for this link.

Proposed resolution

Change the theme call.

Remaining tasks

Patch, review, commit

User interface changes

None

API changes

None

Data model changes

None

CommentFileSizeAuthor
#2 remove_dialog_option-2640284-2.patch950 byteshussainweb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hussainweb created an issue. See original summary.

hussainweb’s picture

hussainweb
Primary language English
CreditAttribution: hussainweb at Axelerant commented
Status: Active » Needs review
FileSize
remove_dialog_option-2640284-2.patch950 bytes

Here is the patch which changes the call to theme('token_tree_link').

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Reviewed & tested by the community
miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Reviewed & tested by the community » Fixed

Committed, thx!

Status: Fixed » Closed (fixed)

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