(In reply to Ritu Swain from comment #0)

1> Open a page with at least three different citations(book, news,journal).
2>select the “basic” citation and use any existing reference. The reference
gets added.

What? We only have book, news and journal at this point... No basic...

(In reply to Alex Monk from comment #1)

(In reply to Ritu Swain from comment #0)

1> Open a page with at least three different citations(book, news,journal).
2>select the “basic” citation and use any existing reference. The reference
gets added.

What? We only have book, news and journal at this point... No basic...

My apologies for the confusion.Here are the steps-

1> Open a page with at least three different citations(book, news,journal).
2> select to add the “basic” citation and choose to use any existing reference. The reference gets added.
3> Now while the reference is still highlighted, select to add “Basic” citation again. When the dialog opens,type some text in “Use this Group” and apply changes.
5> Notice the citation is replaced by the text that was put in the “Use this Group” and is highlighted.
6> Now click to undo at least 2-3 times.
7> Notice now the tools in the editor toolbar behave oddly , for ex- link does not get added,the language text is not highlighted and lists do not get added.

Is this the same thing?

• Make a basic reference, apply changes
• Go back and give it a non-blank group, apply changes
• Undo a few times
• Uncaught TypeError: Cannot read property 'length' of undefined

(which is related to but not quite the same as bug 69125)

(In reply to Alex Monk from comment #3)

Is this the same thing?

• Make a basic reference, apply changes
• Go back and give it a non-blank group, apply changes
• Undo a few times
• Uncaught TypeError: Cannot read property 'length' of undefined

(which is related to but not quite the same as bug 69125)

in Chrome JS Console- i get this error

Uncaught TypeError: Cannot read property 'length' of undefined.
Error: Unbalanced input passed to document.

in FF JS Console error is-
TypeError:KeyedNodes is undefined
Error: Unbalanced input passed to document.

after this, the tools in the toolbar when accessed behave weirdly.

Yes, but is that the same issue you were talking about in this bug?

yes, it is the same issue.

Seems to be caused by the internal list not being (properly?) updated when undoing a group change (it stores references by group name).

• Bug 69125 has been marked as a duplicate of this bug. ***

I looked at this for a while but it's beyond my knowledge of the DM:
When you edit the reference's group, the reference disappears from the old node group (in InternalList) and reappears in the new node group. But after undoing, it seems to have remained there and not moved back.
I think ve.dm.MWReferenceModel.prototype.updateInternalItem handles it for manual changes, but it is not called on undo.

Change 195901 had a related patch set uploaded (by Alex Monk):
Handle undoing of reference group changes

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

Change 195901 merged by jenkins-bot:
Handle undoing of reference group changes

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