Test updates and bundle 1.3.3

diff --git a/ext/oci8/README b/ext/oci8/README
index cc369f833dd8db44cc2baa83d52324ecd8b4272d..178e456286b942564d53b46845edf4d446054530 100644 (file)
--- a/ext/oci8/README
+++ b/ext/oci8/README
@@ -169,12 +169,11 @@ directory will contain logs of any failures.
 6. DRCP and FAN Support
 -----------------------
 
-The PHP 5.3 OCI8 Beta extension has support for the Oracle Database
-Resident Connection Pool (DRCP) and Fast Application Notification
-(FAN).
+The PHP OCI8 extension has support for the Oracle Database Resident
+Connection Pool (DRCP) and Fast Application Notification (FAN).
 
-This release is for Beta testing only.  Questions and issues can be
-raised on the Oracle OTN forum (free registration required)
+Questions and issues can be raised on the Oracle OTN forum (free
+registration required):
 http://www.oracle.com/technology/forums/php.html
 
 
@@ -184,9 +183,9 @@ The OCI8 extension will compile with Oracle libraries from version
 9iR2 onwards.  However, full functionality (e.g. DRCP support) is only
 available when Oracle 11g is used.
 
-For other, general database functionality, the version of the Oracle
-libraries used by PHP does not necessarily have to match the version
-of the database.
+For general database functionality the version of the Oracle libraries
+used by PHP does not necessarily have to match the version of the
+database.
 
 
 6.2. Database Resident Connection Pooling (DRCP)
@@ -329,16 +328,15 @@ restriction of Oracle Database 11g.
 
 6.4.2 Closing Connections
 
-With the PHP 5.3 OCI8 Beta extension, persistent connections can now
-be closed by the user, allowing greater control over connection
-resource usage.  Persistent connections will now also be closed
-automatically when there is no PHP variable referencing them, such as
-at the end of scope of a PHP user function.  This will rollback any
-uncommitted transaction.  These changes to persistent connections make
-them behave similarly to non-persistent connections, simplifying the
-interface, allowing for greater application consistency and
-predictability.  Use oci8.old_oci_close_semantics=1 to retain the
-historical behavior.
+With the PHP OCI8 extension, persistent connections can now be closed
+by the user, allowing greater control over connection resource usage.
+Persistent connections will now also be closed automatically when
+there is no PHP variable referencing them, such as at the end of scope
+of a PHP user function.  This will rollback any uncommitted
+transaction.  These changes to persistent connections make them behave
+similarly to non-persistent connections, simplifying the interface,
+allowing for greater application consistency and predictability.  Use
+oci8.old_oci_close_semantics=1 to retain the historical behavior.
 
 6.4.3 LOGON Triggers can be used to set session properties
 

diff --git a/ext/oci8/package2.xml b/ext/oci8/package2.xml
index b6c4cde7a8018792093777d0bf1b796d11f8094d..25bf8792483f11426aefa59491ae768ad76e7416 100644 (file)
--- a/ext/oci8/package2.xml
+++ b/ext/oci8/package2.xml
@@ -33,24 +33,21 @@ http://pear.php.net/dtd/package-2.0.xsd">
   <active>yes</active>
  </lead>
 
- <date>2008-04-17</date>
- <time>16:00:00</time>
+ <date>2008-06-19</date>
+ <time>12:00:00</time>
 
  <version>
-  <release>1.3.2</release>
-  <api>1.3.2</api>
+  <release>1.3.3</release>
+  <api>1.3.3</api>
  </version>
  <stability>
-  <release>beta</release>
-  <api>beta</api>
+  <release>stable</release>
+  <api>stable</api>
  </stability>
  <license uri="http://www.php.net/license">PHP</license>
  <notes>
-Refine DRCP connection pooling functionality
-Do scope-end release for oci_pconnect (oci8.old_oci_close_semantics=1 gives old behavior)
-Fixed bug #44372 (compilation with Oracle 10gR1 libraries)
-Fixed PECL bug #12431 (PEAR install using x86_64 RPM builds)
-Allow builds with PHP 4.3.9 onwards
+Changed OCI NLS initialization for consistency and technical correctness
+Removed obsolete macros
  </notes>
  <contents>
   <dir name="/">
@@ -154,6 +151,14 @@ Allow builds with PHP 4.3.9 onwards
     <file name="connect.inc" role="test" />
     <file name="connect_old.phpt" role="test" />
     <file name="connect.phpt" role="test" />
+    <file name="connect_scope1.phpt" role="test" />
+    <file name="connect_scope2.phpt" role="test" />
+    <file name="connect_scope_try1.phpt" role="test" />
+    <file name="connect_scope_try2.phpt" role="test" />
+    <file name="connect_scope_try3.phpt" role="test" />
+    <file name="connect_scope_try4.phpt" role="test" />
+    <file name="connect_scope_try5.phpt" role="test" />
+    <file name="connect_scope_try6.phpt" role="test" />
     <file name="connect_with_charset_001.phpt" role="test" />
     <file name="connect_without_oracle_home_old.phpt" role="test" />
     <file name="connect_without_oracle_home.phpt" role="test" />
@@ -204,7 +209,6 @@ Allow builds with PHP 4.3.9 onwards
     <file name="fetch_all3.phpt" role="test" />
     <file name="fetch_all.phpt" role="test" />
     <file name="fetch_array.phpt" role="test" />
-    <file name="fetch_assoc.php" role="test" />
     <file name="fetch_assoc.phpt" role="test" />
     <file name="fetch_into1.phpt" role="test" />
     <file name="fetch_into2.phpt" role="test" />
@@ -264,6 +268,7 @@ Allow builds with PHP 4.3.9 onwards
     <file name="lob_null.phpt" role="test" />
     <file name="lob_temp1.phpt" role="test" />
     <file name="lob_temp.phpt" role="test" />
+    <file name="minfo.phpt" role="test" />
     <file name="num.phpt" role="test" />
     <file name="oci8safemode.phpt" role="test" />
     <file name="oci_execute_segfault.phpt" role="test" />
@@ -328,6 +333,25 @@ Allow builds with PHP 4.3.9 onwards
  </extsrcrelease>
  <changelog>
 
+<release>
+ <version>
+  <release>1.3.2</release>
+  <api>1.3.2</api>
+ </version>
+ <stability>
+  <release>beta</release>
+  <api>beta</api>
+ </stability>
+ <license uri="http://www.php.net/license">PHP</license>
+ <notes>
+Refine DRCP connection pooling functionality
+Do scope-end release for oci_pconnect (oci8.old_oci_close_semantics=1 gives old behavior)
+Fixed bug #44372 (compilation with Oracle 10gR1 libraries)
+Fixed PECL bug #12431 (PEAR install using x86_64 RPM builds)
+Allow builds with PHP 4.3.9 onwards
+ </notes>
+</release>
+
  <release>
  <version>
   <release>1.3.1</release>

diff --git a/ext/oci8/php_oci8.h b/ext/oci8/php_oci8.h
index b3ac72e71a17e123c8d33a023268fac116091dda..69910e3fde7ce20976f982b2663a3189281adb65 100644 (file)
--- a/ext/oci8/php_oci8.h
+++ b/ext/oci8/php_oci8.h
@@ -46,7 +46,7 @@
  */
 #undef PHP_OCI8_VERSION
 #endif
-#define PHP_OCI8_VERSION "1.3.3-dev"
+#define PHP_OCI8_VERSION "1.3.3"
 
 extern zend_module_entry oci8_module_entry;
 #define phpext_oci8_ptr &oci8_module_entry

diff --git a/ext/oci8/tests/bind_char_1.phpt b/ext/oci8/tests/bind_char_1.phpt
index fd76e607385d8d893f22b9f58180865f54ee88d0..d2dddc9086dee32c4398df81bf27fc2bdb8d212b 100644 (file)
--- a/ext/oci8/tests/bind_char_1.phpt
+++ b/ext/oci8/tests/bind_char_1.phpt
@@ -1,7 +1,15 @@
 --TEST--
 SELECT oci_bind_by_name with SQLT_AFC aka CHAR
 --SKIPIF--
-<?php if (!extension_loaded('oci8')) die ("skip no oci8 extension"); ?>
+<?php
+if (!extension_loaded('oci8')) die ("skip no oci8 extension");
+require(dirname(__FILE__)."/connect.inc");
+$sv = oci_server_version($c);
+$sv = preg_match('/11.1/', $sv, $matches);
+if ($sv !== 1) {
+       die ("skip expected output only valid when using Oracle 11g database");
+}
+?>
 --FILE--
 <?php
 

