#14559 breaks save game compatibility

[macrosblackd]

By virtue of removing an item id from the JSON without any logic to handle the conversion, older save games are going to be broken. This was just brought to my attention due to a forum post regarding this PR.
#14559

[TheRafters]

#14559 also breaks UPS as details in another forum post http://smf.cataclysmdda.com/index.php?topic=12053.0
Calling all @Core0verload 's, please fix. Discussion on the UPS affects in this PR never happened, it got overwhelmed by talking about battery weights.

[BevapDin]

By virtue of removing an item id from the JSON without any logic to handle the conversion, older save games are going to be broken

What id had been removed? The linked PR adds a few items and moves some definitions around, but I can't see entries that have been removed.

[macrosblackd]

Guess you're right - I coulda swore I saw a missing ID - that's what I get for looking at anything with 4 hours of sleep.

[Core0verload]

I did NOT removed any IDs. As for "Broken UPS charging" and "some tools don't work without UPS" - there is #14976 that's I'm now trying to fix. Sadly, it's easier to talk about "balance" than to test my changes with various tools and point out the issues.

This issue can be closed as a dupe of #14976.

[illi-kun]

Sadly, it's easier to talk about "balance" than to test my changes with various tools and point out the issues.

That's mostly your responsibility to test changes you're proposing to merge, no need to blame anyone else about that.

[macrosblackd]

Since I was mistaken about the save breakage, closing - I need more sleep.

[Core0verload]

That's mostly your responsibility to test changes you're proposing to merge, no need to blame anyone else about that.

I'm just still a bit mad about all that pointless balance talk.