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

Bug #12973 Broken code in Yadis.php, lines 305-307
Submitted: 2008-01-24 19:41 UTC
From: saeven Assigned: padraic
Status: Closed Package: Services_Yadis (version 0.2.1)
PHP Version: 5.2.5 OS: Irrelevant
Roadmaps: (Not assigned)    
Subscription  


 [2008-01-24 19:41 UTC] saeven (Alexandre Lemaire)
Description: ------------ Having installed Yadis through Pear, the class immediately fails a parse check, seeing that lines 305 through 307 are broken: $this->_yadisUrl = $xri->setHttpRequestOptions($this->getHttpRequestOptions()) ->setNamespace($this->_namespace) ->toUri($yadisId); Adjusting the fluent calls to a single continuous line solves the problem: $this->_yadisUrl = $xri->setHttpRequestOptions($this->getHttpRequestOptions())->setNamespace($this->_namespace)->toUri($yadisId);

Comments

 [2008-03-02 08:42 UTC] domster (Dominic Scheirlinck)
Actually, it looks like this bug is simply the result of a missing semicolon at the end of line 304. That line should read: $xri = Services_Yadis_Xri::getInstance(); I've left that method call with separated '->' operators, and things are working fine. Is it possible you just added a semi-colon to fix this, as well as putting that call on a single line?
 [2008-06-19 08:35 UTC] padraic (Pádraic Brady)
To be fixed in next point release in early July.
 [2008-11-02 00:27 UTC] shupp (Bill Shupp)
This still hasn't been fixed. What's with the delay? It was reported in January, acknowledged by the author in June, and it's now November. Is this package orphaned?
 [2008-12-07 09:17 UTC] doconnor (Daniel O'Connor)
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.