vgstation-coders / vgstation13

Butts
GNU Affero General Public License v3.0
266 stars 546 forks source link

Vaults don't always respect the bounds of vault areas when spawning #9370

Closed Probe1 closed 4 years ago

Probe1 commented 8 years ago

Description of issue

I removed a 60x60 chunk of vault10 on metaclubs z6. While testing my new area inside the 60x60 bounds, a syndicate vault spawned directly below it. The center of the vault was inside vault10 area but the top of it was well outside it and deleted a fair bit of my structure.

Difference between expected and actual behavior

When spawning a vault check to see if the edges are within bounds and to retry if not. Didn't check, spawned the top of a vault out of the vault10 area.

Steps to reproduce

Man this is such a bitch to reproduce that if I hadn't seen it happen while testing it wouldn't have been seen at all. Try deleting all vault areas and mapping in an area that is too small to fit a vault. That will probably provide a reproducible environ.

Specific information for locating

Length of time in which bug has been known to occur

Server revision

Issue bingo

Please check whatever applies. More checkboxes checked increase your chances of not being yelled at by every contributor.

PJB3005 commented 8 years ago

Yes, this is a problem when large vault spawn on the exact edge of the area.

The vault gets placed with the picked tile from the area being the bottom left, so you have to be careful.

Probe1 commented 5 years ago

Closed in https://github.com/vgstation-coders/vgstation13/pull/23838