jenkins-bot has submitted this change and it was merged.

Change subject: TransactionProcessor: Fix offset bug
......................................................................


TransactionProcessor: Fix offset bug

Use ve.BranchNode.getNodeFromOffset instead of
ve.dm.Document.getNodeFromOffset so correct nodes will be retrieved to
emit update events to.


Change-Id: Iaf559f0424584a3dde065e548e403c4a53207312
---
M modules/ve/dm/ve.dm.TransactionProcessor.js
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Catrope: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/ve/dm/ve.dm.TransactionProcessor.js 
b/modules/ve/dm/ve.dm.TransactionProcessor.js
index 40d3b79..5c66493 100644
--- a/modules/ve/dm/ve.dm.TransactionProcessor.js
+++ b/modules/ve/dm/ve.dm.TransactionProcessor.js
@@ -346,7 +346,7 @@
        }
 
        this.synchronizer.pushAttributeChange(
-               this.document.getNodeFromOffset( this.cursor + 1 ),
+               this.document.documentNode.getNodeFromOffset( this.cursor + 1 ),
                op.key,
                from, to
        );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaf559f0424584a3dde065e548e403c4a53207312
Gerrit-PatchSet: 6
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Inez <i...@wikia-inc.com>
Gerrit-Reviewer: Catrope <roan.katt...@gmail.com>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to