[MIRROR] Removes amount_list_postion from reagent containers, adds related unit test. [MDB IGNORE] (#21869)

* Removes amount_list_postion from reagent containers, adds related unit test. (#76057)

We had more issues like what #76013 addressed, now they're gone.
Variable transfer amount is now explicit.
Amount is now inferred from current value, performance concern here is
minimal. Less work and mistakes when making new types.

* Removes amount_list_postion from reagent containers, adds related unit test.

* update modular

* Fixes the failing unit test, hopefully

---------

Co-authored-by: AnturK <AnturK@users.noreply.github.com>
Co-authored-by: Tom <8881105+tf-4@users.noreply.github.com>
Co-authored-by: GoldenAlpharex <jerego1234@hotmail.com>
This commit is contained in:
SkyratBot
2023-06-19 17:32:22 -04:00
committed by GitHub
co-authored by AnturK Tom GoldenAlpharex
parent 89226bbaf0
commit ee58caa1f6
21 changed files with 55 additions and 38 deletions
@@ -242,7 +242,7 @@
/obj/item/reagent_containers/cup/beaker/oldstation
name = "cultivation broth"
amount_per_transfer_from_this = 50
amount_per_transfer_from_this = 60 // SKYRAT EDIT - Beakers capacity 50u -> 60u - ORIGINAL: amount_per_transfer_from_this = 50
list_reagents = list(
// Required for CELL_LINE_TABLE_COW
/datum/reagent/consumable/nutriment/protein = 10,