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

Hacksaw bar cutting #52341

Merged
merged 2 commits into from
Oct 19, 2021
Merged

Conversation

jharg42
Copy link
Contributor

@jharg42 jharg42 commented Oct 18, 2021

Summary

Bugfixes "Tweaks to hacksaw and torch cutting"

Purpose of change

Fixes #52176. In addition to hacksaws not working on window frames with metal bars, acetylene torches also didn't work for the same reason (missing JSON). Cutting bars off of shop-style windows that DON'T have an alarm made the glass vanish as well. In addition, Sacchi indicated in Discord that they hadn't intended for acetylene torch fuel consumption to be affected by player speed and that swapping to time instead of speed would make more sense.

Describe the solution

Add the JSON for cutting t_window_bars_noglass, modify JSON for cutting t_window_bars, change the JSON for the player action of cutting with the acetylene torch.

Describe alternatives you've considered

If removing the glass on non-alarmed windows when cutting the bars of was desired, adding the glass to the byproducts left over would make sense.

Testing

Load into the game with the modified JSON files, spawn the terrain tiles in question and cut the bars off with hacksaw and torch to verify results. Check that cutting bars with the acetylene torch no longer uses more fuel if the player speed is lowered (used being overencumbered to lower speed).

Additional context

@BrettDong BrettDong added [JSON] Changes (can be) made in JSON Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. labels Oct 18, 2021
@kevingranade kevingranade merged commit 8e9913d into CleverRaven:master Oct 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. [JSON] Changes (can be) made in JSON
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hacksaws struggle to hack it as saws
3 participants