View Single Post
Old 11-05-2013, 11:36 PM   #2071
JimmXinu
Plugin Developer
JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.JimmXinu ought to be getting tired of karma fortunes by now.
 
JimmXinu's Avatar
 
Posts: 6,328
Karma: 3966249
Join Date: Dec 2011
Location: Midwest USA
Device: Kindle Paperwhite(10th)
Quote:
Originally Posted by davidfor View Post
Ok, it was failing for stories marked as incomplete. All the dates are there in timestamp format followed by a noscript tag and the date. The others use the noscript version. You could replace that line with the following to be consistent with the other dates.
Hmmm. For no reason that I understand, the noscript wasn't coming through correctly when getting the text for the whole span. So I rashly assumed that Soup wouldn't find it by name either. Changed.

Quote:
Originally Posted by davidfor View Post
Whoops just noticed something in the description. They all have a "[" at the end. Replace lines 209-211 (they all start with "desc =") with:

Code:
desc = "%s"%lc4.text
desc = desc[:desc.index("[More")]
That looks like another premium/non-premium difference. Non-premium doesn't get the '[More'. So I've made it conditional.

Attached is a version with these latest changes.

UPDATE Nov 6, 2013 - Remove obsolete beta versions

Last edited by JimmXinu; 11-06-2013 at 11:44 AM. Reason: Remove obsolete beta versions
JimmXinu is online now