BUG: number of items fetched for RSS is hardcoded

Found a bug? Tell us!!
Post Reply
Romain Wartel

BUG: number of items fetched for RSS is hardcoded

Post by Romain Wartel »

15 items only can be published via RSS.

This limitation in NOT configurable in the configuration file, it is hardcoded in rss.php:

$entries = serendipity_fetchEntries(null, true, 15, false, (isset($modified_since) ? $modified_since : false), 'timestamp DESC', '', false, true);

This should be replaced by:

$entries = serendipity_fetchEntries(null, true, $serendipity['fetchLimit'], false, (isset($modified_since) ? $modified_since : false), 'timestamp DESC', '', false, true);

Regards,
Romain.
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: BUG: number of items fetched for RSS is hardcoded

Post by garvinhicking »

"15" is usually a standard item value for RSS feeds. Since there are many blogs that use "1" for a fetchlimit, then their rss feeds would also only contain 1 item, which is not really wanted.

So we would need to introduce another config item, but I think editing the file at the moment is a good idea, because it really is a standard setting...

Many thanks for your idea and suggestion though!

Regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Post Reply