Opened 20 years ago
Closed 20 years ago
#539 closed defect (fixed)
timeline RSS <image> <link> url is just the http://hostname/, not link to timeline
Reported by: | radix | Owned by: | daniel |
---|---|---|---|
Priority: | low | Milestone: | 0.8 |
Component: | timeline | Version: | 0.7.1 |
Severity: | minor | Keywords: | timeline rss image link url |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
While the <channel> <link> is correct, the <image> <link> just points to e.g. http://twistedmatrix.com/ instead of http://twistedmatrix.com/trac/imagination/trac/timeline. It's a very simple fix to timeline_rss.cs:
—- /home/trac/TRAC/trac-0.7.1/templates/timeline_rss.cs 2004-05-20 21:59:41.000000000 +0000 +++ /home/trac/usr/share/trac/templates/timeline_rss.cs 2004-06-11 17:04:57.000000000 +0000 @@ -25,7 +25,7 @@
<image>
<title><?cs var:$project.name?></title> <url><?cs if !$header_logo.src_abs ?><?cs var:$base_url ?><?cs /if ?><?cs var $header_logo.src ?></url>
- <link><?cs var:$base_url ?></link>
+ <link><?cs var:$base_url ?><?cs var:$trac.href.timeline ?></link>
</image> <?cs each:item = $timeline.items ?>
<?cs if:item.type == #1 ?><-- Changeset —>
Attachments (1)
Change History (4)
comment:1 by , 20 years ago
comment:2 by , 20 years ago
Milestone: | → 0.8 |
---|---|
Owner: | changed from | to
Priority: | normal → low |
Severity: | normal → minor |
Status: | new → assigned |
comment:3 by , 20 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Applied and fixed in [769]. Many thanks for the patch!
Sorry, I assumed that the comments were <pre>d. I'll attach the patch.