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

Tacoma Commune: make the ranch camp map unrotatable #34100

Merged
merged 1 commit into from
Sep 19, 2019

Conversation

mlangsdorf
Copy link
Contributor

Summary

SUMMARY: Bugfixes "Tacoma Commune: make the ranch camp map unrotatable"

Purpose of change

Addresses #33525

A fair bit of the Tacoma Commune missions still use hard coded C++ functions to update the map, and those functions do not understand map rotation at all. Completing those missions will cause all
kinds of weird errors as NPCs teleport around the map and new construction is built into old construction.

Make the ranch camp maps unable to rotate until I have time to go through and move the remaining Tacoma Commune missions into JSON.

Describe the solution

Set "rotation": false for the "ranch_camp" overmap special.

Describe alternatives you've considered

Moving all the remaining hardcoded mission starts to JSON would be preferable, but will also take several days of effort and I'd like to have a fix now.

I could try teaching all of the hardcoded mission start functions about rotation, but that would also take several days.

Addresses CleverRaven#33525

A fair bit of the Tacoma Commune missions still use hard coded C++
functions to update the map, and those functions do not understand
map rotation at all.  Completing those missions will cause all
kinds of weird errors as NPCs teleport around the map and new
construction is built into old construction.

Make the ranch camp maps unable to rotate until I have time to go
through and move the remaining Tacoma Commune missions into JSON.
@mlangsdorf mlangsdorf added <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display Missions Quests and missions labels Sep 19, 2019
@ZhilkinSerg ZhilkinSerg merged commit eec4808 into CleverRaven:master Sep 19, 2019
@mlangsdorf mlangsdorf deleted the ranch_fix branch September 19, 2019 14:13
misterprimus pushed a commit to misterprimus/Cataclysm-DDA that referenced this pull request Sep 21, 2019
Addresses CleverRaven#33525

A fair bit of the Tacoma Commune missions still use hard coded C++
functions to update the map, and those functions do not understand
map rotation at all.  Completing those missions will cause all
kinds of weird errors as NPCs teleport around the map and new
construction is built into old construction.

Make the ranch camp maps unable to rotate until I have time to go
through and move the remaining Tacoma Commune missions into JSON.
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) [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display Missions Quests and missions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants