UTF-8 strings (e.g. Russian, Turkish, etc.) must use drupal_strtolower() rather than strtolower()

CommentFileSizeAuthor
#6 drupal_functions_for-2493389-6.patch674 bytesnilesh.addweb
strtolower.patch1.4 KBmfb

Comments

  • WorldFallz committed 2bd899a on 7.x-4.x authored by mfb 
    Issue #2493389 by mfb: drupal_strtolower() must be used on UTF-8 strings
WorldFallz’s picture

WorldFallz commented
Status: Needs review » Fixed

yowza-- good catch. Thanks for the patch!

mfb’s picture

mfb
they or he
commented

Thanks. Might be worth checking that other string functions are ok? Drupal also provides the UTF-8 compatible drupal_strlen() and drupal_substr() functions.

WorldFallz’s picture

WorldFallz commented

yep... totally agreed. I'll try to get to it next. if you get to it before me, you can just attach a subsequent patch here.

Thanks again!

WorldFallz’s picture

WorldFallz commented
Title: drupal_strtolower() must be used on UTF-8 strings » drupal_ functions for string handling
Status: Fixed » Active
nilesh.addweb’s picture

nilesh.addweb commented
Status: Active » Needs review
StatusFileSize
new drupal_functions_for-2493389-6.patch674 bytes

Attached patch uses drupal_strlen() and drupal_substr().

  • WorldFallz committed 2bd899a on 8.x-1.x authored by mfb 
    Issue #2493389 by mfb: drupal_strtolower() must be used on UTF-8 strings
sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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