Skip to content
This repository has been archived by the owner on May 30, 2022. It is now read-only.

Flock Control Panel Doesn't Clear Dead Enemies #140

Open
amylizzle opened this issue Apr 10, 2022 · 5 comments
Open

Flock Control Panel Doesn't Clear Dead Enemies #140

amylizzle opened this issue Apr 10, 2022 · 5 comments
Labels
bug Something isn't working

Comments

@amylizzle
Copy link
Contributor

Labels

[BUG]

Description

If an enemy is dead/destroyed, it remains in the enemy list

Steps to reproduce

  1. Designate someone/something as enemy
  2. Kill it
  3. It's still an enemy
  4. Destroy it's body
  5. it's still an enemy and jumping to it causes a runtime

Expected Behavior

Dead things should be removed

Additional Information & Screenshots

No response

@keywordlabeler keywordlabeler bot added the bug Something isn't working label Apr 10, 2022
@TobleroneSwordfish
Copy link
Contributor

I'll wait until #123 is merged to try fix this, or it'll merge conflict horribly

@FlameArrow57
Copy link
Contributor

FlameArrow57 commented Apr 11, 2022

Should it remove dead enemies though? Not sure what Flock perspective is on human/etc. death, but if someone was to be cloned, they wouldn't be marked as an enemy.

@amylizzle
Copy link
Contributor Author

Should it remove dead enemies though? Not sure what Flock perspective is on human/etc. death, but if someone was to be cloned, they wouldn't be marked as an enemy.

That is a consideration, but some concessions should be made for the sake of usability imo. More specifically, the enemies panel ought to tell the player about active threats, not dormant ones. Also when the body is entirely destroyed it runtimes so that at least needs to be handled.

@TobleroneSwordfish
Copy link
Contributor

Currently I think drones will only attempt to cage a dead enemy if it's still marked as such, so it needs to persist through death. Could easily add a (dead) marker to the flock panel though.

@amylizzle
Copy link
Contributor Author

There's still a bug with this where you can jump to things that have a null loc and it causes a runtime.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants