The default title given to feeds is "Drupal - Site slogan". However, if there is no site slogan, a not uncommon situation, then feeds have a title of "Drupal - ".

The attached patch adds a dash only when the site slogan is not empty.

CommentFileSizeAuthor
#6 node.module.d5.feedname.patch828 bytesFreso
#2 node.module.feedname.patch826 bytesRok Žlender
node_feed.patch782 bytestangent
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tangent’s picture

tangent CreditAttribution: tangent commented

Please review. This patch is RTBC.

Rok Žlender’s picture

Rok Žlender CreditAttribution: Rok Žlender commented
Status: Needs review » Reviewed & tested by the community
FileSize
node.module.feedname.patch826 bytes

Patch works.
- no slogan no dash
- if there is slogan dash comes back

Rerolled now applies without offset.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)
drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Version: 6.x-dev » 5.x-dev
Status: Closed (fixed) » Patch (to be ported)

The committed version actually only has the title in the feed, and does not include the optional slogan. This should be backported to Drupal 5.

If the slogan is missed, please refile against Drupal 6.

Freso’s picture

Freso CreditAttribution: Freso commented
Status: Patch (to be ported) » Needs review
FileSize
node.module.d5.feedname.patch828 bytes

Ported patch. Not tested, but is identical (except for line numbers) to the D6 one.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Needs review » Fixed

Committed to 5.x.

Status: Fixed » Closed (fixed)

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