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

Fish caught by fish traps rot too fast #58549

Open
mqrause opened this issue Jun 19, 2022 · 4 comments
Open

Fish caught by fish traps rot too fast #58549

mqrause opened this issue Jun 19, 2022 · 4 comments
Labels
Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. (S2 - Confirmed) Bug that's been confirmed to exist

Comments

@mqrause
Copy link
Contributor

mqrause commented Jun 19, 2022

Describe the bug

Fish traps produce up to 5 dead fish with an age between 0-3 hours 3 hours after they've been set. If you're not around to butcher the fish right away, chances are high you lose out on a lot of fish because it's rotten once you're done butchering.

Steps to reproduce

  1. Set up a fish trap (high survival skill recommended for multiple fish caught)
  2. Wait for a few more than 3 hours (e.g. sleep while the trap does its catching)
  3. See rotten fish, or fish that produce rotten fillets when butchered, depending on how much time passed and how old the fish were when spawned

Expected behavior

From my understanding, these kinds of trap catch live fish, so they should only start rotting the moment you extract them from the trap/extract the trap from the water, giving you enough time to process them.

Screenshots

No response

Versions and configuration

cc694a8

Additional context

No response

@mqrause mqrause added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Jun 19, 2022
@Maleclypse Maleclypse added (S2 - Confirmed) Bug that's been confirmed to exist Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Jun 19, 2022
@RoyBerube
Copy link
Contributor

Cataclysm-DDA/src/iuse.cpp

Lines 1903 to 1905 in 9ac0fc9

here.add_item_or_charges( pos, item::make_corpse( chosen_fish->type->id,
calendar::turn + rng( 0_turns,
3_hours ) ) );

Take away the time rng from make_corpse() and the fish would be fresh.

@mqrause
Copy link
Contributor Author

mqrause commented Jun 20, 2022

While that certainly improves the situation, I wouldn't consider it a real solution for the issue.

@chrispikula
Copy link
Contributor

Maybe the fish trap could fake leashing fish with radius zero? You'd still have to kill the fish. This would prevent multiple fish from being in the fish trap, though, as monsters can't stack.

@mqrause
Copy link
Contributor Author

mqrause commented Jun 21, 2022

Well, the basket fish trap is 12 liters, so I would assume a bunch of fish would fit in there (depending on their size of course). I was thinking maybe it could be more appropriately implemented as a an actual trap or a furniture that allows you to extract the fish by examining, spawning fresh fish at that moment.
Though the current code explicitly mentions it's done as it is for the convenience of not having to interact with the item after setting it down. But I don't exactly agree with that for the reason of this issue.

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. (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

No branches or pull requests

4 participants