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

Fixes revive() causing you to be unable to shoot at unseen tiles #15860

Closed
wants to merge 1 commit into from

Conversation

Notamaniac
Copy link
Contributor

@Notamaniac Notamaniac commented May 14, 2024

About The Pull Request

Allows mobs to shoot at unseen tiles when revive() gets called on them. I'm going to admit, this was more of a game of 'call the missing proc that wasn't called in login()'. I don't fully understand why this works, and my only guess that is that the screenmob.client.screen = list() call in the show_hud() deletes these voids that can be shot at, and that calling add_click_catcher() readds them. This is me talking out of my ass though. No immediate bugs from testing though.

Fixes #15550

Why It's Good For The Game

You no longer need to reconnect when leaving robotic cradles or entering EORD from the round ending to shoot at unseen tiles.

Changelog

🆑
fix: Admin revives, the robotic cradle, and entering EORD via round end prefs should no longer cause you to be unable to shoot at unseen tiles
/:cl:

@tgstation-server tgstation-server added the Fix Fixes an issue with the game. label May 14, 2024
Copy link
Member

@TiviPlus TiviPlus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wierd needs to be fixed properly later

@Lumipharon Lumipharon mentioned this pull request May 14, 2024
@Lumipharon
Copy link
Contributor

I found the root issue so gonna close this in favour of #15864 but thank you for narrowing down to exactly where the issue was.

@Lumipharon Lumipharon closed this May 14, 2024
@Notamaniac Notamaniac deleted the shooting_void_fix branch July 2, 2024 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fixes an issue with the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't shoot at not-visible tiles after using robot cradle
4 participants