Edgewall Software
Modify

Opened 6 years ago

Closed 5 years ago

Last modified 9 months ago

#7386 closed defect (fixed)

Trac fails on URIs

Reported by: erlehmann Owned by: cboos
Priority: normal Milestone: 0.12
Component: wiki system Version:
Severity: normal Keywords: review, traclinks
Cc:
Release Notes:
API Changes:

Description

Look at this link: <http://en.wikipedia.org/wiki/Mornington_Crescent_(game)> - it isn't highlighted correctly - the last bracket should be part of the URI. Now take <xmpp:erlehmann@jaim.at?roster> or <urn:isbn:0-345-33971-1>, which aren't highlighted at all.

A detailed solution with a proper regex that also matches links in parentheses can be found at <http://dieweltistgarnichtso.net/index.php?/archives/32-Constructing-a-regular-expression-that-matches-URIs.html#extended> .

Attachments (1)

t7386-alt-shref-r7606.diff (2.5 KB) - added by cboos 6 years ago.
Add alternative syntax for short links: <realm_or_protocol:target>

Download all attachments as: .zip

Change History (8)

comment:1 in reply to: ↑ description Changed 6 years ago by rblank

  • Milestone set to 1.0

Replying to erlehmann:

Look at this link: (…)

And I actually clicked on the link, and thought "wtf?" :-)

We could indeed do better than the current state.

comment:2 Changed 6 years ago by cboos

  • Component changed from web frontend to wiki system
  • Keywords traclinks added
  • Milestone changed from 1.0 to 0.12
  • Owner set to cboos

Strange that I didn't write anything on the ticket, as I have implemented it a while back after reading this ticket (or was it a duplicate?)

Changed 6 years ago by cboos

Add alternative syntax for short links: <realm_or_protocol:target>

comment:3 Changed 6 years ago by cboos

  • Keywords review added
  • Status changed from new to assigned

No, the patch was actually targeting this ticket, sorry for not answering earlier.

comment:4 Changed 5 years ago by cboos

  • Resolution set to fixed
  • Status changed from assigned to closed

Patch applied in r8696.

comment:5 Changed 9 months ago by rjollos

  • Keywords changed from traclinks review to traclinks, review

comment:6 Changed 9 months ago by rjollos

  • Keywords traclink added; traclinks removed

comment:7 Changed 9 months ago by rjollos

  • Keywords traclinks added; traclink removed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain cboos.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from cboos 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.