Skip to content

Commit 5a52807

Browse files
author
Julien Pauli
committed
Merge branch 'PHP-5.6'
* PHP-5.6: Updated NEWS Updated NEWS Start adding new attribute to control multi statements
2 parents 8ab9396 + 0e67973 commit 5a52807

6 files changed

+366
-4
lines changed

ext/pdo_mysql/mysql_driver.c

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -556,10 +556,14 @@ static int pdo_mysql_handle_factory(pdo_dbh_t *dbh, zval *driver_options TSRMLS_
556556
#ifdef CLIENT_MULTI_RESULTS
557557
|CLIENT_MULTI_RESULTS
558558
#endif
559+
;
559560
#ifdef CLIENT_MULTI_STATEMENTS
560-
|CLIENT_MULTI_STATEMENTS
561+
if (!driver_options) {
562+
connect_opts |= CLIENT_MULTI_STATEMENTS;
563+
} else if (pdo_attr_lval(driver_options, PDO_MYSQL_ATTR_MULTI_STATEMENTS, 1 TSRMLS_CC)) {
564+
connect_opts |= CLIENT_MULTI_STATEMENTS;
565+
}
561566
#endif
562-
;
563567

564568
#if defined(PDO_USE_MYSQLND)
565569
int dbname_len = 0;

ext/pdo_mysql/pdo_mysql.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ static PHP_MINIT_FUNCTION(pdo_mysql)
126126
#if MYSQL_VERSION_ID > 50605 || defined(PDO_USE_MYSQLND)
127127
REGISTER_PDO_CLASS_CONST_LONG("MYSQL_ATTR_SERVER_PUBLIC_KEY", (zend_long)PDO_MYSQL_ATTR_SERVER_PUBLIC_KEY);
128128
#endif
129-
129+
REGISTER_PDO_CLASS_CONST_LONG("MYSQL_ATTR_MULTI_STATEMENTS", (zend_long)PDO_MYSQL_ATTR_MULTI_STATEMENTS);
130130

131131
#ifdef PDO_USE_MYSQLND
132132
mysqlnd_reverse_api_register_api(&pdo_mysql_reverse_api TSRMLS_CC);

ext/pdo_mysql/php_pdo_mysql_int.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,8 +172,9 @@ enum {
172172
PDO_MYSQL_ATTR_SSL_CAPATH,
173173
PDO_MYSQL_ATTR_SSL_CIPHER,
174174
#if MYSQL_VERSION_ID > 50605 || defined(PDO_USE_MYSQLND)
175-
PDO_MYSQL_ATTR_SERVER_PUBLIC_KEY
175+
PDO_MYSQL_ATTR_SERVER_PUBLIC_KEY,
176176
#endif
177+
PDO_MYSQL_ATTR_MULTI_STATEMENTS,
177178
};
178179

179180
#endif
Lines changed: 95 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,95 @@
1+
--TEST--
2+
PDO::MYSQL_ATTR_MULTI_STATEMENTS
3+
--SKIPIF--
4+
<?php
5+
require_once(dirname(__FILE__) . DIRECTORY_SEPARATOR . 'skipif.inc');
6+
require_once(dirname(__FILE__) . DIRECTORY_SEPARATOR . 'mysql_pdo_test.inc');
7+
MySQLPDOTest::skip();
8+
$db = MySQLPDOTest::factory();
9+
?>
10+
--INI--
11+
error_reporting=E_ALL
12+
--FILE--
13+
<?php
14+
require_once(dirname(__FILE__) . DIRECTORY_SEPARATOR . 'mysql_pdo_test.inc');
15+
16+
$dsn = MySQLPDOTest::getDSN();
17+
$user = PDO_MYSQL_TEST_USER;
18+
$pass = PDO_MYSQL_TEST_PASS;
19+
20+
$table = sprintf("test_%s", md5(mt_rand(0, PHP_INT_MAX)));
21+
$db = new PDO($dsn, $user, $pass);
22+
$db->exec(sprintf('DROP TABLE IF EXISTS %s', $table));
23+
$create = sprintf('CREATE TABLE %s(id INT)', $table);
24+
$db->exec($create);
25+
$db->exec(sprintf('INSERT INTO %s(id) VALUES (1)', $table));
26+
$stmt = $db->query(sprintf('SELECT * FROM %s; INSERT INTO %s(id) VALUES (2)', $table, $table));
27+
$stmt->closeCursor();
28+
$info = $db->errorInfo();
29+
var_dump($info[0]);
30+
$stmt = $db->query(sprintf('SELECT id FROM %s', $table));
31+
var_dump($stmt->fetchAll(PDO::FETCH_ASSOC));
32+
// A single query with a trailing delimiter.
33+
$stmt = $db->query('SELECT 1 AS value;');
34+
var_dump($stmt->fetchAll(PDO::FETCH_ASSOC));
35+
36+
// New connection, does not allow multiple statements.
37+
$db = new PDO($dsn, $user, $pass, array(PDO::MYSQL_ATTR_MULTI_STATEMENTS => false));
38+
$stmt = $db->query(sprintf('SELECT * FROM %s; INSERT INTO %s(id) VALUES (3)', $table, $table));
39+
var_dump($stmt);
40+
$info = $db->errorInfo();
41+
var_dump($info[0]);
42+
43+
$stmt = $db->query(sprintf('SELECT id FROM %s', $table));
44+
var_dump($stmt->fetchAll(PDO::FETCH_ASSOC));
45+
// A single query with a trailing delimiter.
46+
$stmt = $db->query('SELECT 1 AS value;');
47+
var_dump($stmt->fetchAll(PDO::FETCH_ASSOC));
48+
49+
$db->exec(sprintf('DROP TABLE IF EXISTS %s', $table));
50+
print "done!";
51+
?>
52+
--EXPECTF--
53+
string(5) "00000"
54+
array(2) {
55+
[0]=>
56+
array(1) {
57+
["id"]=>
58+
string(1) "1"
59+
}
60+
[1]=>
61+
array(1) {
62+
["id"]=>
63+
string(1) "2"
64+
}
65+
}
66+
array(1) {
67+
[0]=>
68+
array(1) {
69+
["value"]=>
70+
string(1) "1"
71+
}
72+
}
73+
bool(false)
74+
string(5) "42000"
75+
array(2) {
76+
[0]=>
77+
array(1) {
78+
["id"]=>
79+
string(1) "1"
80+
}
81+
[1]=>
82+
array(1) {
83+
["id"]=>
84+
string(1) "2"
85+
}
86+
}
87+
array(1) {
88+
[0]=>
89+
array(1) {
90+
["value"]=>
91+
string(1) "1"
92+
}
93+
}
94+
done!
95+

ext/pdo_mysql/tests/pdo_mysql_class_constants.phpt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ if (!extension_loaded('mysqli') && !extension_loaded('mysqlnd')) {
2626
"MYSQL_ATTR_SSL_CAPATH" => true,
2727
"MYSQL_ATTR_SSL_CIPHER" => true,
2828
"MYSQL_ATTR_COMPRESS" => true,
29+
"MYSQL_ATTR_MULTI_STATEMENTS" => true,
2930
);
3031

3132
if (!MySQLPDOTest::isPDOMySQLnd()) {

0 commit comments

Comments
 (0)