Edgewall Software

Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

#10196 closed defect (fixed)

[patch] Intertrac link does not work for Image() — at Version 6

Reported by: Dirk Stöcker Owned by: Dirk Stöcker
Priority: normal Milestone: 0.12.3
Component: rendering Version: 0.12-stable
Severity: normal Keywords: intertrac patch
Cc: Branch:
Release Notes:

[[Image()]] macro supports InterTrac source: links

API Changes:
Internal Changes:

Description

When I do an Intertrac link, for normal links it works correctly. For Image() this fails.

Link:

[[trac:source:/trunk/doc/images/bkgnd_pattern.png]]

source:/trunk/doc/images/bkgnd_pattern.png

Image:

[[Image(trac:source:/trunk/doc/images/bkgnd_pattern.png)]]

source:/trunk/doc/images/bkgnd_pattern.png in The Trac Project

Supplying the resulting link directly works:

[[Image(http://trac.edgewall.org/browser/trunk/doc/images/bkgnd_pattern.png?format=raw)]]

http://trac.edgewall.org/browser/trunk/doc/images/bkgnd_pattern.png

The directly expanded link does not:

[[Image(http://trac.edgewall.org/intertrac/source:/trunk/doc/images/bkgnd_pattern.png?format=raw)]]

http://trac.edgewall.org/intertrac/source:/trunk/doc/images/bkgnd_pattern.png

Probably the "?format=raw" is not correctly handled by the "intertrac/source:" URL?

Change History (7)

comment:1 by Remy Blank, 13 years ago

Keywords: intertrac added
Milestone: next-minor-0.12.x

comment:2 by Dirk Stöcker, 13 years ago

Any progress with this?

by Dirk Stöcker, 12 years ago

Attachment: intertrac.patch added

A possible solution for this problem

comment:3 by Dirk Stöcker, 12 years ago

Keywords: patch added
Summary: Intertrac link does not work for Image()[patch] Intertrac link does not work for Image()

I attached a patch, which can solve this issue and makes Image() links work when they have a "format=raw" attached.

comment:4 by Christian Boos, 12 years ago

Milestone: next-minor-0.12.x0.12.3
Owner: set to Christian Boos

Thanks! Will try, test and apply if all goes well ;-)

comment:5 by Christian Boos, 12 years ago

Owner: changed from Christian Boos to Dirk Stöcker

I don't like so much the idea of having to write:

[[Image(trac:source:/trunk/doc/images/bkgnd_pattern.png?format=raw)]]

because the following is not supposed to work either:

[[Image(source:/trunk/doc/images/bkgnd_pattern.png?format=raw)]]

I followed-up with some additional changes which make the following syntax work:

[[Image(trac:source:/trunk/doc/images/bkgnd_pattern.png)]]

comment:6 by Christian Boos, 12 years ago

Release Notes: modified (diff)
Resolution: fixed
Status: newclosed

Fixed in [10894:10896].

Note: See TracTickets for help on using tickets.