Summary | off by one error in RSS feeds on portal page (number of articles to display)! |
Queue | IMP |
Queue Version | Git master |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | mrubinsk (at) horde (dot) org |
Requester | gavin.mccullagh (at) gcd (dot) ie |
Created | 08/19/2010 (5411 days ago) |
Due | |
Updated | 09/08/2010 (5391 days ago) |
Assigned | |
Resolved | 09/08/2010 (5391 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | Yes |
Assigned to Michael Rubinsky
Summary ⇒ off by one error in RSS feeds on portal page (number of articles to display)!
Priority ⇒ 1. Low
Fix off by one error
Bug: 9188Submitted By: gavin (dot) mccullagh (at) gcd (dot) ie
http://git.horde.org/diff.php/horde/lib/Block/feed.php?rt=horde-git&r1=fabc7e43b5dc1d9e651f5a343eb49266e46a0b63&r2=7981d3a7ef96c967da0ddc0862ceb5e208d04e54
Fix off by one error
Bug: 9188Submitted by: gavin (dot) mccullagh (at) gcd (dot) ie
http://cvs.horde.org/diff.php/horde/lib/Block/feed.php?rt=horde&r1=1.1.2.1&r2=1.1.2.2&ty=u
so that the fix goes into Kolab.
Gavin
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ off by one error in RSS feeds on portal page (number of articles to display)
Queue ⇒ IMP
Milestone ⇒
Patch ⇒ Yes
New Attachment: rss_limit_off_by_one.patch
State ⇒ Unconfirmed
https://issues.kolab.org/issue4496
but issue appears to also exist in latest Horde.
http://git.horde.org/co.php/horde/lib/Block/feed.php?r=fabc7e43b5dc1d9e651f5a343eb49266e46a0b63
If you add an RSS feed to the Horde portal page and specify N articles
to be displayed, you will get N+1 articles displayed.
Attached is a patch that fixes this issue.