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

Fix furniture door pathfinding #34720

Merged
merged 1 commit into from
Oct 14, 2019
Merged

Fix furniture door pathfinding #34720

merged 1 commit into from
Oct 14, 2019

Conversation

KnightosMaster
Copy link
Contributor

Summary

SUMMARY: Bugfixes "NPCs now properly open furniture doors"

Purpose of change

Fixes #34646 - NPCs unable to open tent flaps

Describe the solution

Modified pathfinding to consider tiles with openable furniture. Also added a check to NPC movement so they won't phase through doors, and instead try to open them.

Testing

Closed door in front of friendly npcs to see if the opened it properly (Tried with both furniture and terrain doors).
Closed door in front of zed to see if they tried to find their way through to me.

Additional context

The canvas flap was the only furniture door with impassable properties (f_canvas_door and f_large_canvas_door). Apart from this, the only doors are: f_counter_gate_c and f_beaded_door. Should the also be impassable? Or should the canvas flap not be impassable. Either way, npcs now properly open them regardless, but I'm not sure how this plays with the pathfinding of monsters.

@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` Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. NPC / Factions NPCs, AI, Speech, Factions, Ownership labels Oct 13, 2019
@ZhilkinSerg ZhilkinSerg merged commit cb7938e into CleverRaven:master Oct 14, 2019
@KnightosMaster KnightosMaster deleted the fix_furn_door_pathf branch October 14, 2019 18:37
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` Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. NPC / Factions NPCs, AI, Speech, Factions, Ownership
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NPCs unable to open tent flaps
2 participants