|
|
|
[
Permlink
| « Hide
]
Daniel Bezruchkin - 13/Oct/06 06:43 PM
Extracting this into your zend directory will get RDF feeds to work.
The attached zip doesn't work. It makes a major error (blank page). Will investigate on the error.
It's because of:
$success = @$doc->loadXML(Zend_Feed::utf8ToUnicodeEntities($string)); in Feed.php The utf8ToUnicodeEntities function doesn't exist (wrong code version?) In the previous version it was: I have the exact same issue. Any idea when this will be resolved? I've used PEAR's RSS class, no good. I've used Magpie/simplepie, no good. This one was able to parse all of the new feeds but cannot parse the 1.0 rdf feeds. So it's the best so far!
This bug depends on ZF-26.
RSS 1.0 lists items outside channel node and Zend_Feed actually can't handle this situation. Rather than fixing the behavior, I would suggest to add a new RDF class, as proposed in the description of this issue. The main difference between RSS 0.91 branch (created by Dave Winer) and RSS 1.0 branch (managed by RSS-DEV Working Group) is that the latter is RDF based while RDF architecture has been completely removed in RSS 0.91, RSS 0.92, RSS 2.0. Additionally, I would suggest to add a new class property to return feed type/version.
I forgot to say that my previous comment has been inspired by http://www.nabble.com/zend-feed-issue--tf4928553s16154.html#a14108105
The only difference between RSS 1.0 and other versions that is related to this issue is that item elements are not contained within the channel element. The attached file patch.diff modifies Zend_Feed_Rss to check for this and also patches the appropriate test in the test suite so that, without the patch to Zend_Feed_Rss, RSS 1.0 feed tests will fail.
Hi Matthew,
I gave a look at the patch you submitted a few days ago. The following line doesn't really makes sense to me.
_importRssValid method is an utility method and we cannot assume in advance the file he's going to fetch is not a valid empty feed. The other part of the patch, the code fragment that should introduce RSS 1.0 compatibility it's fine, but I think it's incomplete. Did you think about how an imported RSS 1.0 feed will be printed out? What do you propose to fix this consequential issue? For the sake of completeness, I'd like to share an additional though. Any news on this feature?
I would suggest to change status to unassigned if work is not in progress. |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||