#25126 [Csd]: DB::getAssoc() does not use default fetch mode

ID: 25126 Updated by: danielc@php.net Reported By: sjaensch at gmx dot net Status: Closed Bug Type: PEAR related Operating System: Linux PHP Version: 4.3.2 Assigned To: cox New Comment: The versions of the files you're using are dated 2003/06/20 and 2003/06/21. The bug was fixed on 2003/08/28. So, of course it's not fixed in the version you're using. There have been significant fixes and enhancements to DB since then. The current version is 1.6.2. Update your installation. Previous Comments: ------------------------------------------------------------------------ [2004-04-19 17:53:43] cipri@php.net > $Id: common.php,v 1.21 2003/06/20 20:41:30 cox Exp $ > $Id: mysql.php,v 1.19 2003/06/21 16:45:26 cox Exp $ Those seem to be quite old files. mysql.php is currently at 1.71, in the DB-1.6.2 package. You should try running "pear upgrade DB" to install the latest version of the PEAR::DB package, and see if that helps. While you're at it, you might like to update the rest of PEAR aswell (running "pear upgrade-all" will do that), since the PEAR shipped with your SUSE seems to be quite outdated :) ------------------------------------------------------------------------ [2004-04-19 17:42:10] Hubert dot Christiaen at pandora dot be This problem is not solved. I had code using the setFetchMode(DB_FETCHMODE_ASSOC) in a previous version (2001, $Id: DB.php,v 1.73.2.2 2001/11/13 01:26:39 and $Id: mysql.php,v 1.69.2.3 2001/11/13 01:26:42 ) and it worked correctly. Now in the version shipped with SuSE Linux 9.0 (version from 6/2004: $Id: common.php,v 1.21 2003/06/20 20:41:30 cox Exp $ and: $Id: mysql.php,v 1.19 2003/06/21 16:45:26 cox Exp $ it does not work correctly. Putting some tracing in mysql.php I found that $this->fetchmode == 2 (which is correct) but that $fetchmode == 1 (which is the default DB_FETCHMODE_ORDERED). So it seems that setFetchMode works correctly but that the new value is not passed to the method at execution time. ------------------------------------------------------------------------ [2003-08-28 09:31:21] cox@php.net This bug has been fixed in CVS. In case this was a PHP problem, snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. In case this was a documentation problem, the fix will show up soon at http://www.php.net/manual/. In case this was a PHP.net website problem, the change will show up on the PHP.net site and on the mirror sites in short time. Thank you for the report, and for helping us make PHP better. ------------------------------------------------------------------------ [2003-08-18 06:40:09] arnaud@php.net Tomas is on vacation. He will take care of it when he returns. ------------------------------------------------------------------------ [2003-08-18 06:22:01] nicos@php.net Here is a patch that should fix the issues. I need feedbacks... Index: DB/common.php =================================================================== RCS file: /repository/pear/DB/DB/common.php,v retrieving revision 1.22 diff -u -u -r1.22 common.php --- DB/common.php 22 Jul 2003 21:54:16 -0000 1.22 +++ DB/common.php 18 Aug 2003 11:20:54 -0000 @@ -1034,7 +1034,7 @@ */ function &getAssoc($query, $force_array = false, $params = array(), - $fetchmode = DB_FETCHMODE_ORDERED, $group = false) + $fetchmode = 'default', $group = false) { settype($params, "array"); if (sizeof($params) > 0) { @@ -1049,6 +1049,9 @@ } else { $res = $this->query($query); } + if ($fetchmode == 'default') { + $fetchmode = $this->fetchmode; + } if (DB::isError($res)) { return $res; Note: I also have a patch that fixes the oci8 problem #25067 ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/25126 -- Edit this bug report at http://bugs.php.net/?id=25126&edit=1