Re: php-5.3.3/ext/pdo_oci/tests/pdo_oci_attr_client test fails as of unexpected deprecated output

From: Date: Mon, 29 Nov 2010 06:54:25 +0000
Subject: Re: php-5.3.3/ext/pdo_oci/tests/pdo_oci_attr_client test fails as of unexpected deprecated output
References: 1  Groups: php.qa 
Request: Send a blank email to php-qa+get-65680@lists.php.net to get a copy of this message
Following Daniel Convissor suggestion, suggested patch uses explode instead of the deprecated split: Index: ext/pdo_oci/tests/pdo_oci_attr_client.phpt ===================================================================
--- ext/pdo_oci/tests/pdo_oci_attr_client.phpt    (revision 305821)
+++ ext/pdo_oci/tests/pdo_oci_attr_client.phpt    (working copy)
@@ -17,7 +17,7 @@ $cv = $dbh->getAttribute(PDO::ATTR_CLIENT_VERSION); var_dump($cv); -$s = split("\.", $cv); +$s = explode(".", $cv); if ($s[0] >= 10 && count($s) > 1 && $s[1] >= 2) {
     if (count($s) != 5) {
         echo "Wrong number of values in array\nVersion was: ";
thx, Eyal On 11/08/2010 04:21 PM, eyal.t wrote:
Hi, The test pdo_oci_attr_client under php-5.3.3/ext/pdo_oci/tests seems to fail as of the following unexpected output: / Deprecated: Function split() is deprecated in /var/www/php-5.3.3/ext/pdo_oci/tests/pdo_oci_attr_client.php on line 14/ Suggest to add to the --EXPECTF-- section such line as follows: Index: /home/eyal/php-5.3.3/ext/pdo_oci/tests/pdo_oci_attr_client.phpt ===================================================================
--- /home/eyal/php-5.3.3/ext/pdo_oci/tests/pdo_oci_attr_client.phpt    (revision 305201)
+++ /home/eyal/php-5.3.3/ext/pdo_oci/tests/pdo_oci_attr_client.phpt    (working copy)
@@ -39,5 +39,6 @@ ?> --EXPECTF-- ATTR_CLIENT_VERSION: string(%d) "%d.%s" +Deprecated: Function split() is deprecated in %s on line %d Version OK, so far as can be portably checked Done if there are no objections, will commit this change. thx, Eyal


« previous php.qa (#65680) next »