MFH: Bug #42173 (INTERVAL and TIMESTAMP type fixes)

diff --git a/ext/oci8/oci8_interface.c b/ext/oci8/oci8_interface.c
index 834cf99ddae308405cf390bb8029a89c2785daf1..f8d791e84edc7d3ac88d9985c5ec1584f4d42ee8 100644 (file)
--- a/ext/oci8/oci8_interface.c
+++ b/ext/oci8/oci8_interface.c
@@ -1183,7 +1183,22 @@ PHP_FUNCTION(oci_field_type)
 #endif
 #ifdef SQLT_TIMESTAMP_TZ
                case SQLT_TIMESTAMP_TZ:
-                       RETVAL_STRING("TIMESTAMP_TZ",1);
+                       RETVAL_STRING("TIMESTAMP WITH TIMEZONE",1);
+                       break;
+#endif
+#ifdef SQLT_TIMESTAMP_LTZ
+               case SQLT_TIMESTAMP_LTZ:
+                       RETVAL_STRING("TIMESTAMP WITH LOCAL TIMEZONE",1);
+                       break;
+#endif
+#ifdef SQLT_INTERVAL_YM
+               case SQLT_INTERVAL_YM:
+                       RETVAL_STRING("INTERVAL YEAR TO MONTH",1);
+                       break;
+#endif
+#ifdef SQLT_INTERVAL_DS
+               case SQLT_INTERVAL_DS:
+                       RETVAL_STRING("INTERVAL DAY TO SECOND",1);
                        break;
 #endif
                case SQLT_DAT:

diff --git a/ext/oci8/oci8_statement.c b/ext/oci8/oci8_statement.c
index fa592bc025d0d25df730b1dc7464d34b6bdd3be8..6b03536772c822f083e6169a2fc2f1e3d6bd654a 100644 (file)
--- a/ext/oci8/oci8_statement.c
+++ b/ext/oci8/oci8_statement.c
@@ -629,6 +629,15 @@ int php_oci_statement_execute(php_oci_statement *statement, ub4 mode TSRMLS_DC)
 #endif
 #ifdef SQLT_TIMESTAMP_TZ
                                                || (outcol->data_type == SQLT_TIMESTAMP_TZ)
