If the -raw version of a token is supposed to be the unprocessed version of that string, shouldn't 'current-page-path' be the one that has check_plain() ran against it rather than 'current-page-path-raw'? I.e. should these lines:

      $alias = drupal_get_path_alias($_GET['q']);
      $values['current-page-path'] = $alias;
      $values['current-page-path-raw'] = check_plain($alias);

instead say:

      $alias = drupal_get_path_alias($_GET['q']);
      $values['current-page-path'] = check_plain($alias);
      $values['current-page-path-raw'] = $alias;

??

Comments

damienmckenna’s picture

Status: Active » Needs review
StatusFileSize
new883 bytes

Here's a patch that corrects the issue.

damienmckenna’s picture

StatusFileSize
new909 bytes

Updated patch that makes the -raw vs non-raw code slightly more consistent with how it should be.

dave reid’s picture

Status: Needs review » Fixed

Well dang me. Fixed in CVS.
http://drupal.org/cvs?commit=444768

Status: Fixed » Closed (fixed)

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