Page MenuHomePhabricator

VisualEditor: ISBN link labels (from magic words) can be changed, but not the underlying links
Closed, ResolvedPublic8 Estimated Story Points

Description

Intention:
Change an ISBN.

Steps to Reproduce:

  1. Open a page with a pre-existing ISBN number (using the magic word).
  1. Click in the link and open the link inspector.
  1. Correct the numbers in the ISBN.
  1. Close the link inspector.

Actual Results:
The link label does not change. This is not surprising, given VisualEditor's approach to links, but it is sometimes unfortunate; see https://fr.wikipedia.org/w/index.php?title=Massimo_Introvigne&diff=100272017&oldid=94374615

However, the link itself does not change, and if you have done nothing else, it is not possible to save the page, because you have supposedly made no changes.

Expected Results:
Ideal behavior:

Changing "ISBN 1234567890" to "ISBN 1234567891" simply updates the plain wikitext magic word.

Possibly acceptable behavior:

Changing "ISBN 1234567890" to "ISBN 1234567891" updates the link to [[Special:BookSources/1234567891|ISBN 1234567891]].

Reproducible: Always


Version: unspecified
Severity: normal
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=52204

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 3:06 AM
bzimport set Reference to bz61558.

Other recent examples of ISBN problems caused by VE (awful syntax, link incoherent with displayed ISBN, nowiki added) :
https://fr.wikipedia.org/w/index.php?title=Bassin_s%C3%A9dimentaire&diff=prev&oldid=104307942
https://fr.wikipedia.org/w/index.php?title=Louis-Jos%C3%A9_Houde&diff=104782988&oldid=104262388

This is a recurring problem on frwiki.

Jdforrester-WMF lowered the priority of this task from Medium to Low.Apr 15 2015, 6:52 PM
Jdforrester-WMF edited a custom field.

The Parsoid output does not look quite right to me:

<a href="./Special:BookSources/0345318641" rel="mw:ExtLink" data-parsoid='{"stx":"magiclink","dsr":[16,34,0,0]}'>ISBN 0-345-31864-1</a>

ExtLink? That href is for a WikiLink...

@ssastry, are the problems described in this comment also possibly due to Parsoid? Thank you!

@ssastry, are the problems described in this comment also possibly due to Parsoid? Thank you!

Looking at [[Special:BookSources/9782981011206|ISBN 978-2-9810112-1-]]<nowiki/>3 .. that looks like VE issues to me. The "3" is not part of the link text so, Parsoid is forced to add a nowiki to preserve HTML semantics. As for editing link text not chaning the underlying link, that is also a VE UX issue.

@Krenair it's mostly orthogonal to this bug, but we just changed the rel type of ISBN links to mw:WikiLink. I agree that it is more consistent that way.

The VE UX issue still exists: VE should detect edits to ISBN/RFC/PMID links and somehow keep the text and the link in sync. I don't know the best way to do that within the VE codebase.

Change 233669 had a related patch set uploaded (by Cscott):
WIP: Specialized inspector for ISBN magic links

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

Change 233669 had a related patch set uploaded (by Cscott):
Specialized inspector for ISBN magic links

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

Change 233669 merged by jenkins-bot:
Specialized inspector for ISBN magic links

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