Ticket #221 (new enhancement)

Opened 3 months ago

Last modified 2 months ago

Sanitize Feed code/Make feeds page more usable

Reported by: tn123 Assigned to: royger
Priority: major Component: wTorrent
Version: Keywords:
Cc:

Description

The feeds page currently lacks some useful features:

  • display the last few items
  • user-defined titles (in case the feed doesn't provide a useful one)

Futhermore the backend code might need some improvements.

Attachments

wtorrent-feeds-wip1.diff (19.0 kB) - added by tn123 on 19/10/08 03:53:54.
Work-In-Progress v1

Change History

19/10/08 03:53:54 changed by tn123

  • attachment wtorrent-feeds-wip1.diff added.

Work-In-Progress v1

19/10/08 04:01:28 changed by tn123

WIP v1 addresses the following:

  • Will show the last few items for each feed
  • Will allow the user to specify a user-defined title for each feed.
  • Codebase was sanitized

WIP v1 has some remaining issues to be evaluated/resolved:

  • max items to show currently is hard coded to 14 days / 30 items. This is enough for e.g. TV show RSS feeds. However one might consider to make this a config setting
  • edit is still not implemented (wasn't before) but throws an exception now
  • The template changes (especially those concerning highlighting new items within 2 last 2 days) don't seem clean to me. Not experienced with smarty.
  • The feeds schema changed. install.php was updated accordingly. However it there is no migration strategy for old existing DBs.

29/10/08 06:11:04 changed by andrew@thek.ro

Hey,

I got SSL feeds working, dont know if you want to add to your diff script. Line Numbers and code:

7621 + curl_setopt($fp, CURLOPT_SSL_VERIFYHOST, FALSE); 7622 + curl_setopt($fp, CURLOPT_SSL_VERIFYPEER, FALSE);


Add/Change #221 (Sanitize Feed code/Make feeds page more usable)