Page MenuHomePhabricator

Reference diffs not working correctly on historical diffs
Closed, ResolvedPublic8 Estimated Story Points

Description

The following simple test case throws an exception, trying to access the internal item with index 2:

image.png (222×824 px, 21 KB)

Event Timeline

The current version of the visual diff makes a lot of assumptions about the internal list being append-only, i.e. that document B's internalList is a superset of document A, however if parse the documents separately from the server that is not the case.

Deskana moved this task from To Triage to TR6: Visual diffs on the VisualEditor board.

I also imagine this is quite a lot of work.

Change 371128 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (81a4760ea)

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

Jdforrester-WMF removed a project: Patch-For-Review.
Jdforrester-WMF set the point value for this task to 8.

Change 371128 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (81a4760ea)

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