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

Allow Large and Huge creatures to move through underbrush #35285

Merged

Conversation

anothersimulacrum
Copy link
Member

Summary

SUMMARY: Bugfixes "Allow large and huge creatures to move through underbrush"

Purpose of change

#35060 prevents Large and Huge creatures from moving through obstacles with the THIN_OBSTACLE flag. This is incorrect behavior, because while large and huge creatures should not be able to move through some obstacles, these obstacles should not be selected with the THIN_OBSTACLE flag, because this flag has another purpose, and is fairly widely used, on other things, such as underbrush.

Describe the solution

Add a new flag, SMALL_PASSAGE, and use that instead.

Testing

Make a character, use enough bear mutagen to become large. Attempt to walk through a window frame, open window, and empty window - notice it doesn't let you move through. Attempt to walk over underbrush, and notice that you can.

PR 35060 changed it so that large and huge creatures could not move
through tight passages, using the THIN_OBSTACLE flag. This is incorrect,
because the THIN_OBSTACLE flag is widely used on things such as
underbrush, that large and huge creatures should be able to move on.
@kevingranade
Copy link
Member

This pull request has been mentioned on Cataclysm: Dark Days Ahead. There might be relevant details there:

https://discourse.cataclysmdda.org/t/uncomfortably-large-mutant-cant-fit-through-windows/21848/10

@ZhilkinSerg ZhilkinSerg added <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` [JSON] Changes (can be) made in JSON Monsters Monsters both friendly and unfriendly. labels Nov 3, 2019
@ZhilkinSerg ZhilkinSerg merged commit 54b00c6 into CleverRaven:master Nov 3, 2019
@anothersimulacrum anothersimulacrum deleted the fix-large-walking branch November 4, 2019 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` [JSON] Changes (can be) made in JSON Monsters Monsters both friendly and unfriendly.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants