Unicode & strtolower

vitamin - March 6, 2007 - 10:08
Project:Node list
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:wmostrey
Status:closed
Description

Why not to use drupal_strtolower() (see includes/unicode.inc) instead of strtolower? This would eliminate the need for user to patch nodelist.module with mb_strtolower(), since this function provide graceful fallback to ordinary strtolower() if none mbstirng extension found.

#1

wmostrey - April 25, 2007 - 08:15
Assigned to:Anonymous» wmostrey
Status:active» needs review

This is a patch against the latest version. Please confirm that all is ok.

AttachmentSize
nodelist.patch 6.89 KB

#2

wmostrey - June 20, 2007 - 12:13
Status:needs review» fixed

Tested and confirmed

#3

Anonymous - July 4, 2007 - 12:17
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.