Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing pubDate data after parsing RSS feed #31

Closed
jamesokelly opened this issue Jul 20, 2017 · 2 comments
Closed

Missing pubDate data after parsing RSS feed #31

jamesokelly opened this issue Jul 20, 2017 · 2 comments
Assignees
Milestone

Comments

@jamesokelly
Copy link

When parsing http://feeds.feedburner.com/ubergizmo the date field is often blank even though the pubDate field contains data inside the item, and the feed seems to parse successfully. BTW I also tested this feed with the MWFeedParser code and then the date is parsed/extracted.

e.g.

<item><title>Facebook Confirms Plans To Launch A New Subscription Service</title>
<link>http://www.ubergizmo.com/2017/07/facebook-to-launch-news-subscription/?utm_source=mainrss</link> 
<pubDate>20 Jul 2017 03:23:26 +0000</pubDate> 
<lastmodifiedDate /> 
<dc:creator>Tyler Lee</dc:creator> 
<category><![CDATA[General]]></category> 
<category><![CDATA[Facebook]]></category> 
<guid isPermaLink="false">284104</guid> 
<content:encoded><![CDATA[ <p><img
class="aligncenter size-full wp-image-187417" src="http://cdn2.ubergizmo.com/wp-content/uploads/2014/02/facebook-logo.jpg" alt="" width="640" height="480" />For the most part, a lot of online publications that are free to read rely on advertisements for generating revenue. However with ad blockers becoming more widely used, and with users turning to alternative sources for their information, new ways of generating revenue need to be invented.<span
id="more-284104"></span></p><p>Subscription services are one of those methods, and it seems that publishers will soon have the <a
href="https://www.thestreet.com/story/14233293/1/facebook-s-campbell-brown-we-are-launching-a-news-subscription-product.html" target="_blank" rel="noopener">option of using Facebook as a subscription service</a>. This was confirmed by Facebook exec Campbell Brown who said, <em>&#8220;One of the things we heard in our initial meetings from many newspapers and digital publishers is that &#8216;we want a subscription product &#8212; we want to be able to see a paywall in Facebook,&#8217; and that is something we&#8217;re doing now. We are launching a subscription product.&#8221;</em></p><p>The inclusion of this feature came about after news publications complained that they had little control over what their stories were being shared on the platform, and how companies such as Facebook and Google were benefiting from the work of newspapers without “fairly” compensating the publishers.</p><p>The tests for this new subscription service is expected to kick off this October, but details such as pricing have yet to be revealed, but we suppose we will have the official details in due time.</p> <a
href="http://www.ubergizmo.com/2017/07/facebook-to-launch-news-subscription/?utm_source=mainrss">Facebook Confirms Plans To Launch A New Subscription Service</a> , original content from <a
href='http://www.ubergizmo.com'>Ubergizmo</a>. Read our <a
href='http://www.ubergizmo.com/terms/'>Copyrights and terms of use</a>.  <div class="feedflare">
<a href="http://feeds.feedburner.com/~ff/ubergizmo?a=5f7imQYQ90Y:3X9vkv7l_0Q:yIl2AUoC8zA"><img src="http://feeds.feedburner.com/~ff/ubergizmo?d=yIl2AUoC8zA" border="0"></img></a> <a href="http://feeds.feedburner.com/~ff/ubergizmo?a=5f7imQYQ90Y:3X9vkv7l_0Q:qj6IDK7rITs"><img src="http://feeds.feedburner.com/~ff/ubergizmo?d=qj6IDK7rITs" border="0"></img></a> <a href="http://feeds.feedburner.com/~ff/ubergizmo?a=5f7imQYQ90Y:3X9vkv7l_0Q:gIN9vFwOqvQ"><img src="http://feeds.feedburner.com/~ff/ubergizmo?i=5f7imQYQ90Y:3X9vkv7l_0Q:gIN9vFwOqvQ" border="0"></img></a>
</div>]]></content:encoded> 
</item>
@nmdias
Copy link
Owner

nmdias commented Jul 22, 2017

Hi @jamesokelly,

Give it a spin from the master branch, it should be parsing the date now.

@nmdias nmdias added this to the 6.1.2 milestone Jul 22, 2017
@jamesokelly
Copy link
Author

Hi Nuno (@nmdias),

Great, thank you that seems to have done the trick.

It's probably my naivety using GitHub, but I found when I did 'pod update' or downloaded the repository using the browser the change wasn't present so I edited it in manually. Perhaps I need to pull your exact commit using the command line?

Anyway my empty dates have gone away - so thanks again.

Kind Regards,
James

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants