Hi @malmac
That error usually means something changed in the code while it was being copied into the snippet editor. A bracket, quote, or the && may have been dropped or converted along the way, which can easily happen when copying code from an email notification rather than directly from the forum.
I re-checked the snippet as posted on a clean Aggregator 5.5.0 site, and it is working correctly there.
Please delete the snippet you saved and use the version below instead. It does exactly the same thing, but I’ve written it with one condition per line and removed the &&, so there is less that can go wrong when copying it.
Please copy it directly from the code box on this page. Once pasted, also check that all quotes are plain straight quotes:
add_filter( 'wpra.importer.wpPosts.query.args', 'qn_custom_feed_newest_first' );
function qn_custom_feed_newest_first( $args ) {
if ( ! is_feed( 'qnfeed' ) ) {
return $args;
}
if ( ! empty( $args['orderby'] ) ) {
return $args;
}
$args['orderby'] = 'date';
$args['order'] = 'DESC';
return $args;
}
If the editor still reports an error, please paste the code here exactly as it appears in your snippet editor and we can compare it.
On the feed limit, 0 does not mean unlimited. The field does not accept 0, and if the value ends up as 0, Aggregator falls back to the default limit of 15 items.
With the snippet active, the limit simply controls how many of the most recent items are included in the feed. As new items are imported, the oldest ones drop off.
For a news feed, I’d normally recommend somewhere around 50 to 100 items, since feed readers generally only need the most recent entries anyway. If you want every imported item to remain in the feed, you can set the limit higher than the total number of items on the site.
