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

Cannot remove NVG HUD insert from some helmet(s) #5799

Closed
3 tasks done
dadlips opened this issue Feb 24, 2024 · 2 comments · Fixed by #5804
Closed
3 tasks done

Cannot remove NVG HUD insert from some helmet(s) #5799

dadlips opened this issue Feb 24, 2024 · 2 comments · Fixed by #5804
Labels
Bug Admin events aren't bugs

Comments

@dadlips
Copy link

dadlips commented Feb 24, 2024

Testmerges

N/A

Round ID

21019

Description of the bug

When attempting to screwdriver some helmets to remove the HUD, the message about the slot being reserved for headgear accessories plays. I'm guessing this is something to do with #5573. I tried putting a helmet accessory into the slot, but could not and got the same message.

dreamseeker_poeEwPQg7t

What's the difference with what should have happened?

Screwdrivering a helmet should drop the HUD instead of trying to insert the screwdriver as an accessory

How do we reproduce this bug?

  1. Screwdriver a helmet to remove your HUD (not sure which ones are effected)
  2. You try to insert the screwdriver into the helmet instead of removing the HUD.

Issue Bingo

@dadlips dadlips added the Bug Admin events aren't bugs label Feb 24, 2024
@private-tristan
Copy link
Contributor

private-tristan commented Feb 24, 2024

MARSOC SOF helmets have inbuilt infinite NVG huds, not meant to be removable, the same way that squad optics can't be removed.

built_in_visors = list(new /obj/item/device/helmet_visor/night_vision/marine_raider, new /obj/item/device/helmet_visor/security)

@SabreML
Copy link
Member

SabreML commented Feb 24, 2024

MARSOC SOF helmets have inbuilt infinite NVG huds, not meant to be removable, the same way that squad optics can't be removed.

Yep, looks like this is what's happening. Not a bug, although I will make a PR to change the failure message when this happens since it's a bit confusing.

github-merge-queue bot pushed a commit that referenced this issue Feb 26, 2024
# About the pull request

Resolves #5799 by adding a failure message when trying to remove a
built-in helmet visor.

# Explain why it's good for the game

Slightly less confusing for players.

# Testing Photographs and Procedure
<details>
<summary>Screenshots & Videos</summary>

Put screenshots and videos here with an empty line between the
screenshots and the `<details>` tags.

</details>


# Changelog
:cl:
qol: Added a failure message when trying to remove a built-in helmet
visor.
/:cl:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Admin events aren't bugs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants