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

Bug #16499 signature implementation mismatch
Submitted: 2009-08-08 16:51 UTC
From: papayasoft Assigned: shupp
Status: Wont fix Package: Services_Digg (version 0.4.7)
PHP Version: 5.2.5 OS: WinXP
Roadmaps: (Not assigned)    
Subscription  


 [2009-08-08 16:51 UTC] papayasoft (David Weinraub)
Description: ------------ The class Services_Digg_Story has several methods that accept a $story parameter, presumably a story id. But the implementation of thoe methods uses $this->id, not the passed $story id. Must be a typo. Test script: --------------- require_once('Services/Digg.php'); Services_Digg::$appKey = 'http://example.com/'; Services_Digg::$uri = 'http://services.digg.com'; $request = Services_Digg::factory('Story')->getDiggActivity(8507815); Expected result: ---------------- The requested story exists, so $request should be a valid instance of DiggAPIActivity. Actual result: -------------- Results in a 404 Exception as the script attempts to access the endpoint url http://services.digg.com/story//activity/diggs Note the blank // in the url. This is precisely where the code is using $this->id rather than the $story id passed as a param.

Comments

 [2009-08-11 03:57 UTC] shupp (Bill Shupp)
-Status: Open +Status: Assigned -Assigned To: +Assigned To: shupp
 [2010-06-06 00:03 UTC] shupp (Bill Shupp)
-Status: Assigned +Status: Wont fix
This package is being deprecated, and superseded by Services_Digg2.