Skip to content

Commit a9500c4

Browse files
SakiTakamachiSakiTakamachi
committed
[pdo] fix set and get attr
1 parent dfaf798 commit a9500c4

14 files changed

+63
-44
lines changed

‎ext/pdo/pdo_dbh.c

+3
Original file line numberDiff line numberDiff line change
@@ -913,6 +913,9 @@ PHP_METHOD(PDO, getAttribute)
913913
return;
914914
case PDO_ATTR_DEFAULT_FETCH_MODE:
915915
RETURN_LONG(dbh->default_fetch_type);
916+
917+
case PDO_ATTR_STRINGIFY_FETCHES:
918+
RETURN_BOOL(dbh->stringify);
916919
default:
917920
break;
918921
}

‎ext/pdo_dblib/dblib_driver.c

+4-4
Original file line numberDiff line numberDiff line change
@@ -290,10 +290,10 @@ static bool dblib_set_attr(pdo_dbh_t *dbh, zend_long attr, zval *val)
290290
}
291291
return SUCCEED == dbsettime(lval);
292292
case PDO_DBLIB_ATTR_STRINGIFY_UNIQUEIDENTIFIER:
293-
if (!pdo_get_long_param(&lval, val)) {
293+
if (!pdo_get_bool_param(&bval, val)) {
294294
return false;
295295
}
296-
H->stringify_uniqueidentifier = lval;
296+
H->stringify_uniqueidentifier = bval;
297297
return true;
298298
case PDO_DBLIB_ATTR_SKIP_EMPTY_ROWSETS:
299299
if (!pdo_get_bool_param(&bval, val)) {
@@ -302,10 +302,10 @@ static bool dblib_set_attr(pdo_dbh_t *dbh, zend_long attr, zval *val)
302302
H->skip_empty_rowsets = bval;
303303
return true;
304304
case PDO_DBLIB_ATTR_DATETIME_CONVERT:
305-
if (!pdo_get_long_param(&lval, val)) {
305+
if (!pdo_get_bool_param(&bval, val)) {
306306
return false;
307307
}
308-
H->datetime_convert = lval;
308+
H->datetime_convert = bval;
309309
return true;
310310
default:
311311
return false;

‎ext/pdo_firebird/firebird_driver.c

+13-1
Original file line numberDiff line numberDiff line change
@@ -1080,7 +1080,7 @@ static int pdo_firebird_get_attribute(pdo_dbh_t *dbh, zend_long attr, zval *val)
10801080
char tmp[INFO_BUF_LEN];
10811081

10821082
case PDO_ATTR_AUTOCOMMIT:
1083-
ZVAL_LONG(val,dbh->auto_commit);
1083+
ZVAL_BOOL(val,dbh->auto_commit);
10841084
return 1;
10851085

10861086
case PDO_ATTR_CONNECTION_STATUS:
@@ -1124,6 +1124,18 @@ static int pdo_firebird_get_attribute(pdo_dbh_t *dbh, zend_long attr, zval *val)
11241124
case PDO_ATTR_FETCH_TABLE_NAMES:
11251125
ZVAL_BOOL(val, H->fetch_table_names);
11261126
return 1;
1127+
1128+
case PDO_FB_ATTR_DATE_FORMAT:
1129+
ZVAL_STRING(val, H->date_format);
1130+
return 1;
1131+
1132+
case PDO_FB_ATTR_TIME_FORMAT:
1133+
ZVAL_STRING(val, H->time_format);
1134+
return 1;
1135+
1136+
case PDO_FB_ATTR_TIMESTAMP_FORMAT:
1137+
ZVAL_STRING(val, H->timestamp_format);
1138+
return 1;
11271139
}
11281140
return 0;
11291141
}

‎ext/pdo_firebird/tests/autocommit_change_mode.phpt

+8-8
Original file line numberDiff line numberDiff line change
@@ -91,42 +91,42 @@ echo "done!";
9191
--EXPECT--
9292
========== not in manually transaction ==========
9393
auto commit ON from ON
94-
int(1)
94+
bool(true)
9595
Success
9696

9797
auto commit OFF from ON
98-
int(0)
98+
bool(false)
9999
Success
100100

101101
auto commit OFF from OFF
102-
int(0)
102+
bool(false)
103103
Success
104104

105105
auto commit ON from OFF
106-
int(1)
106+
bool(true)
107107
Success
108108

109109
========== in manually transaction ==========
110110
begin transaction
111111

112112
auto commit ON from ON, expect error
113-
int(1)
113+
bool(true)
114114
SQLSTATE[HY000]: General error: Cannot change autocommit mode while a transaction is already open
115115

116116
auto commit OFF from ON, expect error
117-
int(1)
117+
bool(true)
118118
SQLSTATE[HY000]: General error: Cannot change autocommit mode while a transaction is already open
119119

120120
end transaction
121121
auto commit OFF
122122
begin transaction
123123

124124
auto commit ON from OFF, expect error
125-
int(0)
125+
bool(false)
126126
SQLSTATE[HY000]: General error: Cannot change autocommit mode while a transaction is already open
127127

128128
auto commit OFF from OFF, expect error
129-
int(0)
129+
bool(false)
130130
SQLSTATE[HY000]: General error: Cannot change autocommit mode while a transaction is already open
131131

132132
end transaction

‎ext/pdo_mysql/mysql_driver.c

+6-2
Original file line numberDiff line numberDiff line change
@@ -532,7 +532,7 @@ static int pdo_mysql_get_attribute(pdo_dbh_t *dbh, zend_long attr, zval *return_
532532
break;
533533

534534
case PDO_ATTR_AUTOCOMMIT:
535-
ZVAL_LONG(return_value, dbh->auto_commit);
535+
ZVAL_BOOL(return_value, dbh->auto_commit);
536536
break;
537537

538538
case PDO_ATTR_DEFAULT_STR_PARAM:
@@ -545,7 +545,7 @@ static int pdo_mysql_get_attribute(pdo_dbh_t *dbh, zend_long attr, zval *return_
545545

546546
case PDO_ATTR_EMULATE_PREPARES:
547547
case PDO_MYSQL_ATTR_DIRECT_QUERY:
548-
ZVAL_LONG(return_value, H->emulate_prepare);
548+
ZVAL_BOOL(return_value, H->emulate_prepare);
549549
break;
550550

551551
#ifndef PDO_USE_MYSQLND
@@ -576,6 +576,10 @@ static int pdo_mysql_get_attribute(pdo_dbh_t *dbh, zend_long attr, zval *return_
576576
}
577577
#endif
578578

579+
case PDO_ATTR_FETCH_TABLE_NAMES:
580+
ZVAL_BOOL(return_value, H->fetch_table_names);
581+
break;
582+
579583
default:
580584
PDO_DBG_RETURN(0);
581585
}

‎ext/pdo_mysql/tests/bug68371.phpt

+7-7
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ $pdo->setAttribute (\PDO::ATTR_ERRMODE, \PDO::ERRMODE_EXCEPTION);
1515

1616
$attrs = array(
1717
// Extensive test: default value and set+get values
18-
PDO::ATTR_EMULATE_PREPARES => array(null, 1, 0),
19-
PDO::MYSQL_ATTR_DIRECT_QUERY => array(null, 0, 1),
18+
PDO::ATTR_EMULATE_PREPARES => array(null, true, false),
19+
PDO::MYSQL_ATTR_DIRECT_QUERY => array(null, false, true),
2020
PDO::MYSQL_ATTR_USE_BUFFERED_QUERY => array(null, false, true),
2121

2222
// Just test the default
@@ -67,16 +67,16 @@ foreach ($attrs as $a => $vals) {
6767

6868
?>
6969
--EXPECTF--
70-
int(1)
70+
bool(true)
7171
OK
7272
OK
73-
int(0)
73+
bool(false)
7474
OK
7575
OK
7676
bool(true)
7777
OK
7878
OK
79-
int(1)
79+
bool(true)
8080
ERR
8181
ERR
8282
int(2)
@@ -93,9 +93,9 @@ array(1) {
9393
[0]=>
9494
string(12) "PDOStatement"
9595
}
96-
ERR
96+
bool(false)
9797
ERR
9898
string(5) "mysql"
99-
ERR
99+
bool(false)
100100
ERR
101101
int(4)

‎ext/pdo_mysql/tests/pdo_mysql___construct_options.phpt

+5-5
Original file line numberDiff line numberDiff line change
@@ -57,15 +57,15 @@ MySQLPDOTest::skip();
5757

5858
$defaults = array(
5959
PDO::ATTR_PERSISTENT => false,
60-
PDO::ATTR_AUTOCOMMIT => 1,
60+
PDO::ATTR_AUTOCOMMIT => true,
6161
/* TODO - why is this a valid option if getAttribute() does not support it?! */
6262
PDO::ATTR_TIMEOUT => false,
63-
PDO::ATTR_EMULATE_PREPARES => 1,
63+
PDO::ATTR_EMULATE_PREPARES => true,
6464
PDO::MYSQL_ATTR_USE_BUFFERED_QUERY => true,
6565
/* TODO getAttribute() does not handle it */
6666
PDO::MYSQL_ATTR_LOCAL_INFILE => false,
6767
/* TODO getAttribute() does not handle it */
68-
PDO::MYSQL_ATTR_DIRECT_QUERY => 1,
68+
PDO::MYSQL_ATTR_DIRECT_QUERY => true,
6969
PDO::MYSQL_ATTR_INIT_COMMAND => '',
7070
);
7171

@@ -154,8 +154,8 @@ MySQLPDOTest::skip();
154154
set_option_and_check(24, PDO::MYSQL_ATTR_INIT_COMMAND, '', 'PDO::MYSQL_ATTR_INIT_COMMAND');
155155
set_option_and_check(25, PDO::MYSQL_ATTR_INIT_COMMAND, 'INSERT INTO nonexistent(invalid) VALUES (1)', 'PDO::MYSQL_ATTR_INIT_COMMAND');
156156

157-
set_option_and_check(33, PDO::MYSQL_ATTR_DIRECT_QUERY, 1, 'PDO::MYSQL_ATTR_DIRECT_QUERY');
158-
set_option_and_check(34, PDO::MYSQL_ATTR_DIRECT_QUERY, 0, 'PDO::MYSQL_ATTR_DIRECT_QUERY');
157+
set_option_and_check(33, PDO::MYSQL_ATTR_DIRECT_QUERY, true, 'PDO::MYSQL_ATTR_DIRECT_QUERY');
158+
set_option_and_check(34, PDO::MYSQL_ATTR_DIRECT_QUERY, false, 'PDO::MYSQL_ATTR_DIRECT_QUERY');
159159

160160
if (defined('PDO::MYSQL_ATTR_LOCAL_INFILE_DIRECTORY')) {
161161
set_option_and_check(35, PDO::MYSQL_ATTR_LOCAL_INFILE_DIRECTORY, null, 'PDO::MYSQL_ATTR_LOCAL_INFILE_DIRECTORY');

‎ext/pdo_mysql/tests/pdo_mysql_attr_autocommit.phpt

+3-3
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ $db = MySQLPDOTest::factory();
1414
$db = MySQLPDOTest::factory();
1515

1616
// autocommit should be on by default
17-
if (1 !== ($tmp = $db->getAttribute(PDO::ATTR_AUTOCOMMIT)))
17+
if (true !== ($tmp = $db->getAttribute(PDO::ATTR_AUTOCOMMIT)))
1818
printf("[001] Expecting int/1 got %s\n", var_export($tmp, true));
1919

2020
// lets see if the server agrees to that
@@ -34,7 +34,7 @@ $db = MySQLPDOTest::factory();
3434
if (!$db->query('SET autocommit = 1'))
3535
printf("[005] Cannot turn on server autocommit mode, %s\n", var_export($db->errorInfo(), true));
3636

37-
if (0 !== ($tmp = $db->getAttribute(PDO::ATTR_AUTOCOMMIT)))
37+
if (false !== ($tmp = $db->getAttribute(PDO::ATTR_AUTOCOMMIT)))
3838
printf("[006] Expecting int/0 got %s\n", var_export($tmp, true));
3939

4040
// off -> on
@@ -48,7 +48,7 @@ $db = MySQLPDOTest::factory();
4848
if (!$row['_autocommit'])
4949
printf("[009] Server autocommit mode should be on, got '%s'\n", var_export($row['_autocommit']));
5050

51-
if (1 !== ($tmp = $db->getAttribute(PDO::ATTR_AUTOCOMMIT)))
51+
if (true !== ($tmp = $db->getAttribute(PDO::ATTR_AUTOCOMMIT)))
5252
printf("[010] Expecting int/1 got %s\n", var_export($tmp, true));
5353

5454
if (MySQLPDOTest::detect_transactional_mysql_engine($db)) {

‎ext/pdo_mysql/tests/pdo_mysql_begintransaction.phpt

+6-6
Original file line numberDiff line numberDiff line change
@@ -17,14 +17,14 @@ if (false == MySQLPDOTest::detect_transactional_mysql_engine($db))
1717

1818
MySQLPDOTest::createTestTable($db, MySQLPDOTest::detect_transactional_mysql_engine($db));
1919

20-
if (1 !== $db->getAttribute(PDO::ATTR_AUTOCOMMIT))
20+
if (true !== $db->getAttribute(PDO::ATTR_AUTOCOMMIT))
2121
printf("[001] Autocommit should be on by default\n");
2222

2323
if (false == $db->beginTransaction())
2424
printf("[002] Cannot start a transaction, [%s] [%s]\n",
2525
$db->errorCode(), implode(' ', $db->errorInfo()));
2626

27-
if (1 !== $db->getAttribute(PDO::ATTR_AUTOCOMMIT))
27+
if (true !== $db->getAttribute(PDO::ATTR_AUTOCOMMIT))
2828
printf("[003] Autocommit should be on by default, beginTransaction() shall not impact it\n");
2929

3030
if (0 == $db->exec('DELETE FROM test'))
@@ -51,7 +51,7 @@ if (false == MySQLPDOTest::detect_transactional_mysql_engine($db))
5151
if (!$db->commit())
5252
printf("[008] [%s] %s\n", $db->errorCode(), implode(' ', $db->errorInfo()));
5353

54-
if (1 !== $db->getAttribute(PDO::ATTR_AUTOCOMMIT))
54+
if (true !== $db->getAttribute(PDO::ATTR_AUTOCOMMIT))
5555
printf("[009] Autocommit should be on after commit()\n");
5656

5757
if (!($stmt = $db->query(sprintf('SELECT id, label FROM test WHERE id = %d', $row['id']))))
@@ -92,7 +92,7 @@ if (false == MySQLPDOTest::detect_transactional_mysql_engine($db))
9292
if (!$db->rollback())
9393
printf("[018] [%s] %s\n", $db->errorCode(), implode(' ', $db->errorInfo()));
9494

95-
if (1 !== $db->getAttribute(PDO::ATTR_AUTOCOMMIT))
95+
if (true !== $db->getAttribute(PDO::ATTR_AUTOCOMMIT))
9696
printf("[019] Autocommit should be on after rollback\n");
9797

9898
if (!($stmt = $db->query(sprintf('SELECT id, label FROM test WHERE id = %d', $row['id']))))
@@ -133,7 +133,7 @@ if (false == MySQLPDOTest::detect_transactional_mysql_engine($db))
133133

134134
// Turn off autocommit using a server variable
135135
$db->exec('SET @@autocommit = 0');
136-
if (1 === $db->getAttribute(PDO::ATTR_AUTOCOMMIT))
136+
if (true === $db->getAttribute(PDO::ATTR_AUTOCOMMIT))
137137
printf("[028] I'm confused, how can autocommit be on? Didn't I say I want to manually control transactions?\n");
138138

139139
if (!$db->beginTransaction())
@@ -156,7 +156,7 @@ if (false == MySQLPDOTest::detect_transactional_mysql_engine($db))
156156
printf("[031] Cannot start a transaction, [%s] [%s]\n",
157157
$db->errorCode(), implode(' ', $db->errorInfo()));
158158

159-
if (1 !== $db->getAttribute(PDO::ATTR_AUTOCOMMIT))
159+
if (true !== $db->getAttribute(PDO::ATTR_AUTOCOMMIT))
160160
printf("[032] Autocommit should be on my default, beginTransaction() should not change that\n");
161161

162162
if (0 == $db->exec('DELETE FROM test'))

‎ext/pdo_mysql/tests/pdo_mysql_commit.phpt

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ if (false == MySQLPDOTest::detect_transactional_mysql_engine($db))
3636

3737
// pdo_transaction_transitions should check this as well...
3838
// ... just to be sure the most basic stuff really works we check it again...
39-
if (1 !== ($tmp = $db->getAttribute(PDO::ATTR_AUTOCOMMIT)))
39+
if (true !== ($tmp = $db->getAttribute(PDO::ATTR_AUTOCOMMIT)))
4040
printf("[003] According to the manual we should be back to autocommit mode, got %s/%s\n",
4141
gettype($tmp), var_export($tmp, true));
4242

‎ext/pdo_mysql/tests/pdo_mysql_get_attribute.phpt

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ if (false == MySQLPDOTest::detect_transactional_mysql_engine($db))
5050
return true;
5151
}
5252

53-
set_and_get(1, $db, PDO::ATTR_AUTOCOMMIT, 1);
53+
set_and_get(1, $db, PDO::ATTR_AUTOCOMMIT, true);
5454
/*
5555
set_and_get(2, $db, PDO::ATTR_AUTOCOMMIT, 0);
5656
set_and_get(3, $db, PDO::ATTR_AUTOCOMMIT, -1);

‎ext/pdo_mysql/tests/pdo_mysql_rollback.phpt

+4-4
Original file line numberDiff line numberDiff line change
@@ -93,8 +93,8 @@ $db->exec('DROP TABLE IF EXISTS test');
9393
$db->exec('DROP TABLE IF EXISTS test2');
9494
?>
9595
--EXPECT--
96-
int(1)
97-
int(0)
98-
int(1)
99-
int(0)
96+
bool(true)
97+
bool(false)
98+
bool(true)
99+
bool(false)
100100
done!

‎ext/pdo_odbc/odbc_driver.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -384,7 +384,7 @@ static int odbc_handle_get_attr(pdo_dbh_t *dbh, zend_long attr, zval *val)
384384
case PDO_ATTR_CONNECTION_STATUS:
385385
break;
386386
case PDO_ODBC_ATTR_ASSUME_UTF8:
387-
ZVAL_BOOL(val, H->assume_utf8 ? 1 : 0);
387+
ZVAL_BOOL(val, H->assume_utf8);
388388
return 1;
389389

390390
}

‎ext/pdo_pgsql/tests/bug68371.phpt

+1-1
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,6 @@ array(1) {
9595
ERR
9696
ERR
9797
string(5) "pgsql"
98-
ERR
98+
bool(true)
9999
ERR
100100
int(4)

0 commit comments

Comments
 (0)