[bugfix] Dialog: beforeClose not work for click overlay (#2707)

packages/dialog/Dialog.js

@@ -41,8 +41,13 @@ export default sfc({
   },
 
   methods: {
+    onClickOverlay() {
+      this.handleAction('overlay');
+    },
+
     handleAction(action) {
       this.$emit(action);
+
       if (this.beforeClose) {
         this.loading[action] = true;
         this.beforeClose(action, state => {
@@ -57,8 +62,11 @@ export default sfc({
     },
 
     onClose(action) {
-      this.$emit('input', false);
+      this.close();
-      this.callback && this.callback(action);
+
+      if (this.callback) {
+        this.callback(action);
+      }
     }
   },
 

packages/dialog/test/index.spec.js

@@ -1,7 +1,7 @@
 import Vue from 'vue';
 import Dialog from '..';
 import DialogVue from '../Dialog';
-import { mount, later, transitionStub } from '../../../test/utils';
+import { mount, later, trigger, transitionStub } from '../../../test/utils';
 
 transitionStub();
 
@@ -39,6 +39,7 @@ test('before close', () => {
     propsData: {
       value: true,
       showCancelButton: true,
+      closeOnClickOverlay: true,
       beforeClose: (action, done) => done(false)
     }
   });
@@ -49,10 +50,19 @@ test('before close', () => {
   expect(wrapper.emitted('input')).toBeFalsy();
 
   wrapper.setProps({
-    beforeClose: (action, done) => done()
+    beforeClose: (action, done) => {
+      if (action === 'cancel') {
+        done();
+      }
+    }
   });
+
+  const overlay = document.querySelector('.van-overlay');
+  trigger(overlay, 'click');
+  expect(wrapper.emitted('input')).toBeFalsy();
+
   cancel.trigger('click');
-  expect(wrapper.emitted('input')).toBeTruthy();
+  expect(wrapper.emitted('input')[0]).toBeTruthy();
 });
 
 test('set default options', () => {

packages/mixins/popup/manager.js

@@ -63,7 +63,14 @@ export default {
     if (context.top) {
       const { vm } = context.top;
       vm.$emit('click-overlay');
-      vm.closeOnClickOverlay && vm.$emit('input', false);
+
+      if (vm.closeOnClickOverlay) {
+        if (vm.onClickOverlay) {
+          vm.onClickOverlay();
+        } else {
+          vm.close();
+        }
+      }
     }
   }
 };