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

You can pick up water even if it's not frozen #33354

Closed
RedShakespeare opened this issue Aug 19, 2019 · 5 comments
Closed

You can pick up water even if it's not frozen #33354

RedShakespeare opened this issue Aug 19, 2019 · 5 comments
Labels
<Bug> This needs to be fixed Items / Item Actions / Item Qualities Items and how they work and interact

Comments

@RedShakespeare
Copy link
Contributor

RedShakespeare commented Aug 19, 2019

Describe the bug

You can pick up liquid water.(tho it will immediately leak onto the ground again, u can still move dirty water to somewhere else.)

Besides, if you try to haul water on the ground, there will be a 0 water carried with you and it will cause a game crash if you examine it.

Steps To Reproduce

Steps to reproduce the behavior:

  1. Make a player with a bottle of water.
  2. Pour the water onto the ground.
  3. Try to pick up water with 'e' or 'g' or ',', or try to haul water with '\'.
  4. You can pick up dirty water for an instance, or you will create a 0 volume water.

Expected behavior

Liquid water shouldn't be picked up or carried just as in the former versions.

Screenshots

image
↑Try to haul water with the player
image
↑Try to pick up water

Versions and configuration

  • Game Version: [0.d-6621-gd201308 or 0.D-9505]
@Night-Pryanik Night-Pryanik added <Bug> This needs to be fixed Items / Item Actions / Item Qualities Items and how they work and interact labels Aug 19, 2019
@EddiTheBambi
Copy link
Contributor

EddiTheBambi commented Aug 19, 2019

The problem with advanced inventory and hauling is a tough nut to crack. You should need to go through crush_frozen_liquid( item_location ) to handle frozen liquids via advanced inventory but I haven't found a good way of getting an item_location from any convenient context related to it. I've played with the idea of disabling frozen liquid handling through advanced inventory for now until a better solution can be found.

@RedShakespeare
Copy link
Contributor Author

RedShakespeare commented Aug 20, 2019

It may be a choice to add a function in map_item_stack.cpp which goes thru all items and checks out if there's liquid or frozen liquid in a tile. Much bothering and may cause other problems like horrible optimization,tho...

@Ramza13
Copy link
Contributor

Ramza13 commented Feb 16, 2020

Could not reproduce this issue.

@Saicchi
Copy link
Contributor

Saicchi commented Apr 3, 2021

Can not reproduce in 0.E-10134-g9b256f63cd [64-bit]. Trying to pick up the spilled water gives the Try a mop instead message.

@Night-Pryanik
Copy link
Contributor

Ok, closing until reconfirmed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Items / Item Actions / Item Qualities Items and how they work and interact
Projects
None yet
Development

No branches or pull requests

5 participants