Feeds Promoted items always = 3

Okay, the bug was in the feedapi_item.module -- the value '3' was hardcoded in the for loop. here's the corrected feedapi_item_nodeapi() function, also corrected the comment for the function ;-)

I replaced the hardcoded '3' with a reference to the settings value in the $feed object, $feed->settings['processors']['feedapi_item']['promote'] -- is this always valid on the $feed object?

/**
 * Implementation of hook_nodeapi().
 */
function feedapi_item_nodeapi(&$node, $op, $teaser, $page) {
  switch ($op) {
  case 'delete':
    $result = db_query("SELECT fiid FROM {feedapi_node_item} WHERE nid = %d", $node->nid);
    if (db_num_rows($result) > 0) {
      $feed_item = db_fetch_object($result);
      feedapi_item_feedapi_item_delete($feed_item);
    }
    break;
  }
  if ($op == 'view' && !$teaser && isset($node->feed->nid)) {
    $feed = $node->feed;
    if (is_array($feed->processors)) {
      if (in_array('feedapi_item', call_user_func_array('array_merge', $feed->processors))) {
        $items = module_invoke('feedapi_item', 'feedapi_item_fetch_items', $feed);
        for ($i = 0; $i != $feed->settings['processors']['feedapi_item']['promote'] && $i != count($items); $i++) {
          $list[] = l($items[$i]->title, 'node/'. $items[$i]->nid);
        }
        $node->content['body']['#value'] .= theme('item_list', $list);
      }
    }
  }
}