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

Bug #13095 destDir does not exist - no error returned
Submitted: 2008-02-10 02:37 UTC
From: till Assigned: cweiske
Status: Closed Package: HTTP_Upload (version 0.9.1)
PHP Version: 5.2.0 OS:
Roadmaps: (Not assigned)    
Subscription  


 [2008-02-10 02:37 UTC] till (Till Klampaeckel)
Description: ------------ I copied the example code, ran it - no error. I am not sure why since move_uploaded_file should return "false" in that case and it should trigger an error. Just wanted to add this. Btw, check out my comment on this bug (in regard to your package): http://pear.php.net/bugs/bug.php?id=12946

Comments

 [2008-10-22 13:26 UTC] doconnor (Daniel O'Connor)
Psst, till, unit test / executable test case?
 [2008-10-22 14:27 UTC] till (Till Klampaeckel)
PSSST! Example code, is example code, is example code. :P http://pear.php.net/manual/en/package.http.http-upload.php I checked cvs.php.net: http://cvs.php.net/viewvc.cgi/pear/HTTP_Upload/Upload.php?revision=1.61&view=markup Line 888, looks like the issue is fixed? (That is if I still understand my 8 months old bug report. ;-)) I would not recommend silencing move_uploaded_file() there. PHP has a very comprehensive error reporting mechanism and people should be able to pipe the warning into a log and also use it to debug their app.
 [2009-03-10 16:27 UTC] kguest (Ken Guest)
this appears to be fixed. tried the sample code myself; got the following message: "Upload error: The destination directory doesn't exist or is a regular file" - created the uploads directory and it just worked (tm) (got a "File was moved to" message instead).
 [2009-08-16 14:32 UTC] cweiske (Christian Weiske)
-Status: Open +Status: Closed -Assigned To: +Assigned To: cweiske
Thank you for your bug report. This issue has been fixed in the latest released version of the package, which you can download at http://pear.php.net/get/