Edgewall Software
Modify

Opened 14 years ago

Closed 13 years ago

Last modified 13 years ago

#10196 closed defect (fixed)

[patch] Intertrac link does not work for Image()

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?

Attachments (1)

intertrac.patch (1.7 KB ) - added by Dirk Stöcker 13 years ago.
A possible solution for this problem

Download all attachments as: .zip

Change History (8)

comment:1 by Remy Blank, 14 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, 13 years ago

Attachment: intertrac.patch added

A possible solution for this problem

comment:3 by Dirk Stöcker, 13 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, 13 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, 13 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, 13 years ago

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

Fixed in [10894:10896].

in reply to:  5 comment:7 by Dirk Stöcker, 13 years ago

Replying to cboos:

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

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

I didn't want to change too much at once. I was happy it worked at all :-)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Dirk Stöcker.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Dirk Stöcker to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.