Posted by DamienMcKenna on September 15, 2011 at 2:33am
5 followers
Issue Summary
The second condition of the query in node_feed() should have a table prefix, to make it like the other parts of the query. The current query is:
<?php
$nids = db_select('node', 'n')
->fields('n', array('nid', 'created'))
->condition('n.promote', 1)
->condition('status', 1)
->orderBy('n.created', 'DESC')
->range(0, variable_get('feed_default_items', 10))
->addTag('node_access')
->execute()
->fetchCol();
?>To make the query look more uniform it should instead be:
<?php
$nids = db_select('node', 'n')
->fields('n', array('nid', 'created'))
->condition('n.promote', 1)
->condition('n.status', 1)
->orderBy('n.created', 'DESC')
->range(0, variable_get('feed_default_items', 10))
->addTag('node_access')
->execute()
->fetchCol();
?>
Comments
#1
I tagged this as Novice as it's a teeny-tiny thing that someone might like to do :)
#2
I tagged Quick fix and wait for 2 weeks. If no one interested I will do.
#3
Here's a patch!
#4
#5
Committed to 7.x and 8.x.
#6
Automatically closed -- issue fixed for 2 weeks with no activity.