diff --git a/ext/oci8/tests/bind_char_2.phpt b/ext/oci8/tests/bind_char_2.phpt
index ccf7b5870fd536631ae2c7f8caeef922ba0b5fd4..183c09acb416cb10954ae4d01a9a9c313c33761e 100644 (file)
--- a/ext/oci8/tests/bind_char_2.phpt
+++ b/ext/oci8/tests/bind_char_2.phpt
@@ -1,7 +1,15 @@
 --TEST--
 SELECT oci_bind_by_name with SQLT_AFC aka CHAR and dates
 --SKIPIF--
-<?php if (!extension_loaded('oci8')) die ("skip no oci8 extension"); ?>
+<?php
+if (!extension_loaded('oci8')) die ("skip no oci8 extension");
+require(dirname(__FILE__)."/connect.inc");
+$sv = oci_server_version($c);
+$sv = preg_match('/11.1/', $sv, $matches);
+if ($sv !== 1) {
+       die ("skip expected output only valid when using Oracle 11g database");
+}
+?>
 --FILE--
 <?php
 

diff --git a/ext/oci8/tests/bind_char_3.phpt b/ext/oci8/tests/bind_char_3.phpt
index a339aa94ffb42c3ecbbb5cc69ca327374ee2ba6f..f1e6a93c0c1ed00228acdebfa50600045b908ad0 100644 (file)
--- a/ext/oci8/tests/bind_char_3.phpt
+++ b/ext/oci8/tests/bind_char_3.phpt
@@ -1,7 +1,15 @@
 --TEST--
 PL/SQL oci_bind_by_name with SQLT_AFC aka CHAR to CHAR parameter
 --SKIPIF--
-<?php if (!extension_loaded('oci8')) die ("skip no oci8 extension"); ?>
+<?php
+if (!extension_loaded('oci8')) die ("skip no oci8 extension");
+require(dirname(__FILE__)."/connect.inc");
+$sv = oci_server_version($c);
+$sv = preg_match('/11.1/', $sv, $matches);
+if ($sv !== 1) {
+       die ("skip expected output only valid when using Oracle 11g database");
+}
+?>
 --FILE--
 <?php
 

diff --git a/ext/oci8/tests/bind_char_4.phpt b/ext/oci8/tests/bind_char_4.phpt
index 2357163990e0555887c19f268d719b8dedfb61c3..ddd07dcb29156e0a66bbbb6ccc1b7060f52fc6f3 100644 (file)
--- a/ext/oci8/tests/bind_char_4.phpt
+++ b/ext/oci8/tests/bind_char_4.phpt
@@ -1,7 +1,15 @@
 --TEST--
 PL/SQL oci_bind_by_name with SQLT_AFC aka CHAR to VARCHAR2 parameter
 --SKIPIF--
-<?php if (!extension_loaded('oci8')) die ("skip no oci8 extension"); ?>
+<?php
+if (!extension_loaded('oci8')) die ("skip no oci8 extension");
+require(dirname(__FILE__)."/connect.inc");
+$sv = oci_server_version($c);
+$sv = preg_match('/11.1/', $sv, $matches);
+if ($sv !== 1) {
+       die ("skip expected output only valid when using Oracle 11g database");
+}
+?>
 --FILE--
 <?php
 

diff --git a/ext/oci8/tests/bug27303.phpt b/ext/oci8/tests/bug27303.phpt
index 6f6d01af4ab34e6b7010b510b7f314d0947147d6..2cc63d662d5bd979c7caaddd644ac7995b406afc 100644 (file)
--- a/ext/oci8/tests/bug27303.phpt
+++ b/ext/oci8/tests/bug27303.phpt
@@ -1,7 +1,15 @@
 --TEST--
 Bug #27303 (OCIBindByName binds numeric PHP values as characters)
 --SKIPIF--
-<?php if (!extension_loaded('oci8')) die("skip no oci8 extension"); ?>
+<?php
+if (!extension_loaded('oci8')) die ("skip no oci8 extension");
+require(dirname(__FILE__)."/connect.inc");
+$sv = oci_server_version($c);
+$sv = preg_match('/11.1/', $sv, $matches);
+if ($sv !== 1) {
+       die ("skip expected output only valid when using Oracle 11g database");
+}
+?>
 --FILE--
 <?php
 

