Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 2.5.0b5
Bug #5791 extended->getBeforeID() and ->getAfterID() don't allow $field to be empty
Submitted: 2005-10-27 10:31 UTC
From: tschlottke at virtualminds dot de Assigned: lsmith
Status: Closed Package: MDB2
PHP Version: Irrelevant OS: irrelevant
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 17 - 14 = ?

 
 [2005-10-27 10:31 UTC] tschlottke at virtualminds dot de
Description: ------------ Earlier versions of MDB2 (in which the two methods where not in the extended module), it was possible to leave $field blank, which is, looking at the code a possible/wanted thing. ... "$seq = $table.(empty($field) ? '' : '_'.$field);" ... Please change the Methods to function getBeforeID($table, $field='', $ondemand = true) and function getAfterID($id, $table, $field='') because php throws warnings otherwise. Actual result: -------------- PHP Warning: Missing argument 2 for MDB2_Extended::getBeforeID() in /usr/share/php/MDB2/Extended.php on line 562

Comments

 [2005-10-27 16:28 UTC] lsmith
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.