Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.
It looks good, but 'commerce_autosku_cleanstring' should be run before converting the case, otherwise transliteration's case settings overrule autosku's settings.
Unless there is a reason behind the current order of things.
Comments
Comment #1
rbayliss commentedLet me know how this looks.
Comment #2
karol haltenberger commentedIt looks good, but 'commerce_autosku_cleanstring' should be run before converting the case, otherwise transliteration's case settings overrule autosku's settings.
Unless there is a reason behind the current order of things.
commerce_autosku.commerce.inc lines 70-80
Comment #3
rbayliss commentedThanks for the review. I can't see any reason to run the cleanstring function later. Fixed and pushed.