diff --git a/ext/oci8/tests/bug27303_2.phpt b/ext/oci8/tests/bug27303_2.phpt
index 3ab90a1a6095a781a574b55678da71136cfccca1..3520b1ea3d20e54730a959ac73ef0238bf19b7cb 100644 (file)
--- a/ext/oci8/tests/bug27303_2.phpt
+++ b/ext/oci8/tests/bug27303_2.phpt
@@ -1,7 +1,15 @@
 --TEST--
 Bug #27303 (OCIBindByName binds numeric PHP values as characters)
 --SKIPIF--
-<?php if (!extension_loaded('oci8')) die("skip no oci8 extension"); ?>
+<?php
+if (!extension_loaded('oci8')) die ("skip no oci8 extension");
+require(dirname(__FILE__)."/connect.inc");
+$sv = oci_server_version($c);
+$sv = preg_match('/11.1/', $sv, $matches);
+if ($sv !== 1) {
+       die ("skip expected output only valid when using Oracle 11g database");
+}
+?>
 --FILE--
 <?php
 

diff --git a/ext/oci8/tests/bug27303_4.phpt b/ext/oci8/tests/bug27303_4.phpt
index 70d6b56880f9085612da7e65ee7b952de0185d0b..f0c960e9396072a1f1b6605c20c957e55e4f589c 100644 (file)
--- a/ext/oci8/tests/bug27303_4.phpt
+++ b/ext/oci8/tests/bug27303_4.phpt
@@ -1,7 +1,15 @@
 --TEST--
 Bug #27303 (OCIBindByName binds numeric PHP values as characters)
 --SKIPIF--
-<?php if (!extension_loaded('oci8')) die("skip no oci8 extension"); ?>
+<?php
+if (!extension_loaded('oci8')) die ("skip no oci8 extension");
+require(dirname(__FILE__)."/connect.inc");
+$sv = oci_server_version($c);
+$sv = preg_match('/11.1/', $sv, $matches);
+if ($sv !== 1) {
+       die ("skip expected output only valid when using Oracle 11g database");
+}
+?>
 --FILE--
 <?php
 

diff --git a/ext/oci8/tests/connect.inc b/ext/oci8/tests/connect.inc
index 452976e8fe4577dfc3ee51e31bc45828d8a3f8e0..8a116290e7e215ad0c38cd0efc55847d87da404e 100644 (file)
--- a/ext/oci8/tests/connect.inc
+++ b/ext/oci8/tests/connect.inc
@@ -2,29 +2,17 @@
 
 include "details.inc";
 
-/* 
- * You should have privileges to create tables in this schema 
- * 
- * */
-/*
-$schema     = "system"; 
-*/
+if (!empty($dbase)) {
+       $c = @oci_connect($user, $password, $dbase);
+}
+else {
+       $c = @oci_connect($user, $password);
+}
 
-$table_name = "tb".substr(str_replace(Array(".", "-"), "_", php_uname("n")), 0, 5);
-$type_name = strtoupper("tp".substr(str_replace(Array(".", "-"), "_", php_uname("n")), 0, 5));
- 
-    if (!empty($dbase)) {
-        $c = ocilogon($user,$password,$dbase);
-    }
-    else {
-        $c = ocilogon($user,$password);
-    }
-
-    if (!empty($schema)) {
-        $schema = $schema.".";
-    }
-    else {
-        $schema = '';
-    }
+if (!$c) {
+       echo "connect.inc: Failed to connect as '$user' to '$dbase'\n";
+       $e = oci_error();
+       echo $e['message']."\n";
+}
 
 ?>

diff --git a/ext/oci8/tests/debug.phpt b/ext/oci8/tests/debug.phpt
index 2910ce3557a8c72fac5435d9a24ea670d5c37de5..deed632eafc1246609be9e598b54be61bf5b7405 100644 (file)
--- a/ext/oci8/tests/debug.phpt
+++ b/ext/oci8/tests/debug.phpt
@@ -1,11 +1,20 @@
 --TEST--
 oci_internal_debug()
 --SKIPIF--
-<?php if (!extension_loaded('oci8')) die("skip no oci8 extension"); ?>
+<?php 
+if (!extension_loaded('oci8')) die("skip no oci8 extension"); 
+ob_start();
+phpinfo(INFO_MODULES);
+$phpinfo = ob_get_clean();
+$iv = preg_match('/Oracle .*Version => 11/', $phpinfo);
+if ($iv !== 1) {
+       die ("skip expected output only valid when using Oracle 11g client libraries");
+}
+?>
 --FILE--
 <?php
 
