To allow #881270: [bookpath], [menupath], [*path] tokens not cleaned: aliases without punctuation removed, lower casing, etc. to proceed, we need to roll back #587148: Handle tokens correctly when more than one module defines a given token because token.module should *not* alter the token values itself. Rather we should add token_requirements and check for duplicate token definitions using hook_token_list(). That way, we can actually show which modules are the ones that conflict, rather than showing a watchdog message every time tokens are used.

CommentFileSizeAuthor
#1 921988-token-dupe-requirements.patch4.84 KBDave Reid
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Status: Active » Needs review
FileSize
4.84 KB
greggles’s picture

I did not test, but this works for me in theory.

Dave Reid’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Needs review » Patch (to be ported)

Committed to 6.x and 5.x:
http://drupal.org/cvs?commit=427016
http://drupal.org/cvs?commit=427018

Needs to be ported to D7.

Dave Reid’s picture

Status: Patch (to be ported) » Fixed

Status: Fixed » Closed (fixed)

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