Page MenuHomePhabricator

[Regression wmf13] Adding an link with the link inspector in insertion mode inserts an anchor with "/wiki/" prepended
Closed, ResolvedPublic1 Estimated Story Points

Description

Steps to reproduce:

1.Open the ink inspector
2.Search for an article
3.Select an article to add it in the page say Latin

Observed Result:
On the page its getting added as '/wiki/Latin'

See the screenshot attached

Event Timeline

Ryasmeen raised the priority of this task from to Needs Triage.
Ryasmeen updated the task description. (Show Details)
Ryasmeen added a project: VisualEditor.
Ryasmeen added a subscriber: Ryasmeen.
Ryasmeen renamed this task from [Regression pre-wmf13] Article URL with namespace '/wiki/' is getting added in the page while adding it from the link inspector to [Regression wmf13] Article URL with namespace '/wiki/' is getting added in the page while adding it from the link inspector.Jul 8 2015, 2:52 PM
Ryasmeen set Security to None.
Jdforrester-WMF renamed this task from [Regression wmf13] Article URL with namespace '/wiki/' is getting added in the page while adding it from the link inspector to [Regression wmf13] Adding an link with the link inspector in insertion mode inserts an anchor with "/wiki/" prepended.Jul 8 2015, 3:38 PM
Jdforrester-WMF triaged this task as High priority.
Jdforrester-WMF added a subscriber: Krenair.

Change 223707 had a related patch set uploaded (by Alex Monk):
Bypass ve.ui.MWInternalLinkAnnotationWidget.prototype.getHref when determining text

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

We should probably be avoiding getHref in ve.ui.MWLinkAnnotationInspector.prototype.getInsertionText for internal links, but the uploaded patch should still fix the immediate issue.

Change 223707 merged by jenkins-bot:
Bypass ve.ui.MWInternalLinkAnnotationWidget.prototype.getHref when determining text

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

Change 223983 had a related patch set uploaded (by Alex Monk):
Bypass ve.ui.MWInternalLinkAnnotationWidget.prototype.getHref when determining text

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

Change 223983 merged by jenkins-bot:
Bypass ve.ui.MWInternalLinkAnnotationWidget.prototype.getHref when determining text

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

I backported and deployed that commit to fix production.