| 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) | ||
[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.