the logic is consistent with box2d-jsb & box2d (#93)

* the logic is consistent with box2d-jsb & box2d

* Add return type to register/unregisterContactFixture, fix lint

Author: tangkaikk

cocos/physics-2d/box2d-wasm/physics-world.ts

@@ -82,6 +82,16 @@ export class B2PhysicsWorld implements IPhysicsWorld {
         PhysicsContactListener._PreSolve = this._onPreSolve;
         PhysicsContactListener._PostSolve = this._onPostSolve;
         this._contactListener = B2.ContactListener.implement(PhysicsContactListener.callback);
+        const orgRegisterContactFixture = this._contactListener.registerContactFixture.bind(this._contactListener);
+        this._contactListener.registerContactFixture = (contactFixture: number): void => {
+            PhysicsContactListener.registerContactFixture(contactFixture);
+            orgRegisterContactFixture(contactFixture);
+        };
+        const orgUnregisterContactFixture = this._contactListener.unregisterContactFixture.bind(this._contactListener);
+        this._contactListener.unregisterContactFixture = (contactFixture: number): void => {
+            PhysicsContactListener.unregisterContactFixture(contactFixture);
+            orgUnregisterContactFixture(contactFixture);
+        };
         this._world.SetContactListener(this._contactListener);
 
         this._aabbQueryCallback = B2.QueryCallback.implement(PhysicsAABBQueryCallback.callback);

cocos/physics-2d/box2d-wasm/platform/physics-contact-listener.ts

@@ -22,37 +22,56 @@
  THE SOFTWARE.
 */
 
+import { B2 } from '../instantiated';
+
 type BeginContactCallback = (contact: number) => void;
 type EndContactCallback = (contact: number) => void;
 type PreSolveCallback = (contact: number, oldManifold: number) => void;
 type PostSolveCallback = (contact: number, impulse: number) => void;
 
 export class PhysicsContactListener {
+    static _contactFixtures: Set<number> = new Set<number>();
     static _BeginContact: BeginContactCallback | null = null;
     static _EndContact: EndContactCallback | null = null;
     static _PreSolve: PreSolveCallback | null = null;
     static _PostSolve: PostSolveCallback | null = null;
+    static _shouldReportContacts: Set<number> = new Set<number>();
 
     static BeginContact (contact: number): void {
-        if (this._BeginContact) {
+        if (!this._BeginContact) return;
+        const fixtureA = B2.ContactGetFixtureA(contact) as number;
+        const fixtureB = B2.ContactGetFixtureB(contact) as number;
+        const fixtures = this._contactFixtures;
+        this._shouldReportContacts.delete(contact);
+
+        if (fixtures.has(fixtureA) || fixtures.has(fixtureB)) {
+            this._shouldReportContacts.add(contact);
             this._BeginContact(contact);
         }
     }
 
     static EndContact (contact: number): void {
-        if (this._EndContact) {
+        if (this._EndContact && this._shouldReportContacts.has(contact)) {
+            this._shouldReportContacts.delete(contact);
             this._EndContact(contact);
         }
     }
 
     static PreSolve (contact: number, oldManifold: number): void {
-        if (this._PreSolve) {
+        if (this._PreSolve && this._shouldReportContacts.has(contact)) {
             this._PreSolve(contact, oldManifold);
         }
     }
+    static registerContactFixture (fixture: number): void {
+        this._contactFixtures.add(fixture);
+    }
+
+    static unregisterContactFixture (fixture: number): void {
+        this._contactFixtures.delete(fixture);
+    }
 
     static PostSolve (contact: number, impulse: number): void {
-        if (this._PostSolve) {
+        if (this._PostSolve && this._shouldReportContacts.has(contact)) {
             this._PostSolve(contact, impulse);
         }
     }