feat: onComponentTypeCheck support for ShapeHitbox

packages/flame/lib/src/collisions/broadphase/quadtree/has_quadtree_collision_detection.dart

@@ -68,16 +68,16 @@ mixin HasQuadTreeCollisionDetection on FlameGame
 
  bool onComponentTypeCheck(PositionComponent one, PositionComponent another) {
  var checkParent = false;
- if (one is CollisionCallbacks) {
- if (!(one as CollisionCallbacks).onComponentTypeCheck(another)) {
+ if (one is GenericCollisionCallbacks) {
+ if (!(one as GenericCollisionCallbacks).onComponentTypeCheck(another)) {
  return false;
  }
  } else {
  checkParent = true;
  }
 
- if (another is CollisionCallbacks) {
- if (!(another as CollisionCallbacks).onComponentTypeCheck(one)) {
+ if (another is GenericCollisionCallbacks) {
+ if (!(another as GenericCollisionCallbacks).onComponentTypeCheck(one)) {
  return false;
  }
  } else {

packages/flame/lib/src/collisions/collision_callbacks.dart

@@ -74,6 +74,14 @@ mixin GenericCollisionCallbacks<T> {
  onCollisionEndCallback?.call(other);
  }
 
+ /// Works only for the QuadTree collision detection.
+ /// If you need to prevent collision of items of different types -
+ /// reimplement [onComponentTypeCheck]. The result of calculation is cached
+ /// so you should not check any dynamical parameters here, the function
+ /// intended to be used as pure type checker.
+ @mustCallSuper
+ bool onComponentTypeCheck(PositionComponent other);
+
  /// Assign your own [CollisionCallback] if you want a callback when this
  /// shape collides with another [T].
  CollisionCallback<T>? onCollisionCallback;
@@ -127,11 +135,7 @@ mixin CollisionCallbacks on Component
  onCollisionEndCallback?.call(other);
  }
 
- /// Works only for the QuadTree collision detection.
- /// If you need to prevent collision of items of different types -
- /// reimplement [onComponentTypeCheck]. The result of calculation is cached
- /// so you should not check any dynamical parameters here, the function
- /// intended to be used as pure type checker.
+ @override
  @mustCallSuper
  bool onComponentTypeCheck(PositionComponent other) {
  final myParent = parent;

packages/flame/lib/src/collisions/hitboxes/shape_hitbox.dart

@@ -213,6 +213,18 @@ mixin ShapeHitbox on ShapeComponent implements Hitbox<ShapeHitbox> {
  }
  }
 
+ @override
+ @mustCallSuper
+ bool onComponentTypeCheck(PositionComponent other) {
+ final myParent = parent;
+ final otherParent = other.parent;
+ if (myParent is CollisionCallbacks && otherParent is PositionComponent) {
+ return myParent.onComponentTypeCheck(otherParent);
+ }
+
+ return true;
+ }
+
  @override
  CollisionCallback<ShapeHitbox>? onCollisionCallback;