Working on NodeFlag I need to allow users access to node, flag, and user tokens for the messages. However, the way the theme_token_help and token_get_list work currently you can select 'all' or a single token type. This patch changes theme_token_help and token_get_list to expect an array of token types like token_replace_multiple. It also changes the default value of types back to global instead of all for theme_token_help and token_get_list.

This patch changes the function prototypes, so modules depending on token.module will need to be updated.

Comments

dopry’s picture

StatusFileSize
new3.32 KB

I bet it helps if I upload the patch... oh this patch also splits token_help and theme_token_help

greggles’s picture

Just for the record, I love this conceptually, and this should be on a 5.x-2.x branch (haven't tested yet, though).

dopry’s picture

StatusFileSize
new1.93 KB

Here is a patch against 6.x.

dopry’s picture

StatusFileSize
new3.45 KB

Here is on for D6 with the doc block updates as well since I forgot them when rewriting the patch.

dave reid’s picture

Version: 5.x-1.x-dev » 7.x-1.x-dev
Component: Code » User interface

Status: Needs review » Needs work

The last submitted patch, token-help-DRUPAL-6--1.patch, failed testing.

dave reid’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs work » Fixed

The module currently allows you to pass an array of types. I don't see what's needed here.

Status: Fixed » Closed (fixed)

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