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

Removed long strings from piano disassembly and replaced the wires. #61371

Merged
merged 2 commits into from
Oct 1, 2022

Conversation

Daved27hundred
Copy link
Contributor

@Daved27hundred Daved27hundred commented Sep 30, 2022

Summary

Content "Pianos give tempered wire instead of regular wire & string upon disassembly"

Purpose of change

After bringing up that pianos are a good source of thread I was told that pianos do not use thread to play sounds, they use wire. After looking it up I found that they use a special type of "spring steel" wire which is made of tempered steel. Instead of making a new wire type I just used the tempered steel wire we have already, I figured it would make pianos a high value target for anyone looking for a quick way to get materials to blacksmith.

Describe the solution

removed the line in furniture_recreation.json that makes pianos drop long string, increased the number of wires dropped to compensate, and replaced the wire with qt_wire

Describe alternatives you've considered

Just making the piano give normal wire, or creating a new "piano wire" wire type.

Testing

N/A

Additional context

If there's any actual reason why the piano gives long string please let me know.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Sep 30, 2022
@dseguin dseguin merged commit f7528c9 into CleverRaven:master Oct 1, 2022
captainego pushed a commit to captainego/Cataclysm-DDA that referenced this pull request Oct 9, 2022
captainego pushed a commit to captainego/Cataclysm-DDA that referenced this pull request Oct 9, 2022
sparr pushed a commit to sparr/Cataclysm-DDA that referenced this pull request Dec 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants