Closed (won't fix)
Project:
Drupal core
Version:
5.x-dev
Component:
aggregator.module
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
17 Sep 2007 at 19:18 UTC
Updated:
21 Jun 2011 at 12:43 UTC
I am overriding theme_aggregator_summary_item() to allow the addition of the "blog this" link on the aggregator/sources page, not just the block (where this is already enabled).
On line 1164 in the function aggregator_page_sources(), there is a sql called to load all the items for a feed:
$items = db_query_range('SELECT i.title, i.timestamp, i.link FROM {aggregator_item} i WHERE i.fid = %d ORDER BY i.timestamp DESC', $feed->fid, 0, variable_get('aggregator_summary_items', 3));
Since I want to add the "blog this" link, I need for the item id to be included in the array so it's available when passed to aggregator_summary_item()
So my request is to just add one more field (iid) to that SELECT statement:
$items = db_query_range('SELECT i.iid, i.title, i.timestamp, i.link FROM {aggregator_item} i WHERE i.fid = %d ORDER BY i.timestamp DESC', $feed->fid, 0, variable_get('aggregator_summary_items', 3));
Comments
Comment #1
dpearcefl commentedConsidering that no new features will be added to D5 and that no one has shown any interest in this issue for a long time, I am closing this issue ticket. If you think we still need this feature request, please reopen it and move it to the D8 issue queue.