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

Comments

tangent’s picture

tangent commented

Please review. This patch is RTBC.

Rok Žlender’s picture

Rok Žlender commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new 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 commented
Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

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

drumm
he/him
Location NY, US
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 commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new 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
commented
Status: Needs review » Fixed

Committed to 5.x.

Status: Fixed » Closed (fixed)

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