http://pear.php.net/bugs/8734 [Closed] Wrong use of list() en-us pear-webmaster@lists.php.net pear-webmaster@lists.php.net hourly 1 2000-01-01T12:00+00:00 http://pear.php.net/bugs/8734 Translation2 Bug Reported by chrismontijn@... 2006-09-19T10:07:49-00:00 PHP: 4.4.2 OS: Windows XP Package Version: 2.0.0beta11 Description: ------------ When using an existing db connection, with fetchmode set to associative, to instantiate an translation2 object at least one (didn't check further) list() function is not working properly. For example in the MDB2.php container on line 217 in function getPage() there is a list() not working properly, because the indices of the result array are tablecolumnnames instead of numbers (starting from zero). Test script: --------------- $dsn = 'somedsn'; $db =& MDB2::connect($dsn); // !This is where it goes wrong $db->setFetchmode(MDB2_FETCHMODE_ASSOC); // instantiate translation2 with this database connection $tr =& Translation2::factory($dbdriver, &$db, $TR_options); $tr->setLang('somelang'); $tr =& $tr->getDecorator('CacheMemory'); $tr->setPageID('someGroup'); $tr->get('somestr'); Expected result: ---------------- I expect to see no notices about undefined offsets. Actual result: -------------- When display notices are on in php.ini page shows following errors: Notice: Undefined offset: 1 in c:\Inetpub\wwwroot\pear\Translation2\Container\mdb2.php on line 217 Notice: Undefined offset: 0 in c:\Inetpub\wwwroot\pear\Translation2\Container\mdb2.php on line 217]]> Translation2 Bug Reported by chrismontijn@... 2006-09-19T10:07:49-00:00 PHP: 4.4.2 OS: Windows XP Package Version: 2.0.0beta11 Description: ------------ When using an existing db connection, with fetchmode set to associative, to instantiate an translation2 object at least one (didn't check further) list() function is not working properly. For example in the MDB2.php container on line 217 in function getPage() there is a list() not working properly, because the indices of the result array are tablecolumnnames instead of numbers (starting from zero). Test script: --------------- $dsn = 'somedsn'; $db =& MDB2::connect($dsn); // !This is where it goes wrong $db->setFetchmode(MDB2_FETCHMODE_ASSOC); // instantiate translation2 with this database connection $tr =& Translation2::factory($dbdriver, &$db, $TR_options); $tr->setLang('somelang'); $tr =& $tr->getDecorator('CacheMemory'); $tr->setPageID('someGroup'); $tr->get('somestr'); Expected result: ---------------- I expect to see no notices about undefined offsets. Actual result: -------------- When display notices are on in php.ini page shows following errors: Notice: Undefined offset: 1 in c:\Inetpub\wwwroot\pear\Translation2\Container\mdb2.php on line 217 Notice: Undefined offset: 0 in c:\Inetpub\wwwroot\pear\Translation2\Container\mdb2.php on line 217]]> 2006-09-19T10:07:49-00:00 http://pear.php.net/bugs/8734#1158662088 This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.]]> This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.]]> 2006-09-19T10:34:48-00:00 http://pear.php.net/bugs/8734#1158661574 A simple solution is to change line 217 in container/mdb2.php while (list($key, $value) = $res->fetchRow()) { to while (list($key, $value) = $res->fetchRow(MDB2_FETCHMODE_ORDERED)) {]]> A simple solution is to change line 217 in container/mdb2.php while (list($key, $value) = $res->fetchRow()) { to while (list($key, $value) = $res->fetchRow(MDB2_FETCHMODE_ORDERED)) {]]> 2006-09-19T10:26:14-00:00