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:

    $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:

    $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();
CommentFileSizeAuthor
#3 table-prefix-1279888-3.patch545 bytesdavisben
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

Priority: Normal » Minor

I tagged this as Novice as it's a teeny-tiny thing that someone might like to do :)

droplet’s picture

Issue tags: +Quick fix

I tagged Quick fix and wait for 2 weeks. If no one interested I will do.

davisben’s picture

Status: Active » Needs review
FileSize
545 bytes

Here's a patch!

droplet’s picture

Status: Needs review » Reviewed & tested by the community
Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x.

Automatically closed -- issue fixed for 2 weeks with no activity.