Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 2.5.0b5

Bug #2055 parser returns useless error
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)    
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 : 13 - 2 = ?

 
 [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

Comments

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