Page MenuHomePhabricator

: in URL after ; parsed as list item
Closed, ResolvedPublic

Description

Author: timwi

Description:
BUG MIGRATED FROM SOURCEFORGE
http://sourceforge.net/tracker/index.php?func=detail&aid=595869&group_id=34373&atid=411192
Originally submitted by Brion Vibber (vibber)<a href="/help/icon_legend.php?context=user_wantsdonations&amp;user_id=446709&amp;return_to=%2F"><IMG src="http://images.sourceforge.net/images/icons/donate.png" alt="Accepting Donations" border="0" width="16" height="16"></a> 2002-08-16 08:21

This source:
;Well :this is ok
;Ooops http://this.renders.wrong :though
;Ooops [http://this.renders.wrong too] :ack!

Renders as:
Well

this is ok

Ooops http

//this.renders.wrong :though

Ooops [http

//this.renders.wrong too] :ack!

which is a bit unintuitive. The URLs perhaps ought to
be parsed before the ;s and :s are separated.

In Usemod, this rendered as the expected:
Well

this is ok

Ooops http://this.renders.wrong

though

Ooops [too]

ack

Phase II seems to have had the same error as phase III.
It doesn't come up often though, as definition lists
are extremely rare. I found one in the FAQ though, and
it barfs when I try to put URL links into the entry (;)
portion.

  • Additional comments ------------------------

Date: 2004-04-24 10:15
Sender: SF user vibber

Tested in current 1.3; still awful, it's also invalid markup:

This source:

Well this is ok
Ooops this.renders.wrong"
class='external'>http://this.renders.wrong :though
Ooops
this.renders.wrong'
class='external'>too (http://
this.renders.wrong) :ack!


Date: 2004-04-29 08:20
Sender: SF user vibber

Partial fix is checked in, but not yet complete. See demo:
http://test.wikipedia.org/wiki/List_link_bug


Version: unspecified
Severity: normal

Details

Reference
bz357

Revisions and Commits

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 6:53 PM
bzimport set Reference to bz357.
bzimport added a subscriber: Unknown Object (MLST).

wmahan_04 wrote:

All the test cases mentioned should be fixed in HEAD.

epriestley added a commit: Unknown Object (Diffusion Commit).Mar 4 2015, 8:19 AM