To cleanup some strings in the punctuation settings, and improve performance and extensibility, we should cache pathauto_punctuation_chars() not just statically so that t() does not even need to be called more than once per several requests. We should also allow this list to be altered so that sites with odd characters do not have to hack the module.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Status: Active » Needs review
FileSize
10.8 KB
Dave Reid’s picture

Without the extra hunks this time.

Dave Reid’s picture

Status: Needs review » Fixed
greggles’s picture

I think the solution for "We should also allow this list to be altered so that sites with odd characters do not have to hack the module." is that those folks should use transliteration module.

But, I defer, of course ;)

Status: Fixed » Closed (fixed)

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