Project

General

Profile

Actions

Patch #1356

closed

In Activity views, changesets titles can be multiline while they should not

Added by Paul Rivier almost 16 years ago. Updated almost 16 years ago.

Status:
Closed
Priority:
Normal
Category:
Feeds
Target version:
Start date:
2008-06-02
Due date:
% Done:

0%

Estimated time:

Description

This concerns both HTML view and Atom feed.

Are concerned entries for repository changeset whose content is
  • multiline
  • first line is short

I can't find an example in Redmine.org development Activity timeline, because Jean-Philippe usualy does not commit with such a changelog, so here is a example :

This commit fixes :
 - #13
 - #34

In this case, truncate would return a truncated string, no matter if it contains newlines. This multilines string end up raw, in XML <entry><title> </title> section of the feed, and in the <a href="/repositories/revision/..."> section of the Html activity page. While the later seem to be properly rendered by web engines, the former makes many feedreaders choke and display erroneous data, mixing fields. I'm not sure what the RFC says about it, but I'm sure most newsreaders are not designed to handle multilines titles.

Attached patch is one working solution and works on my instance. It uses a helper, very similar to truncate, but ensuring truncated string is NOT multiline. Feel free to comment.


Files


Related issues

Related to Redmine - Feature #34025: Raise the maximum length of the title element in the Atom feed from 100 to 300 charactersClosedGo MAEDA

Actions
Actions

Also available in: Atom PDF