-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Test tube incorrectly claims to fill with various liquids it cannot hold. #48921
Comments
The problem would appear to be the "You refill the test tube" message when there isn't 10 ml of the liquid in question in said test tube. |
|
It's not a bug that you can't put a unit in something too small for it; it is a bug that you're told you've refilled the test tube when you haven't.
Given that there's still a bug - just not the one the OP thought - I'm not sure why you're saying this. |
@actual-nh Seems I've misunderstood the issue. Thanks for clearing that up. |
Quite welcome; you're perfectly correct that the issue as originally stated isn't a bug. @Psiclops, to reduce confusion could you change the title to something like "Test tube falsely stated to be refilled"? EDIT: Thanks! |
Cataclysm-DDA/src/activity_actor.cpp Lines 4340 to 4341 in 9afa65c
Needs to account for failures |
When using (r)eload, the test tube will only fill up with certain liquids, although it claims success for all.
Successful reload liquid into test tube so far. (with message "You refill the test tube.")
Liquid mercury (10)
ethanol (10)
denatured alcohol (10)
nicotine liquid (2)
hot sauce (2)
dragon blood (10)
All of the mutagenic serums (1)
Fails (with message "You refill the test tube.")
lye
blood
antiseptic
animal cooking oil
vegetable cooking oil
rum
purifier
mutagen
hydrochloric acid
gin
whiskey
maple syrup
saline solution
cough syrup, regular and non-drowsy
hydrogen peroxide
heartburn medicine
lemonade
vinegar
water
clean water
It seems that some liquids have a different amount of space that (1) unit takes up than others. If the liquid cannot fit in a test tube, it should have a failure message instead of claiming to succeed.
Since a test tube holds 10ml, it looks like (1) dragon blood is 1ml, but (1) hot sauce is 5ml, and any (1) mutagenic serum is 10ml. It stands to reason that a test tube could hold any liquid you tried to pour into it, but in the terms of a minimally usable amount it will not hold most liquids in the game.
forgot to add game version info, editing in below
Dark Days Ahead [dda],
Disable NPC Needs [no_npc_food],
Bionic Professions [package_bionic_professions],
Magiclysm [magiclysm],
Graphical Overmap [Graphical_Overmap],
Graphical Overmap Magiclysm [Graphical_Overmap_Magiclysm]
]
The text was updated successfully, but these errors were encountered: