CleverRaven / Cataclysm-DDA

Cataclysm - Dark Days Ahead. A turn-based survival game set in a post-apocalyptic world.
http://cataclysmdda.org
Other
10.32k stars 4.14k forks source link

tried to put an item (water_clean, amount 1) in a container (heattank) that cannot contain it #76382

Open dev-dsp opened 3 weeks ago

dev-dsp commented 3 weeks ago

Describe the bug

Siphoning clean water between heated tanks of my vehicle doesn't work. I've just purified the water in the tank that is beneath a funnel and collects water (32L), now trying to move it to a clean water heated tank (1L), and getting that crash.

Attach save file

Nairn-trimmed.tar.gz

Steps to reproduce

  1. e 5 e
  2. s
  3. choose one of heated tanks
  4. choose another heated tank

Expected behavior

Able to move liquids between vehicle tanks

Screenshots

image image

Versions and configuration

Additional context

DEBUG : tried to put an item (water_clean, amount 1) in a container (heattank) that cannot contain it

FUNCTION : int item::fill_with(const item&, int, bool, bool, bool, bool, Character*) FILE : src/item.cpp LINE : 12360 VERSION : cdda-experimental-2024-09-08-2121 515ac7b

mqrause commented 3 weeks ago

The water of one tank shows as frozen for me: grafik

So it's somehow the revival of #63536?

dev-dsp commented 3 weeks ago

Hmmmmmmm It was definitely not frozen (as seen on my screenshot) And I've tried to put 32L->1L, so it makes sense that it's not possible to move frozen water out of a tank.. Okay, then it's minor, probably just UI was not updated somehow

dev-dsp commented 3 weeks ago

If it shows as frozen on save load, then it's probably not that easy to reproduce and can be closed, unless there are any ideas on what could've happened here

mqrause commented 3 weeks ago

It's still an issue, since as far as I'm aware, you should be able to pour a liquid into a tank with the same liquid but frozen.

dev-dsp commented 3 weeks ago

Please read my comments again :sweat_smile: I've tried to merge 32L (which turns out to be frozen, just not showing it in UI) to 1L (which is not frozen)

mqrause commented 3 weeks ago

I meant it shouldn't be closed, because the error message you put in the title doesn't actually come from the UI issue but the fact the liquids can't be combined for some reason.

The UI not displaying the actual state and thus also allowing you to select a frozen liquid for siphoning is a different issue on top of that.