if a php bolean makes it as far as the parameter callback, it must really need

to be boolean; express is as native pgsql 't' or 'f'.

Add a test case for Bug #33876, which is a partially bogus bug.

1 files changed, 150 insertions, 0 deletions

diff --git a/ext/pdo_pgsql/tests/bug_33876.phpt b/ext/pdo_pgsql/tests/bug_33876.phpt
new file mode 100644
index 0000000000..ea10622786
--- /dev/null
+++ b/ext/pdo_pgsql/tests/bug_33876.phpt
@@ -0,0 +1,150 @@
+--TEST--
+PDO PgSQL Bug #33876
+--SKIPIF--
+<?php
+if (!extension_loaded('pdo') || !extension_loaded('pdo_pgsql')) die('skip not loaded');
+require 'ext/pdo_pgsql/tests/config.inc';
+require 'ext/pdo/tests/pdo_test.inc';
+PDOTest::skip();
+?>
+--FILE--
+<?php
+require 'ext/pdo/tests/pdo_test.inc';
+$db = PDOTest::test_factory('ext/pdo_pgsql/tests/common.phpt');
+$db->setAttribute(PDO_ATTR_ERRMODE, PDO_ERRMODE_SILENT);
+
+$db->exec('CREATE TABLE test (foo varchar(5) NOT NULL, bar bool NOT NULL)');
+$db->exec("INSERT INTO test VALUES('false','f')");
+$db->exec("INSERT INTO test VALUES('true', 't')");
+
+$res = $db->prepare('SELECT foo from test where bar = ?');
+
+# this is the portable approach to binding a bool
+$res->bindValue(1, false, PDO_PARAM_BOOL);
+if (!$res->execute())
+	print_r($res->errorInfo());
+else
+	print_r($res->fetchAll(PDO_FETCH_ASSOC));
+
+# this is the portable approach to binding a bool
+$res->bindValue(1, true, PDO_PARAM_BOOL);
+if (!$res->execute())
+	print_r($res->errorInfo());
+else
+	print_r($res->fetchAll(PDO_FETCH_ASSOC));
+
+
+# true gets cast to string (because the implied default is string)
+# true-as-string is 1, so this "works"
+if (!$res->execute(array(true)))
+	print_r($res->errorInfo());
+else
+	print_r($res->fetchAll(PDO_FETCH_ASSOC));
+
+# Expected to fail; unless told otherwise, PDO assumes string inputs
+# false -> "" as string, which pgsql doesn't like
+if (!$res->execute(array(false)))
+	print_r($res->errorInfo());
+else
+	print_r($res->fetchAll(PDO_FETCH_ASSOC));
+
+# And now using emulator prepares
+echo "EMUL\n";
+
+
+$res = $db->prepare('SELECT foo from test where bar = ?', array(
+	PDO_PGSQL_ATTR_DISABLE_NATIVE_PREPARED_STATEMENT => true));
+
+# this is the portable approach to binding a bool
+$res->bindValue(1, false, PDO_PARAM_BOOL);
+if (!$res->execute())
+	print_r($res->errorInfo());
+else
+	print_r($res->fetchAll(PDO_FETCH_ASSOC));
+
+# this is the portable approach to binding a bool
+$res->bindValue(1, true, PDO_PARAM_BOOL);
+if (!$res->execute())
+	print_r($res->errorInfo());
+else
+	print_r($res->fetchAll(PDO_FETCH_ASSOC));
+
+
+# true gets cast to string (because the implied default is string)
+# true-as-string is 1, so this "works"
+if (!$res->execute(array(true)))
+	print_r($res->errorInfo());
+else
+	print_r($res->fetchAll(PDO_FETCH_ASSOC));
+
+# Expected to fail; unless told otherwise, PDO assumes string inputs
+# false -> "" as string, which pgsql doesn't like
+if (!$res->execute(array(false)))
+	print_r($res->errorInfo());
+else
+	print_r($res->fetchAll(PDO_FETCH_ASSOC));
+
+
+
+--EXPECT--
+Array
+(
+    [0] => Array
+        (
+            [foo] => false
+        )
+
+)
+Array
+(
+    [0] => Array
+        (
+            [foo] => true
+        )
+
+)
+Array
+(
+    [0] => Array
+        (
+            [foo] => true
+        )
+
+)
+Array
+(
+    [0] => 22P02
+    [1] => 7
+    [2] => ERROR:  invalid input syntax for type boolean: ""
+)
+EMUL
+Array
+(
+    [0] => Array
+        (
+            [foo] => false
+        )
+
+)
+Array
+(
+    [0] => Array
+        (
+            [foo] => true
+        )
+
+)
+Array
+(
+    [0] => Array
+        (
+            [foo] => true
+        )
+
+)
+Array
+(
+    [0] => 22P02
+    [1] => 7
+    [2] => ERROR:  invalid input syntax for type boolean: ""
+)