TerraGov-Marine-Corps icon indicating copy to clipboard operation
TerraGov-Marine-Corps copied to clipboard

Datumizes storage

Open Xander3359 opened this issue 1 year ago • 10 comments

About The Pull Request

Turns storage into a datum instead of hard coding it onto item/storage

Why It's Good For The Game

Storage datums are more robust than hardcoding it onto a typepath

Changelog

:cl: refactor: Storage refactor /:cl:

Xander3359 avatar Feb 25 '24 19:02 Xander3359

I cant wait for this to cause thirty inexplicable bugs that you spend another year chasing down like Lumi and his throws

yyzsong avatar Feb 25 '24 19:02 yyzsong

I cant wait for this to cause thirty inexplicable bugs that you spend another year chasing down like Lumi and his throws

Nah I'd win

Xander3359 avatar Feb 25 '24 19:02 Xander3359

image

yyzsong avatar Mar 03 '24 11:03 yyzsong

if you are going to port stuff actually port it and stop trying to ignore the port ban

spookydonut avatar Mar 11 '24 10:03 spookydonut

As discussed align it as close to tg as reasonable, ping me when you want this reviewed

TiviPlus avatar Mar 26 '24 20:03 TiviPlus

sooo on the test merge of this pr the current bugs I personally found where: empty containers including flare pouches and all med pill bottles you cant take out pill bottles from backpacks you cant quick pull pills and other objects via using alt-click(again tested on pill bottles)

Aporhtonoma avatar Apr 08 '24 21:04 Aporhtonoma

Bypassing checks on pill bottle contents isn't going to let people make infinite RR pills or something, right?

Lumipharon avatar Apr 09 '24 08:04 Lumipharon

Bypassing checks on pill bottle contents isn't going to let people make infinite RR pills or something, right?

RR Pills are gone + you can't restock them anyways so? Keeping in mind, loadout vendor literally buys items from the vendors, so RR in the vendor (if it exists) would run out at the normal rate wether you vend it by hand or via loadout.

Xander3359 avatar Apr 09 '24 12:04 Xander3359

Please elaborate on why this is good for the game

lbnesquik avatar Apr 09 '24 22:04 lbnesquik

Please elaborate on why this is good for the game

It is code quality. Allows to decouple storage, so everything can easily become one without having to do shitcode

BraveMole avatar Apr 09 '24 22:04 BraveMole

We did it chat. Special thanks to anyone who helped by bug reporting during the test merges :D

Xander3359 avatar Apr 12 '24 20:04 Xander3359