Re: Re: php-5.3.3/ext/pdo_oci/tests/pdo_oci_attr_client test fails as of unexpected deprecated output
| From: | Patrick ALLAERT | Date: | Mon, 29 Nov 2010 13:22:29 +0000 |
| Subject: | Re: Re: php-5.3.3/ext/pdo_oci/tests/pdo_oci_attr_client test fails as of unexpected deprecated output | ||
| References: | 1 2 | Groups: | php.qa |
| Request: | Send a blank email to php-qa+get-65681@lists.php.net to get a copy of this message | ||
2010/11/29 Eyal Teutsch <eyal.t@zend.com>:
> 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
Looks good, did you tested your change and does it pass?
In such a case you can commit it! (Given that you received commit
access to the tests).
Regards,
--
Patrick Allaert
---
http://code.google.com/p/peclapm/ - Alternative PHP
Monitor