-require dirname(__FILE__)."/connect.inc";
+require(dirname(__FILE__)."/details.inc");
 
 oci_internal_debug(true);
 
@@ -21,14 +30,32 @@ echo "Done\n";
 ?>
 --EXPECTF--
 OCI8 DEBUG: OCINlsEnvironmentVariableGet at (%s:%d) 
-OCI8 DEBUG L1: Got a cached connection: (%s) at (%s:%d) 
-Done
+OCI8 DEBUG L1: Got NO cached connection at (%s:%d) 
+OCI8 DEBUG: OCIEnvNlsCreate at (%s:%d) 
+OCI8 DEBUG: OCIHandleAlloc at (%s:%d) 
+OCI8 DEBUG: OCIHandleAlloc at (%s:%d) 
+OCI8 DEBUG: OCISessionPoolCreate at (%s:%d) 
+OCI8 DEBUG: OCIAttrSet at (%s:%d) 
+OCI8 DEBUG L1: create_spool: (%s:%d) 
+OCI8 DEBUG L1: using shared pool: (%s:%d) 
+OCI8 DEBUG: OCIHandleAlloc at (%s:%d) 
+OCI8 DEBUG: OCIHandleAlloc at (%s:%d) 
+OCI8 DEBUG: OCIAttrSet at (%s:%d) 
+OCI8 DEBUG: OCIAttrSet at (%s:%d) 
+OCI8 DEBUG: OCIAttrGet at (%s:%d) 
+OCI8 DEBUG: OCIAttrGet at (%s:%d) 
+OCI8 DEBUG L1: (%s:%d) 
+OCI8 DEBUG: OCISessionGet at (%s:%d) 
+OCI8 DEBUG: OCIAttrGet at (%s:%d) 
+OCI8 DEBUG: OCIAttrGet at (%s:%d) 
+OCI8 DEBUG: OCIContextGetValue at (%s:%d) 
+OCI8 DEBUG: OCIContextGetValue at (%s:%d) 
+OCI8 DEBUG: OCIMemoryAlloc at (%s:%d) 
+OCI8 DEBUG: OCIContextSetValue at (%s:%d) 
+OCI8 DEBUG: OCIAttrSet at (%s:%d) 
+OCI8 DEBUG L1: New Non-Persistent Connection address: (%s) at (%s:%d) 
+OCI8 DEBUG L1: num_persistent=(%s:%d) 
 OCI8 DEBUG: OCISessionRelease at (%s:%d) 
 OCI8 DEBUG: OCIHandleFree at (%s:%d) 
 OCI8 DEBUG: OCIHandleFree at (%s:%d) 
-OCI8 DEBUG: OCISessionPoolDestroy at (%s:%d) 
-OCI8 DEBUG: OCIHandleFree at (%s:%d) 
-OCI8 DEBUG: OCIHandleFree at (%s:%d) 
-OCI8 DEBUG: OCIHandleFree at (%s:%d) 
-OCI8 DEBUG: OCIHandleFree at (%s:%d) 
-OCI8 DEBUG: OCIHandleFree at (%s:%d)
+Done

diff --git a/ext/oci8/tests/details.inc b/ext/oci8/tests/details.inc
index 226ab359de8bd6321cc1bb74763da13222856813..9e8377c43c03c5431496158c056e9e6169338d8e 100644 (file)
--- a/ext/oci8/tests/details.inc
+++ b/ext/oci8/tests/details.inc
@@ -33,9 +33,17 @@ if (false !== getenv('PHP_OCI8_TEST_DB')) {
 } else {
        $user                                           = "system";
        $password                                       = "system";
-       $dbase                                          = "oracle";
-       $oracle_on_localhost            = FALSE;
+       $dbase                                          = "localhost/XE";
+       $oracle_on_localhost            = TRUE;
        $test_drcp                                      = FALSE;
 }
 
+/*
+ * Common object names for scripts to use
+ */
+
+$table_name = "tb".substr(str_replace(Array(".", "-"), "_", php_uname("n")), 0, 5);
+$type_name = strtoupper("tp".substr(str_replace(Array(".", "-"), "_", php_uname("n")), 0, 5));
+$schema = '';
+
 ?>