+#endif
+#ifdef SQLT_TIMESTAMP_LTZ
+                                               || (outcol->data_type == SQLT_TIMESTAMP_LTZ)
+#endif
+#ifdef SQLT_INTERVAL_YM
+                                               || (outcol->data_type == SQLT_INTERVAL_YM)
+#endif
+#ifdef SQLT_INTERVAL_DS
+                                               || (outcol->data_type == SQLT_INTERVAL_DS)
 #endif
                                                ) {
                                                outcol->storage_size4 = 512; /* XXX this should fit "most" NLS date-formats and Numbers */

diff --git a/ext/oci8/tests/bug42173.phpt b/ext/oci8/tests/bug42173.phpt
new file mode 100644 (file)
index 0000000..501ed75
--- /dev/null
+++ b/ext/oci8/tests/bug42173.phpt
@@ -0,0 +1,168 @@
+--TEST--
+Bug #42173 (TIMESTAMP and INTERVAL query and field functions)
+--SKIPIF--
+<?php if (!extension_loaded('oci8')) die("skip no oci8 extension"); ?>
+--FILE--
+<?php
+
+require(dirname(__FILE__).'/connect.inc');
+
+$stmts = array(
+
+"drop table ts_test",
+
+"create table ts_test (
+c1 TIMESTAMP,
+c2 TIMESTAMP (5),
+c3 TIMESTAMP WITH TIME ZONE,
+c4 TIMESTAMP (2) WITH TIME ZONE,
+c5 TIMESTAMP WITH LOCAL TIME ZONE,
+c6 INTERVAL YEAR TO MONTH,
+c7 INTERVAL YEAR(2) TO MONTH,
+c8 INTERVAL DAY TO SECOND,
+c9 INTERVAL DAY(2) TO SECOND(3)
+)",
+
+"insert into ts_test values (
+timestamp'1999-01-03 10:00:00.123',
+timestamp'1999-01-04 10:00:00.123456',
+timestamp'1999-01-05 10:00:00.123456+1:0',
+timestamp'1999-01-06 10:00:00.123456-1:0',
+timestamp'1999-01-06 10:00:00.123456-1:0',
+interval'1-2' year to month,
+interval'10-4' year to month,
+interval'1 2:20:20.123' day to second,
+interval'1 2:20:20.12345' day to second)");
+
+foreach ($stmts as $sql) {
+       $s = oci_parse($c, $sql);
+       $r = @oci_execute($s);
+}
+
+$s = oci_parse($c, "select * from ts_test");
+$r = oci_execute($s);
+$row = oci_fetch_array($s, OCI_ASSOC);
+var_dump($row);
+
+foreach ($row as $name => $field) {
+       echo "\nColumn $name\n";
+       var_dump(oci_field_is_null($s, $name));
+       var_dump(oci_field_name($s, $name));
+       var_dump(oci_field_type($s, $name));
+       var_dump(oci_field_type_raw($s, $name));
+       var_dump(oci_field_scale($s, $name));
+       var_dump(oci_field_precision($s, $name));
+       var_dump(oci_field_size($s, $name));
+}
+
+// Cleanup
+
+$s = oci_parse($c, "drop table ts_test");
+$r = @oci_execute($s);
+
+echo "Done\n";
+
+?>
+--EXPECTF--
+array(9) {
+  ["C1"]=>
+  string(28) "03-JAN-99 10.00.00.123000 AM"
+  ["C2"]=>
+  string(27) "04-JAN-99 10.00.00.12346 AM"
+  ["C3"]=>
+  string(35) "05-JAN-99 10.00.00.123456 AM +01:00"
+  ["C4"]=>
+  string(31) "06-JAN-99 10.00.00.12 AM -01:00"
+  ["C5"]=>
+  string(28) "%s"
+  ["C6"]=>
+  string(6) "+01-02"
+  ["C7"]=>
+  string(6) "+10-04"
+  ["C8"]=>
+  string(19) "+01 02:20:20.123000"
+  ["C9"]=>
+  string(16) "+01 02:20:20.123"
+}
+
+Column C1
+bool(false)
+string(2) "C1"
+string(9) "TIMESTAMP"
+int(187)
+int(6)
+int(0)
+int(11)
+
+Column C2
+bool(false)
+string(2) "C2"
+string(9) "TIMESTAMP"
+int(187)
+int(5)
+int(0)
+int(11)
+
+Column C3
+bool(false)
+string(2) "C3"
+string(23) "TIMESTAMP WITH TIMEZONE"
+int(188)
+int(6)
+int(0)
+int(13)
+
+Column C4
+bool(false)
+string(2) "C4"
+string(23) "TIMESTAMP WITH TIMEZONE"
+int(188)
+int(2)
+int(0)
+int(13)
+
+Column C5
+bool(false)
+string(2) "C5"
+string(29) "TIMESTAMP WITH LOCAL TIMEZONE"
+int(232)
+int(6)
+int(0)
+int(11)
+
+Column C6
+bool(false)
+string(2) "C6"
+string(22) "INTERVAL YEAR TO MONTH"
+int(189)
+int(0)
+int(2)
+int(5)
+
+Column C7
+bool(false)
+string(2) "C7"
+string(22) "INTERVAL YEAR TO MONTH"
+int(189)
+int(0)
+int(2)
+int(5)
+
+Column C8
+bool(false)
+string(2) "C8"
+string(22) "INTERVAL DAY TO SECOND"
+int(190)
+int(6)
+int(2)
+int(11)
+
+Column C9
+bool(false)
+string(2) "C9"
+string(22) "INTERVAL DAY TO SECOND"
+int(190)
+int(3)
+int(2)
+int(11)
+Done