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

Change subject: ce.MWTransclusionNode: make sure model exists before getting 
its document
......................................................................

ce.MWTransclusionNode: make sure model exists before getting its document

Bug: T168932
Change-Id: I8d458888ed0b80205ce17b41f3506410c550630f
---
M modules/ve-mw/ce/nodes/ve.ce.MWTransclusionNode.js
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor 
refs/changes/29/367429/1

diff --git a/modules/ve-mw/ce/nodes/ve.ce.MWTransclusionNode.js 
b/modules/ve-mw/ce/nodes/ve.ce.MWTransclusionNode.js
index 0515721..8542ecc 100644
--- a/modules/ve-mw/ce/nodes/ve.ce.MWTransclusionNode.js
+++ b/modules/ve-mw/ce/nodes/ve.ce.MWTransclusionNode.js
@@ -189,7 +189,7 @@
        // Parent method
        var elements = 
ve.ce.GeneratedContentNode.prototype.getRenderedDomElements.apply( this, 
arguments );
 
-       if ( this.getModelHtmlDocument() ) {
+       if ( this.model && this.getModelHtmlDocument() ) {
                ve.init.platform.linkCache.styleParsoidElements(
                        $( elements ),
                        this.getModelHtmlDocument()

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8d458888ed0b80205ce17b41f3506410c550630f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: DLynch <[email protected]>

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

Reply via email to