Esanders has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/349226 )

Change subject: Only remove from internal list if unroot from correct document
......................................................................

Only remove from internal list if unroot from correct document

Bug: T163267
Change-Id: I11ff56fbd437e46aab516a50235f8077eea0af5b
---
M modules/ve-cite/ve.dm.MWReferenceNode.js
1 file changed, 6 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Cite 
refs/changes/26/349226/1

diff --git a/modules/ve-cite/ve.dm.MWReferenceNode.js 
b/modules/ve-cite/ve.dm.MWReferenceNode.js
index 4da71c9..172573b 100644
--- a/modules/ve-cite/ve.dm.MWReferenceNode.js
+++ b/modules/ve-cite/ve.dm.MWReferenceNode.js
@@ -365,9 +365,13 @@
 
 /**
  * Handle the node being detached from the root
+ *
+ * @param {ve.dm.DocumentNode} oldRoot Old document root
  */
-ve.dm.MWReferenceNode.prototype.onUnroot = function () {
-       this.removeFromInternalList();
+ve.dm.MWReferenceNode.prototype.onUnroot = function ( oldRoot ) {
+       if ( this.getDocument().getDocumentNode() === oldRoot ) {
+               this.removeFromInternalList();
+       }
 };
 
 /**

-- 
To view, visit https://gerrit.wikimedia.org/r/349226
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I11ff56fbd437e46aab516a50235f8077eea0af5b
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Cite
Gerrit-Branch: master
Gerrit-Owner: Esanders <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to