note 50812 added to function.mysql-query

I've make a little change to safe_query function posted previously to accept direct variable if only 1 parameters is needed. + I dump the safe_query crash result into a query_log_file for quick and easy query debug. function safe_query($query, $values) { $query_parts = preg_split("/\?/", $query); $safe_query = array_shift($query_parts); $needed_values = count($query_parts); if (is_array($values)) { foreach ($values as $data) { $safe_query .= mysql_escape_string($values[$i]).array_shift($query_parts); } } else { $safe_query .= mysql_escape_string($values).array_shift($query_parts); } if (count($query_parts)) { $value_sent = (is_array($values)) ? count($values) : 1; $cf = fopen(QUERY_ERROR_LOG, "a"); flock( $cf, LOCK_EX ); fputs($cf, date("Y/m/d H:i:s").' -- '.$GLOBALS['PHP_SELF'].' -- '.mysql_error()."\n"); fputs($cf, 'Query '.$query.' needs '.$needed_values.' values, you only sent '.$value_sent."\n\n"); flock( $cf, LOCK_UN ); fclose($cf); die('Une erreure est survenue. S.V.P. faire parvenir un message à support@mysite.com afin de nous informer du problème. Merci'); } if (!$q_result = mysql_query($safe_query)) { $cf = fopen(QUERY_ERROR_LOG, "a"); flock( $cf, LOCK_EX ); fputs($cf, date("Y/m/d H:i:s").' -- '.$GLOBALS['PHP_SELF'].' -- '.mysql_error()."\n"); fputs($cf, $safe_query."\n\n"); flock( $cf, LOCK_UN ); fclose($cf); die('Une erreure est survenue. S.V.P. faire parvenir un message à support@mysite.com afin de nous informer du problème. Merci'); } return $q_result; } QUERY_ERROR_LOG is a define variable ex: define('QUERY_ERROR_LOG','/home/user/query_error_log.txt') you can call the safe_query like this: $query = "SELECT * FROM test WHERE a='?' AND b='?'" ; $values = array(1, 2, 3); $result = safe_query($query, $values); or like this $query = "SELECT * FROM test WHERE a='?'"; $result = safe_query($query, 1); ---- Manual Page -- http://www.php.net/manual/en/function.mysql-query.php Edit -- http://master.php.net/manage/user-notes.php?action=edit+50812 Delete: added to the manual -- http://master.php.net/manage/user-notes.php?action=delete+50812&report=yes&reason=added+to+the+manual Delete: bad code -- http://master.php.net/manage/user-notes.php?action=delete+50812&report=yes&reason=bad+code Delete: spam -- http://master.php.net/manage/user-notes.php?action=delete+50812&report=yes&reason=spam Delete: useless -- http://master.php.net/manage/user-notes.php?action=delete+50812&report=yes&reason=useless Delete: other reasons -- http://master.php.net/manage/user-notes.php?action=delete+50812&report=yes Reject -- http://master.php.net/manage/user-notes.php?action=reject+50812&report=yes Search -- http://master.php.net/manage/user-notes.php