Page MenuHomePhabricator

Logic error in {{BASEPAGENAME}}
Closed, ResolvedPublic

Description

Author: emddudley

Description:
{{BASEPAGENAME}} does not work when viewing an old version of a page, such as in
the provided URL. In the {{talkarchive}} template, there should be a link to the
current talk page instead of the text "[[User_talk:|current talk page]]".


Version: unspecified
Severity: normal
URL: http://en.wikipedia.org/w/index.php?title=User_talk:Srleffler&oldid=45380066

Details

Reference
bz6062

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 9:19 PM
bzimport added a project: MediaWiki-Parser.
bzimport set Reference to bz6062.
bzimport added a subscriber: Unknown Object (MLST).

emddudley wrote:

The link at the top of the provided URL has been removed pending this bug's
resolution. I found the bug using:

[[{{{1|{{TALKSPACE}}:{{BASEPAGENAME}}}}}|current talk page]]

Normally, it returns something like "[[User_talk:Mdd4696|current talk page]]",
but when viewing old revisions, it returns "[[User_talk:|current talk page]]".

robchur wrote:

Actually, the problem I found was a more generic one; a logic error caused the
last part of the title, after /, to be discarded...if there was no / in the
first place, the entire title was discarded. Oops.

Fixed in SVN trunk, r14376.