Support fragments in mw.util.getUrl()
Closed, ResolvedPublic
Actions

Description

Currently mw.util.getUrl() does not support fragments.

mw.util.getUrl( 'Title Ä#Fragment Ä', { useskin: 'vector' } )

evaluates to /wiki/Title_%C3%84%23Fragment_%C3%84?useskin=vector.

Expected result: /wiki/Title_%C3%84?useskin=vector#Fragment_.C3.84

Details

	Subject	Repo	Branch	Lines +/-
	mediawiki.util: Use correct encoding for fragment in getUrl	mediawiki/core	master	+56 -7
	Support fragments in mw.util.getUrl()	mediawiki/core	master	+33 -5

Customize query in gerrit

Related Objects
Search...

Status	Assigned	Task
Resolved	Unicornisaurous	T103715 Support fragments in getUrl() of mw.Title
Resolved	Unicornisaurous	T103553 Support fragments in mw.util.getUrl()
Resolved	Fomafix	T103661 Reimplement Sanitizer::escapeId in JavaScript

Event Timeline

Fomafix created this task.Jun 23 2015, 4:02 PM

Fomafix raised the priority of this task from to Needs Triage.

Fomafix updated the task description. (Show Details)

Fomafix added a project: JavaScript.

Fomafix subscribed.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 23 2015, 4:02 PM

Fomafix added a subtask: T103661: Reimplement Sanitizer::escapeId in JavaScript.Jun 24 2015, 8:33 AM

He7d3r added a project: good first task.Jun 24 2015, 3:14 PM

He7d3r set Security to None.

Fomafix added a parent task: T103715: Support fragments in getUrl() of mw.Title.Jun 24 2015, 5:21 PM

Aklapper added a project: MediaWiki-General.Jun 24 2015, 5:44 PM

Fomafix updated the task description. (Show Details)Jun 24 2015, 8:42 PM

Fomafix mentioned this in T103661: Reimplement Sanitizer::escapeId in JavaScript.Jul 5 2015, 2:13 PM

Anomie closed subtask T103661: Reimplement Sanitizer::escapeId in JavaScript as Declined.Jul 13 2015, 5:26 PM

Fomafix reopened subtask T103661: Reimplement Sanitizer::escapeId in JavaScript as Open.Sep 2 2015, 4:24 AM

Ricordisamoa closed subtask T103661: Reimplement Sanitizer::escapeId in JavaScript as Declined.Oct 7 2015, 7:41 AM

I'm going to propose (and mentor) this as a task for Google-Code-In-2015.

matmarex triaged this task as Medium priority.Oct 27 2015, 9:51 PM

Imported as https://codein.withgoogle.com/dashboard/tasks/5656198728646656/

Aklapper moved this task from Proposed to Imported into GCI site on the Google-Code-In-2015 board.Dec 5 2015, 4:35 PM

I've claimed this on the GCI website.

Change 260217 had a related patch set uploaded (by Unicornisaurous):
Support fragments in mw.util.getUrl()

https://gerrit.wikimedia.org/r/260217

gerritbot added a project: Patch-For-Review.Dec 19 2015, 11:01 PM

Change 260217 merged by jenkins-bot:
Support fragments in mw.util.getUrl()

https://gerrit.wikimedia.org/r/260217

ReleaseTaggerBot added projects: MW-1.27-release (WMF-deploy-2016-01-12_(1.27.0-wmf.10)), MW-1.27-release-notes.Dec 20 2015, 8:00 PM

Change 260328 had a related patch set uploaded (by Gerrit Patch Uploader):
mediawiki.util: Use correct encoding for fragment in getUrl

https://gerrit.wikimedia.org/r/260328

matmarex closed this task as Resolved.Dec 21 2015, 4:19 PM

matmarex removed a project: Patch-For-Review.

matmarex changed the status of subtask T103661: Reimplement Sanitizer::escapeId in JavaScript from Declined to Resolved.Dec 21 2015, 4:25 PM

Change 260328 merged by jenkins-bot:
mediawiki.util: Use correct encoding for fragment in getUrl