Page MenuHomePhabricator

oo.EmitterList: moveItem() is broken when moving an item to a lower index
Closed, ResolvedPublic

Event Timeline

Change 348310 had a related patch set uploaded (by AndyRussG):
[oojs/core@master] EmitterList: Fix moving an item to a lower index

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

The new test case in above patch shows how to reproduce... :)

Krinkle triaged this task as High priority.Apr 17 2017, 7:53 PM
Krinkle moved this task from Inbox to Confirmed Problem on the OOjs board.

Change 348310 merged by jenkins-bot:
[oojs/core@master] EmitterList: Fix moving an item to a lower index

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

Krinkle assigned this task to Mooeypoo.
Krinkle moved this task from Confirmed Problem to Done on the OOjs board.