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

Bug #10275 [Patch] XML-RPC usage in Channels.php
Submitted: 2007-03-05 16:41 UTC Modified: 2007-03-05 23:19 UTC
From: tias Assigned: tias
Status: Closed Package: PEAR (version 1.5.1)
PHP Version: 5.2.0 OS: Linux
Roadmaps: 1.5.2    
Subscription  


 [2007-03-05 16:41 UTC] tias (Tias Guns)
Description: ------------ Hello, In Command/Channels.php: The command 'update-channels' still used XML-RPC, which gave errors. The command 'channel-update <channel>' did not have this problem. After looking at the update-channels code, and talking to cellog on IRC, I decided to replace all the doUpdateAll code, with a call to doUpdate for every channel. While doing this, I made the output of doUpdate a bit more human (= identical to the output doUpdateAll used to give) Test script: --------------- The patch can be found at: http://tias.ulyssis.org/patch/pear/Channels_1.5.1_noXMLRPC.patch Expected result: ---------------- [tias@katana]bin ./pear update-channels Updating channel "pear.php.net" Channel "pear.php.net" is up to date Updating channel "pecl.php.net" Channel "pecl.php.net" is up to date Actual result: -------------- [tias@katana]bin ./pear update-channels Warning: fopen(XML/RPC.php): failed to open stream: No such file or directory in Remote.php on line 187 Warning: fopen(XML/RPC.php): failed to open stream: No such file or directory in /home/tias/WASP/pear/inst_cmd4/PEAR/PEAR/Remote.php on line 187 For this remote PEAR operation you need to load the xmlrpc extension or install XML_RPC

Comments

 [2007-03-05 23:17 UTC] cellog (Greg Beaver)
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. I updated the patch, there was no accounting for PEAR_Errors returned, and success has 2 s's at the end. Otherwise, nice work
 [2007-03-05 23:19 UTC] cellog (Greg Beaver)
give credit where due