Edgewall Software

Ticket #7386 (assigned defect)

Opened 6 months ago

Last modified 3 months ago

Trac fails on URIs

Reported by: erlehmann Owned by: cboos
Priority: normal Milestone: 0.12
Component: wiki system Version:
Severity: normal Keywords: traclinks review
Cc:

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

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

Change History

in reply to: ↑ description   Changed 3 months 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.

  Changed 3 months ago by cboos

  • keywords traclinks added
  • owner set to cboos
  • component changed from web frontend to wiki system
  • milestone changed from 1.0 to 0.12

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 3 months ago by cboos

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

  Changed 3 months 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.

Add/Change #7386 (Trac fails on URIs)

Author



Change Properties
<Author field>
Action
as assigned
as The resolution will be set. Next status will be 'closed'
to The owner will change from cboos. Next status will be 'new'
 
Note: See TracTickets for help on using tickets.