This issue evolved out of #316019, which resulted in two patches that I realized should go two different ways. One of them ended up here.

Currently theme_aggregator_block_item(...) does not know anything about the source of the feed it is presenting. This is not optimal as you many times would like to be able to show information about the blog source in a feed block listing and I believe this is a bug as the only way currently to achieve this is to hack core.

Therefor I am proposing a patch that makes the feed source title (named 'ftitle') sent to theme_aggregator_block_item(...) for theming purposes when a category aggregator is being rendered. I'm attaching a patch against Drupal 6.

CommentFileSizeAuthor
aggregator.core-feed_title_to_theme_v6.patch1.43 KBztyx
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ztyx’s picture

ztyx CreditAttribution: ztyx commented

Oh, here's a working a link: #316019: Add source feed title to theming

ztyx’s picture

ztyx CreditAttribution: ztyx commented
Status: Active » Needs review
ztyx’s picture

ztyx CreditAttribution: ztyx commented

If this patch is accepted you might also want to look at #324082: Present source feed in category aggregator.

ztyx’s picture

ztyx CreditAttribution: ztyx commented

Clarification: This patch adds the (source feed title) variable $item->ftitle to theme_aggregator_block_item(). This patch enables you to be able to present the source feeds like in this screenshot.

dpearcefl’s picture

dpearcefl CreditAttribution: dpearcefl commented

Is this still an issue using current Drupal 6?

Status: Needs review » Needs work

The last submitted patch, aggregator.core-feed_title_to_theme_v6.patch, failed testing.

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.