Edit report at https://bugs.php.net/bug.php?id=64852&edit=1
ID: 64852 Updated by: vr...@php.net Reported by: vr...@php.net Summary: Allow binding arrays in PDO Status: Open Type: Bug Package: PDO related Operating System: Any PHP Version: 5.5.0RC1 -Assigned To: +Assigned To: wez Block user comment: N Private report: N New Comment: Wez, do you think it's reasonable? Previous Comments: ------------------------------------------------------------------------ [2013-05-16 05:22:04] vr...@php.net Description: ------------ Binding arrays would be useful in these queries: WHERE id IN ? INSERT INTO ... VALUES ? Most database systems don't support binding complex data structures on server side but PDO supports client side data binding so it can expand arrays to scalars and use them. Example: <?php $stmt = $pdo->prepare("SELECT * FROM table WHERE id IN ?"); $stmt->execute([ array('1', '2', '3') ]); // This will be executed with client side data binding: // SELECT * FROM table WHERE id IN ('1', '2', '3') // With server side data binding: // SELECT * FROM table WHERE id IN (?, ?, ?) -- bind values: '1', '2', '3' ?> It means that with server side data binding, arrays will be expanded to variable number of placeholders and the elements in the array will be bound. There is a risk that the same statement would be used with a differently structured array or with non-array. Example: <?php $stmt = $pdo->prepare("SELECT * FROM table WHERE id IN ?"); // Expands query to: SELECT * FROM table WHERE id IN (?, ?, ?) $stmt->execute([ array(1, 2, 3) ]); // This subsequent call should throw. $stmt->execute([ array(1, 2, 3, 4) ]); // This subsequent call should also throw. $stmt->execute([ 1 ]); ?> This is a very rare usage and throwing an error in this case seems like a good solution. === Named parameters === Named parameters could expand to name-key pair: <?php $stmt = $pdo->prepare("SELECT * FROM table WHERE id IN :ids"); // Expands query to: SELECT * FROM table WHERE id IN (:ids_0, :ids_1, :ids_2) $stmt->execute([ 'ids' => array(1, 2, 3) ]); ?> However, there could be a collision: "WHERE id = :ids_0 OR id IN :ids". PDO could solve it by some sort of escaping - e.g. by prepending a magic string to all array names or by prepending something else also to all non-array names. Or it could just throw as this would be a rare and easily fixable problem. === Nested arrays === Expanding arrays should be recursive to support these queries: <?php $stmt = $pdo->prepare("SELECT * FROM table WHERE (type, type_id) IN ?"); // Expands to SELECT * FROM table WHERE (type, type_id) IN ((?, ?), (?, ?)) $types = array(); $types[] = array('article', 1); $types[] = array('comment', 11); $stmt->execute([ $types ]); ?> === Braces or no braces === Array should expand to comma-delimited, braces-enclosed string. This expansion would support queries "WHERE (type, type_id) IN ?". It unfortunately wouldn't support this query: INSERT INTO ... VALUES (...), (...), ... This query needs braces in inner array and no braces in outer array so there's no consistent way to support this type of query. === Empty arrays === Empty arrays should be treated same as other arrays so they should expand to (). "INSERT INTO table () VALUES ()" is a valid query, at least in MySQL (it inserts a row with all default values). This would cause a syntax error in query "WHERE id IN ()" but that's a good behavior as there's no way to tell database to not match anything. "WHERE id IN (NULL)" would be a solution in this particular query (as NULL doesn't equal to NULL) but "WHERE id NOT IN (NULL)" wouldn't return NULL rows. So empty array must expand to (). === Debugging === PDO should disclose a method returning the real query sent to the server. It would be useful even without arrays, especially with client-side binding. === Implementation === Implementation would be tricky as the statement couldn't be prepared until it's executed. It means that PDO wouldn't talk to the database server in prepare() even with server-side binding (this is the current behavior with client-side binding). The query would be both prepared and executed in execute(). ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=64852&edit=1