From aa11fc6605d9eed1d1197f2e893f2c8a1dca9c8c Mon Sep 17 00:00:00 2001
From: Yomotsu <admin@yomotsu.net>
Date: Fri, 31 Dec 2021 21:31:33 +0900
Subject: [PATCH] fix: cancel emits controlend twice

---
 src/CameraControls.ts | 18 ++++++------------
 1 file changed, 6 insertions(+), 12 deletions(-)

diff --git a/src/CameraControls.ts b/src/CameraControls.ts
index 7b0c690..e0748bb 100644
--- a/src/CameraControls.ts
+++ b/src/CameraControls.ts
@@ -257,14 +257,6 @@ export class CameraControls extends EventDispatcher {
 			const lastDragPosition = new THREE.Vector2() as _THREE.Vector2;
 			const dollyStart = new THREE.Vector2() as _THREE.Vector2;
 
-			const cancelDragging = (): void => {
-
-				this._state = ACTION.NONE;
-				this._activePointers.length = 0;
-				endDragging();
-
-			};
-
 			const onPointerDown = ( event: PointerEvent ) => {
 
 				if ( ! this._enabled ) return;
@@ -785,9 +777,9 @@ export class CameraControls extends EventDispatcher {
 					this._domElement.ownerDocument.removeEventListener( 'touchmove', onTouchMove, { passive: false } as AddEventListenerOptions );
 					this._domElement.ownerDocument.removeEventListener( 'touchend', onTouchEnd );
 
-				}
+					this.dispatchEvent( { type: 'controlend' } );
 
-				this.dispatchEvent( { type: 'controlend' } );
+				}
 
 			};
 
@@ -823,9 +815,11 @@ export class CameraControls extends EventDispatcher {
 
 			this.cancel = (): void => {
 
-				cancelDragging();
+				if ( this._state === ACTION.NONE ) return;
 
-				this.dispatchEvent( { type: 'controlend' } );
+				this._state = ACTION.NONE;
+				this._activePointers.length = 0;
+				endDragging();
 
 			};