From 5389bcf4fb587f10461439a06bb845a7101232c5 Mon Sep 17 00:00:00 2001
From: neverland <chenjiahan@youzan.com>
Date: Fri, 20 Dec 2019 20:39:29 +0800
Subject: [PATCH] fix(ImagePreview): onClose should only trigger once (#5341)

---
 src/image-preview/index.js           |  1 +
 src/image-preview/test/index.spec.js | 20 ++++++++++++++++++++
 2 files changed, 21 insertions(+)

diff --git a/src/image-preview/index.js b/src/image-preview/index.js
index 08742fe30..74101998a 100644
--- a/src/image-preview/index.js
+++ b/src/image-preview/index.js
@@ -54,6 +54,7 @@ const ImagePreview = (images, startPosition = 0) => {
   });
 
   if (options.onClose) {
+    instance.$off('close');
     instance.$once('close', options.onClose);
   }
 
diff --git a/src/image-preview/test/index.spec.js b/src/image-preview/test/index.spec.js
index 727970b0c..93c144bd7 100644
--- a/src/image-preview/test/index.spec.js
+++ b/src/image-preview/test/index.spec.js
@@ -102,6 +102,26 @@ test('onClose option', async done => {
   done();
 });
 
+test('onClose should only trigger once', async done => {
+  const onClose = jest.fn();
+  const instance = ImagePreview({
+    images,
+    startPostion: 1,
+    onClose
+  });
+
+  ImagePreview({
+    images,
+    startPostion: 1,
+    onClose
+  });
+
+  instance.close();
+
+  expect(onClose).toHaveBeenCalledTimes(1);
+  done();
+});
+
 test('onChange option', async done => {
   const instance = ImagePreview({
     images,