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

Bug #15750 Unreachable code in PEAR_Downloader
Submitted: 2009-01-24 04:43 UTC
From: doconnor Assigned: dufuz
Status: Closed Package: PEAR (version CVS)
PHP Version: 5.2.6 OS:
Roadmaps: 1.8.0alpha1    
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:
Solve the problem : 32 - 27 = ?

 [2009-01-24 04:43 UTC] doconnor (Daniel O'Connor)
Description: ------------ PEAR_Downloader::_getPackageDownloadURL() has old code which can't be executed. This shows up in the PEAR_Downloader_Package tests. Code: if ($chan->supportsREST($preferred_mirror) && (($base2 = $chan->getBaseURL('REST1.3', $preferred_mirror)) || ($base = $chan->getBaseURL('REST1.0', $preferred_mirror))) ) { // SNIP! return $url; } else { return $this->raiseError($parr['channel'] . ' is using a unsupported protocol - This should never happen.'); } // Oh look, more code here! Test script: --------------- 1. Apply unit test patch 2. Run unit tests Expected result: ---------------- It works Actual result: -------------- It doesn't


 [2009-01-24 04:49 UTC] doconnor (Daniel O'Connor)
 [2009-01-24 15:59 UTC] dufuz (Helgi Þormar Þorbjörnsson)
Killed the dead code, also in downloadDep
 [2009-01-30 23:44 UTC] dufuz (Helgi Þormar Þorbjörnsson)
This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on by the end of next Sunday (CET). If this was a problem with the 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.