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

Request #17421 Update API method needs new args
Submitted: 2010-05-20 23:47 UTC
From: meltingice Assigned: izi
Status: Closed Package: Services_Twitter (version 0.5.1)
PHP Version: 5.3.2 OS: FreeBSD 8.0
Roadmaps: (Not assigned)    
Subscription  


 [2010-05-20 23:47 UTC] meltingice (Ryan LeFevre)
Description: ------------ Twitter's updated "update" API call is missing newly added arguments including lat, long, place_id and display_coordinates. Test script: --------------- <? $twitter->statuses->update(array("status"=>$msg, "lat"=>$lat, "long"=>$long)); ?> Expected result: ---------------- XML response should include $resp->place as an object containing information about the lat/long pair sent to Twitter. Actual result: -------------- lat/long ignored

Comments

 [2010-05-20 23:50 UTC] meltingice (Ryan LeFevre)
 [2010-05-22 13:40 UTC] izi (David Jean Louis)
-Status: Open +Status: Closed -Assigned To: +Assigned To: izi
This bug has been fixed in SVN. 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. Hi, I've fixed this in the trunk, you can view the changes here : http://svn.php.net/viewvc?view=revision&revision=299620 I took your patch as a base but changed the float type to lat/long types for better validation in Twitter.php, I have also added tests so that the code coverage stays at 100%. Thanks, -- David