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

Request #10637 HTTP_WebDav_Server is throwing E_NOTICE's
Submitted: 2007-04-06 15:32 UTC
From: jorrit Assigned: jorrit
Status: Closed Package: HTTP_WebDAV_Server (version 1.0.0RC4)
PHP Version: Irrelevant OS: Irrelevant
Roadmaps: (Not assigned)    
Subscription  


 [2007-04-06 15:32 UTC] jorrit (Jorrit Schippers)
Description: ------------ HTTP_WebDav_Server is throwing E_NOTICE's at some points in the code. These are masked by @'s, but masking notices with @'s does not prevent custom error handlers to be called. It would be nice if there would be real notice avoidance instead of masking notices with @'s.

Comments

 [2007-04-06 21:49 UTC] hholzgra (Hartmut Holzgraefe)
I'll try to catch more error conditions ahead of time, i don't think removing the actual @ signs in front of the file system functions is a good idea though as any uncaught warnings will screw up the XML responses or maybe i could just disable display_errors temporarily during webdav method execution, i'll look into that, too
 [2007-04-06 21:57 UTC] hholzgra (Hartmut Holzgraefe)
just found that i have display_errors disabled already, so i think i can close this. all obvious warning cases should be handled in a more graceful way now, and for whatever remains uncaught calling a user error handler probably makes sense
 [2007-04-06 22:13 UTC] jorrit (Jorrit Schippers)
I meant instances of @ usage at for instance line 146. I am willing to make a patch at some point, if it would be considered to be used.
 [2007-04-06 22:19 UTC] hholzgra (Hartmut Holzgraefe)
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. http://news.php.net/php.pear.cvs/45729 should have fixed all of this
 [2007-04-06 22:35 UTC] jorrit (Jorrit Schippers)
You are very quick, thanks. I'll keep an eye on the development. You have made a typo on line 1608 and line 1611 (uset->user)