Mooeypoo has uploaded a new change for review.

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

Change subject: Verify welcome dialog exists before trying to close it
......................................................................

Verify welcome dialog exists before trying to close it

When deactivating, verify first that the welcome dialog was
initialized before calling 'close' on it.

Change-Id: Ife98b396f3d8641e2bd313c549fe867bcd84db9b
---
M modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js
1 file changed, 3 insertions(+), 1 deletion(-)


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

diff --git a/modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js 
b/modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js
index 72144e6..078d506 100644
--- a/modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js
+++ b/modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js
@@ -342,7 +342,9 @@
        }
 
        // Just in case this wasn't closed before
-       this.welcomeDialog.close();
+       if ( this.welcomeDialog ) {
+               this.welcomeDialog.close();
+       }
 
        if ( noDialog || this.activating || !this.edited ) {
                this.cancel( trackMechanism );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ife98b396f3d8641e2bd313c549fe867bcd84db9b
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Mooeypoo <mor...@gmail.com>

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

Reply via email to