Home » Database » MDB2 » Bug #2055
parser returns useless error
Details
| Submitted | 2004-08-04 16:45 UTC |
|---|---|
| From | worker at mompopmedia dot com |
| Assigned | lsmith |
| Status | Closed |
| Package | MDB2 |
| PHP Version | 5.0.0 |
| OS | Mandrake 10 |
| Roadmaps | (Not assigned) |
Comments
[2004-08-04 16:45 UTC] worker at mompopmedia dot com
Description:
------------
Hi,
When parsing a xml schema file, upon a raised error, all I get is a MDB2_ERROR_MANAGER_PARSE description of the problem. However, in MDB2_Tools_Parser::raiseError() usefull details on the error is collected, but never added to the raised error.
I beleive around line 450, it should be:
$this->error = MDB2_Driver_Common::raiseError(MDB2_ERROR_MANAGER_PARSE, null, null, $error);
However, this does not resolve the issue entirely. Errors raised in other parts of the parser still return a simple "manager schema parse error" which does not say what the error is all about. However, I haven't found the place in the code that would need to be altered to fix this.
Thanks,
Michael
Thanks,
Michael
[2004-08-05 12:17 UTC] smith at backendmedia dot com
I am currently in the process of refactoring the MDB2_Manager. So I can improve the error handling there as I go. However I was not planning on changing the parser in that process. So if you can provide patches for any parser issues it would be good. Also note the places where the Manager doesnt pass on relevant information provided by the Parser. So in general keep it coming :-)
[2004-08-05 16:22 UTC] smith at backendmedia dot com
This bug has been fixed in CVS.
In case this was a documentation problem, the fix will show up at the
end of next Sunday (CET) on pear.php.net.
In case this was a pear.php.net website problem, the change will show
up on the website in short time.
Thank you for the report, and for helping us make PEAR better.