#40097 [WFx]: Using functions on each implode()-item

From: Date: Fri, 12 Jan 2007 20:48:28 +0000
Subject: #40097 [WFx]: Using functions on each implode()-item
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-107549@lists.php.net to get a copy of this message
ID: 40097 User updated by: marcel dot normann at experimentierkasten dot de Reported By: marcel dot normann at experimentierkasten dot de Status: Wont fix Bug Type: Feature/Change Request Operating System: all PHP Version: 6CVS-2007-01-11 (CVS) New Comment: Derick, you are right. But I have revised a lot of code from other developers last months and I found SQL injections made possible by unsecure use of implode() in almost every application. I think that situation can be improved by offering a callback in implode(). Previous Comments: ------------------------------------------------------------------------ [2007-01-11 08:09:07] derick@php.net You can use array walk for that: array_walk(implode('-', $string), 'callbackFunc'); ------------------------------------------------------------------------ [2007-01-11 07:44:07] marcel dot normann at experimentierkasten dot de Description: ------------ implode() is often used for doing such things like $sSql = "select * from somewhere where something in ('".implode("','",$aArray)."')"; This is nice for SQL injection. Maybe it would be a good idea to add a possibility to implode() for manipulating each implode()-item: string implode ( string glue, array pieces [, callback callback]) $aArray = array('some','boring','items'); echo implode('-',$aArray,'ucfirst'); // = Some-Boring-Items This example is not very usefull but imgine the use of mysql_escape_string() instead. ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=40097&edit=1

« previous php.bugs (#107549) next »