| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.5.0b4 |
| Bug #13481 | getTableConstraintDefinition() may return incomplete results | ||
|---|---|---|---|
| Submitted: | 2008-03-24 10:53 UTC | ||
| From: | hschletz | Assigned: | hschletz |
| Status: | Closed | Package: | MDB2_Driver_pgsql (version 1.5.0b1) |
| PHP Version: | Irrelevant | OS: | |
| Roadmaps: | (Not assigned) | ||
[2008-03-24 10:53 UTC] hschletz
(Holger Schletz)
Description:
------------
Under certain circumstances, getTableConstraintDefinition() uses the wrong constraint name for SQL queries, resulting in missing fields in the returned array. The name is determined correctly for the first query, but overwritten for subsequent queries.
The attached patch re-uses the constraint name once it is determined.
Comments
[2008-03-24 13:27 UTC] afz
(Ali Fazelzadeh)
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.
[2008-05-10 18:23 UTC] hschletz
(Holger Schletz)
It's not fixed yet in CVS. The commit you are referring to claims to fix #13481, but actually addresses a different issue.
I also tested the latest CVS version and the problem still persists.
[2008-05-13 13:38 UTC] quipo
(Lorenzo Alberton)
You're right, the fix is totally wrong. I'll revert and apply your patch instead.
[2008-05-13 13:38 UTC] quipo
(Lorenzo Alberton)
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.