We're using the $options parameter in Pathauto now to help fix #881270: [bookpath], [menupath], [*path] tokens not cleaned: aliases without punctuation removed, lower casing, etc., but CCK doesn't include $options in content_token_values() and even worse doesn't pass $options along to other modules.

CommentFileSizeAuthor
#1 932680-content-token-values-options.patch2.24 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 932680-content-token-values-options.patch2.24 KB
greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Needs review » Reviewed & tested by the community

Looks solid to me.

gg4’s picture

gg4 commented

Thanks Dave. Looks good to me as well.

This patch is needed for #420240: "termpath" token support for hierarchical vocabularies for use with Pathauto to function correctly.

::

yched’s picture

yched commented

Looks like a no brainer. Will commit asap.

yched’s picture

yched commented

Sorry for the delay - committed to both 2.x and 3.x branches

yched’s picture

yched commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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