more bugs

when players skin their furnaces with the new industrial, they ca no longer upgrade their furnaces, 

also found this bug

NullReferenceException: Object reference not set to an instance of an object at Oxide.Plugins.FurnaceUpgrades+OvenComponent.ConsumeFuel (Item fuel, ItemModBurnable burnable) [0x000b6] in <11e2c01ad53245f28a686485b468dbc8>:0 at Oxide.Plugins.FurnaceUpgrades+OvenComponent.Cook () [0x001df] in <11e2c01ad53245f28a686485b468dbc8>:0 at InvokeHandlerBase1[T].DoTick () [0x00120] in <e79bdbd88757469297860e5604f01eb1>:0 at InvokeHandlerBase1[T].LateUpdate () [0x0001f] in <e79bdbd88757469297860e5604f01eb1>:0

 

This is a FurnaceUpgrades null-handling bug, most likely triggered while creating the charcoal/byproduct item.

Cook() obtains ItemModBurnable without validating it, then ConsumeFuel() assumes that component is valid. It also calls ItemManager.Create(...) and immediately uses the returned item without checking whether creation succeeded.


When reskinning for industrial skin, it fails 

Important weakness

The plugin does not contain a hook for a furnace being reskinned after it has already been upgraded. There is no OnEntitySkinChanged, spray-can, or equivalent migration handler. Its upgrades are stored against the deployed furnace’s network ID, not against the owner and skin combination.