HitboxComponent on an object with a CarryableComponent

[ac-arcana]

Anyone know how to combine these two components? The Carryable Component must be a rigidbody, but the HitboxComponent IS a rigidbody....

Placing the HitboxComponent on a plain Node3D doesn't seem to work. I can't place it on the parent object of the item because it is a Cogito_Object already.

The collider needs to be a HitboxComponent to deal damage, but I wont be able to pick it up too if its not a CogitoObject.

[ac-arcana]

I made this work with the laser now by changing the hit_scan_damage function to look for children.

func hit_scan_damage(collider):
if collider is HitboxComponent:
collider.damage(item_reference.wieldable_damage)
else:
var hitbox_component_array = collider.find_children("", "HitboxComponent")
if hitbox_component_array:
hitbox_component_array[0].damage(item_reference.wieldable_damage)

I'm not sure if there was a different way you intended to support using these components together @Phazorknight ?

[Phazorknight]

Looking into this now, but not sure I'll be able to fix it today.
The current implementation is definitely not ideal so I might redo it.

The idea is to make the HitboxComponent work by just droping into your objects like any of the interactions or the ImpactSounds.
It will still always require the HealthAttribute component.

[ac-arcana]

If its meant to be a component dropped onto the object, then it might be best to just look for child hitbox components when dealing damage.
The way it is now requires the hitbox component to be on the physics body.
I didn't honestly check to see if CogitoObject can be a child on the object though?

[Phazorknight]

CogitoObjects should always be the root of the object PackedScene as they are being used to save. I haven't really tried it but I'd assume that nesting CogitoObjects inside other PackedScene could cause unexpected behaviour, especially for scene persistence / saving.

[Phazorknight]

Ok, I've implemented this in 6f5930b

The commit notes should mention how it works, but it's pretty straight forward. Simply attach it to a CogitoObject (it HAS to be a cogito object) and make sure it has a reference to the object's HealthAttribute. This should make it compatible with all other components, for example the red barrel in the Lab scene can be carried around and still works with the HitboxComponent.

All current damage dealers have been updated to work this way. This means currently these can only damage CogitoObjects.

[ac-arcana]

Awesome you did this so fast! Thanks for the update!

[ac-arcana]

Closing