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

update butchery to use requirements #42372

Merged
merged 4 commits into from Jul 26, 2020

Conversation

KorGgenT
Copy link
Member

Summary

SUMMARY: Features "update butchery to use requirements"

Purpose of change

Moving data from hardcode into json, and also making the messaging clearer when tools are missing.

Describe the solution

use a requirement for various parts of butchery:
field dress, quarter, skin, dismember: field_dress
dissect: dissect_large, dissect_small
full: full_butchery_large, butchery_small
quick: butchery_large, butchery_small

Additionally, uses the same 6 tile range as crafting does.

Describe alternatives you've considered

Well we could keep butchery tools hardcoded i guess

Testing

spawned in various creates with and without tools.
image

Additional context

Inspired by 4c9000b

@KorGgenT KorGgenT added [C++] Changes (can be) made in C++. Previously named `Code` Mechanics: Character / Player Character / Player mechanics labels Jul 22, 2020
Copy link
Contributor

@I-am-Erk I-am-Erk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's still the butchery_rack_pseudo in there that we could probably change since it's not a term that's used and the item needs to go away.

data/json/furniture_and_terrain/furniture-tools.json Outdated Show resolved Hide resolved
@jbytheway
Copy link
Contributor

There's a problem with pseudo-items: to the best of my knowledge the player has no way to search for furniture that has a pseudo-item providing a particular quality. That feels like a necessary step (but maybe it needn't come before this PR is merged).

@kevingranade kevingranade merged commit 97b975a into CleverRaven:master Jul 26, 2020
@KorGgenT KorGgenT deleted the butchery-requirements branch July 26, 2020 03:19
@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/butchering-in-0-e/24288/2

@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/butchering-in-0-e/24288/4

chaosvolt added a commit to chaosvolt/MST_Extra_Mod that referenced this pull request Aug 1, 2020
* Added overrides to the qualities introduced by
CleverRaven/Cataclysm-DDA#42372 to fix quick
butchery needing a surface, because that is an astoundingly tedious
idea.
* Also reduced surface demands for dissection so that low-level surfaces
work for large creatures, and small creatures don't need any surface.
* This also fixes dismembering them, I'm not even sure what absurdity in
the code is tying dismemberment requirement to one of the other butchery
requirements, but needing a clean fancy tarp to messily hack apart a
body is a terrible idea.
* Added an override to tarps, adding surface qaulity because
CleverRaven/Cataclysm-DDA#42553 forgot them.
* Cleaned out an obsolete item that hasn't been used since back when I
was still a contributor. It was pinging the zero-weight and there was
really no need to update it after literally 3-4 years of it being
dummied out.
* Zero-weight fixes for psuedo-items.
@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/dissecting-in-0-e/24327/7

@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/butchering-in-0-e/24288/33

@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/butchering-in-0-e/24288/35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` Mechanics: Character / Player Character / Player mechanics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants