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

Elevator Gibs Upwards #4269

Closed
2 of 5 tasks
tiramisuapimancer opened this issue Jul 29, 2024 · 4 comments
Closed
2 of 5 tasks

Elevator Gibs Upwards #4269

tiramisuapimancer opened this issue Jul 29, 2024 · 4 comments
Labels
bug Something isn't working could not reproduce Bug did not reoccur during testing.

Comments

@tiramisuapimancer
Copy link

tiramisuapimancer commented Jul 29, 2024

Description of issue

Classic Baycode serpentid-eviscerating issue where elevators will gib things when moving upwards. Except it might be related to #4268 and issues with serpentids being unable to move or fall downwards.

Notably my gibs were placed on the below z-level (on tiles they couldn't have reasonably "fallen" onto after my gibbing on the above Z), although I never myself moved to that below level and when I was ghosted due to the gibbing I remained on the tile and z level I had been on before gibbing.

Difference between expected and actual behavior

You should only be gibbed by moving floors if you are below them.

Steps to reproduce

Be floating on empty space above an elevator platform when it moves upwards. Become bits and pieces

Length of time in which bug has been known to occur

Client version, Server revision & Game ID

Client Version: 515
Server Revision: 65f73b4 - dev - 2024-07-28
was on a friend's local
Current map: Tradeship Ocelot Alpha

Issue bingo

  • Issue could be reproduced at least once
  • Issue could be reproduced by different players
  • Issue could be reproduced in multiple rounds
  • Issue happened in a recent (less than 7 days ago) round
  • Couldn't find an existing issue about this
@tiramisuapimancer
Copy link
Author

floating adherent was not gibbed by upward elevator movement

@tiramisuapimancer
Copy link
Author

reproducing issue proving difficult even using further serpentids

@MistakeNot4892 MistakeNot4892 added the bug Something isn't working label Jul 29, 2024
@MistakeNot4892
Copy link
Contributor

	if(!moving_upwards)
		for(var/turf/T in destination)
			for(var/atom/movable/AM in T)
				if(isliving(AM))
					var/mob/living/M = AM
					M.gib()
				else if(AM.simulated)
					qdel(AM)

I suspect the elevator is not what gibbed you in your repro case, at least not directly. Might be some kind of weird input issue or delayed move up or down from fall() as a knock-on effect from serpentid flight being busted per your other issue.

@MistakeNot4892 MistakeNot4892 added the could not reproduce Bug did not reoccur during testing. label Jul 30, 2024
@MistakeNot4892
Copy link
Contributor

Going to close this for now as the issue as described is explicitly gated, I suspect this will be tied to whatever is going on with Serpentid movement in general. Will reopen if it recurs after that is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working could not reproduce Bug did not reoccur during testing.
Projects
None yet
Development

No branches or pull requests

2 participants