Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Card readers only despawn M249 turrets but labs can spawn with antimaterial turrets #36415

Closed
Amoebka opened this issue Dec 24, 2019 · 4 comments · Fixed by #36462
Closed

Card readers only despawn M249 turrets but labs can spawn with antimaterial turrets #36415

Amoebka opened this issue Dec 24, 2019 · 4 comments · Fixed by #36462
Labels
<Bug> This needs to be fixed Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. Monsters Monsters both friendly and unfriendly.
Milestone

Comments

@Amoebka
Copy link
Contributor

Amoebka commented Dec 24, 2019

Describe the bug

When a player uses an ID card on a card reader, turrets on the same overmap are despawned. However, the code only checks for the hardcoded mon_turret_rifle type, which only covers M249 turrets. Labs can sometimes spawn with .50 cal turrets as guards instead (not sure why?), and those are NOT despawned by card readers when the doors are opened.

Steps To Reproduce

Steps to reproduce the behavior:

  1. Teleport around until you find a lab with .50 cal turret at the door.
  2. Swipe your ID on the reader.
  3. The turret isn't despawned.

Expected behavior

Card reader's iexamine checks for some kind of JSON flag all turrets have instead of a specific monster id when it despawns the security.

Versions and configuration

  • OS: win10
  • Game Version: 0.D-10626-gb9b3dc2
  • Graphics version: tiles
  • Mods loaded: none

Additional context

Are antimaterial turrets even supposed to spawn indoors?

@ymber
Copy link
Member

ymber commented Dec 24, 2019

M2HB turrets shouldn't be spawning there. The way turrets are despawned is a weird hack from ancient cata and it should be reworked.

@Night-Pryanik Night-Pryanik added <Bug> This needs to be fixed Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. Monsters Monsters both friendly and unfriendly. labels Dec 25, 2019
@kevingranade kevingranade added this to the 0.E milestone Dec 25, 2019
@kevingranade
Copy link
Member

We do need some kind of workaround for this, because the expectation is that if you enter the lab by using appropriate credentials, you at least don't get shot in the face by the door guard.

@ymber
Copy link
Member

ymber commented Dec 25, 2019

The quick bandaid fix would be to add the other turrets to the check in remove_submap_turrets.

@Night-Pryanik
Copy link
Contributor

The quick bandaid fix would be to add the other turrets to the check in remove_submap_turrets.

Simple checking the TURRET flag instead of harcoded mon_turret_rifle id should be enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. Monsters Monsters both friendly and unfriendly.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants