tresni / freenas-rss-extension

Automatically exported from code.google.com/p/freenas-rss-extension
0 stars 0 forks source link

Add a "Recent Downloads" Page #33

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Trying to see what has recently been downloaded is cumbersome with the History 
page as you need to look through each feed.

Attached is a patch which makes three changes:
1. Modifies rss_cron.php to add a downloadDate (equal to now) to the item 
before saving it
2. Modifies rss_class_history.php to set the downloadDate to be pubDate, if it 
is not set.
3. Adds rss_recent.php which shows the 20 most recently downloaded items.

You probably want to review the patch as PHP is not my native language.

Original issue reported on code.google.com by Mr.M.Dav...@gmail.com on 4 Oct 2010 at 8:52

Attachments:

GoogleCodeExporter commented 9 years ago
I like the idea.  Not sure why your patch file is completely replacing the 
rss_cron file as it looks like most of it doesn't change, I'm guessing a 
whitespace or line-ending issue.

I like the idea, but I don't know how interested most people would be with 
this.  Willing to add if I see more people interested in it, but right now I'm 
going to leave as an open enhancement.  If you are interested, star this issue, 
*DO NOT ADD A "Me too!" COMMENT* as that will do nothing but anger other people 
who have stared the issue.

Original comment by tre...@gmail.com on 10 Oct 2010 at 4:37

GoogleCodeExporter commented 9 years ago
Definitely sounds like a white space issue. Was doing the editing from a 
Windows editor.

There's a slight bug in the modified rss_feed_edit.php file. The added line 
doesn't close the PHP tag for the gettext("Recent Downloads") call.

Original comment by Mr.M.Dav...@gmail.com on 10 Oct 2010 at 9:24