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

new refrigerated tank doesn't show up as a tank or vehicle electronic #39546

Closed
joe9000 opened this issue Apr 13, 2020 · 7 comments
Closed

new refrigerated tank doesn't show up as a tank or vehicle electronic #39546

joe9000 opened this issue Apr 13, 2020 · 7 comments
Labels
<Bug> This needs to be fixed Vehicles Vehicles, parts, mechanics & interactions

Comments

@joe9000
Copy link

joe9000 commented Apr 13, 2020

Describe the bug

the new refrigerated tank doesn't appear as a tank in the [e]xamine vehicle screen, nor can i fill it.

Steps To Reproduce

  1. create vehicle, refrigerated tank, a rubber hose, and a water source
  2. install refrigerated tank in vehicle
  3. [e]xamine vehicle: refrigerated tank does not appear in tanks section
  4. [e]xamine a water source adjacent to vehicle: unable to fill refrigerated tank

Expected behavior

be able to access refrigerated tank like an existing 60l tank installed in vehicle.

#workaround
add "insulation": 10, to containers.json for id: metal_tank

@Maleclypse
Copy link
Member

I'm confused because feel like this worked though I may not have tested it thoroughly enough. These flags don't seem like they conflict with each other, "flags": [ "FLUIDTANK", "FRIDGE", "ENABLED_DRAINS_EPOWER" ]. I'll test again and see if I can figure out where this is going awry.

@Maleclypse
Copy link
Member

Plus side, debug spawned in a solar vehicle, toolbox, and refridgerated tank. Installed tank. examined vehicle controls and under control electronics was a new fridge I could turn on. Downside, still not visible as a fuel tank.

@RedShakespeare
Copy link
Contributor

There appear be two problems:

  1. One vehicle part can work as a fluid tank only if it is made by an item which has type "CONTAINER" and attributes "watertight" and "seals" of which have to be both true. I tried to rewrite the item-ver fridgetank like this
{
    "type": "CONTAINER",
    "looks_like": "VEHICLE_PART",
    "copy-from": "washing_machine",
    "id": "fridgetank",
    "name": { "str": "refrigerated tank" },
    "description": "A 60L refrigerated tank for keeping liquids cool.  Provides some insulation from outside weather.",
    "insulation": 2,
    "volume": "60 L",
    "weight": "48 kg",
    "contains": "60 L",
    "seals": true,
    "watertight": true,
    "qualities": [ [ "CONTAIN", 1 ] ]
  },

and it turned out that this vehicle part gained the ability to be refilled.

  1. It looks like the flag "FRIDGE" works only by changing the temperature flag of the space inside the vehicle part itself, which means only if you can put items with "D"/"d" key into the vehicle part will these items get cooled down. I tested this by filling this tank with raw milk and turned on the fridge, and unfortunately after 12 hour in game waiting the milk inside this tank got rotten just the same time as milk in common plastic bottles.

So, maybe the author should add some src codes to realize "cooling down liquld in a tank", or this item should be deleted from the game.

@Maleclypse
Copy link
Member

Right now this is waiting until pockets are implemented to see how that impacts things and to not cause additional problems for the pockets code.

@Night-Pryanik Night-Pryanik added the Vehicles Vehicles, parts, mechanics & interactions label Apr 30, 2020
@stale
Copy link

stale bot commented May 30, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not 'bump' or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered.

@stale stale bot added the stale Closed for lack of activity, but still valid. label May 30, 2020
@anothersimulacrum anothersimulacrum added the <Bug> This needs to be fixed label May 30, 2020
@stale stale bot removed the stale Closed for lack of activity, but still valid. label May 30, 2020
@Maleclypse
Copy link
Member

Tests suggest this issue is fixed.

@anothersimulacrum
Copy link
Member

Fixed in #42384

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 Vehicles Vehicles, parts, mechanics & interactions
Projects
None yet
Development

No branches or pull requests

5 participants