Fixed #71062 pg_convert() doesn't accept ISO 8601 for datatype timestamp

This commit is contained in:
Anatol Belski 2016-04-15 13:38:49 +02:00
parent 163d26d584
commit d263616525
2 changed files with 40 additions and 1 deletions

View File

@ -6183,7 +6183,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn *pg_link, const char *table_name, con
ZVAL_STRINGL(&new_val, "NOW()", sizeof("NOW()")-1);
} else {
/* better regex? */
if (php_pgsql_convert_match(Z_STRVAL_P(val), Z_STRLEN_P(val), "^([0-9]{4}[/-][0-9]{1,2}[/-][0-9]{1,2})([ \\t]+(([0-9]{1,2}:[0-9]{1,2}){1}(:[0-9]{1,2}){0,1}(\\.[0-9]+){0,1}([ \\t]*([+-][0-9]{1,4}(:[0-9]{1,2}){0,1}|[-a-zA-Z_/+]{1,50})){0,1})){0,1}$", 1) == FAILURE) {
if (php_pgsql_convert_match(Z_STRVAL_P(val), Z_STRLEN_P(val), "^([0-9]{4}[/-][0-9]{1,2}[/-][0-9]{1,2})(([ \\t]+|T)(([0-9]{1,2}:[0-9]{1,2}){1}(:[0-9]{1,2}){0,1}(\\.[0-9]+){0,1}([ \\t]*([+-][0-9]{1,4}(:[0-9]{1,2}){0,1}|[-a-zA-Z_/+]{1,50})){0,1})){0,1}$", 1) == FAILURE) {
err = 1;
} else {
ZVAL_STRING(&new_val, Z_STRVAL_P(val));

View File

@ -0,0 +1,39 @@
--TEST--
Bug #71062 pg_convert() doesn't accept ISO 8601 for datatype timestamp
--SKIPIF--
<?php include("skipif.inc"); ?>
--FILE--
<?php
include('config.inc');
$db = pg_connect($conn_str);
$table = "public.test_table_bug71062_bug71062";
pg_query($db, "CREATE TABLE $table ( test_field TIMESTAMPTZ )");
// ISO 8601 (with 'T' between date and time)
$date_string_php_iso8601 = date_create('8 Dec 2015 5:38')->format(DateTime::ISO8601);
// ISO 8601 with the 'T' removed
$modified_format = 'Y-m-d H:i:sO';
$date_string_modified_iso8601 = date_create('8 Dec 2015 5:38')->format($modified_format);
printf("trying format %s \n", DateTime::ISO8601);
pg_convert($db, $table, ['test_field' => $date_string_php_iso8601]);
printf("trying format %s \n", $modified_format);
pg_convert($db, $table, ['test_field' => $date_string_modified_iso8601]);
print "done\n";
pg_query($db, "DROP TABLE $table");
?>
==OK==
--EXPECT--
trying format Y-m-d\TH:i:sO
trying format Y-m-d H:i:sO
done
==OK==