Merge pull request #506 from akashic-games/fix-destroyed-check-for-e

Fix destroyed check for E

Author: ShinobuTakahashi

CHANGELOG.md

@@ -1,5 +1,8 @@
 # ChangeLog
 
+## 3.18.3
+* `g.E#destroy()` で破棄済みチェックを行うように修正
+
 ## 3.18.2
 * `OperationHandler#onOperaiton()` で `instanceof` での判定をやめ `Array.isArray()` を利用するように修正
 

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@akashic/akashic-engine",
-  "version": "3.18.2",
+  "version": "3.18.3",
   "description": "The core library of Akashic Engine",
   "main": "index.js",
   "dependencies": {

package.json

@@ -360,6 +360,19 @@ describe("test E", () => {
 		expect((e as any)._onPointUp).toBeUndefined();
 	});
 
+	it("destroy - check for multiple executions", () => {
+		const e2 = new E({ scene: runtime.scene });
+		const spy = jest.spyOn(e2.scene, "unregister");
+
+		e2.destroy();
+		expect(e2.destroyed()).toBeTruthy();
+		expect(spy.mock.calls.length).toBe(1);
+
+		e2.destroy();
+		expect(spy.mock.calls.length).toBe(1); // 破棄済みの場合、 destory() を実行しても呼ばれないのでカウントは増えない。
+		spy.mockClear();
+	});
+
 	it("modified", () => {
 		resetUpdated(runtime);
 		expect(runtime.game._modified).toBe(false);

src/__tests__/ESpec.ts

@@ -513,6 +513,8 @@ export class E extends Object2D implements CommonArea {
 	 * 子孫を持っている場合、子孫も破棄される。
 	 */
 	destroy(): void {
+		if (this.destroyed()) return;
+
 		if (this.parent) this.remove();
 
 		if (this.children) {