[SUBMISSION] Ballistic Plating

Info
Name: Ballistic Plating
Type: Armor
Palette: Doom / Custom
Summon: BallisticPlating
Use type: Instant
Duration: Instant
Brightmaps: No
Actor modification: No
ACS: No

Description:
Upgrades the player's armor with Ballistic Plating.
Adds 50 armor (can go over the current armor's maximum) and increases protection by 20%.
Applies to body-worn armor only, has no effect on any Armor Bonuses.
Only one Ballistic Plating can be used at once, but damaged Ballistic Plating can be repaired by simply picking up another one.
Ballistic Plating does not carry over if another armor is picked up.
Works with vanilla armors (Green, Blue, Megasphere) as well as custom armors from these Realm667 files:
- ArmorBelt.wad
- HeavyArmor.wad
- QuakeItemPack.wad

Credits
Code: DeVloek
GLDEFS: DeVloek
Sounds: DeVloek
Sprites: id Software, DeVloek, TOMISURRO, Captain Toenail, Fuzzyblaze
Sprite Edit: DeVloek
Idea base: irl ballistic armor plates

Sprite
 

Download
https://drive.google.com/file/d/1hfjwTlf3sBaM79JRHjNpsDc0b9nwVxW7/view?usp=sharing

Let me know if I should make this compatible with other armors. I think I covered all custom armors found on R667, at least those that make sense being upgraded. Armors like the Tech Armor and Super Shield are already quite overpowered without further upgrades so I left them out.

Pretty cool item. A few things to comment on:

• I'd make the bonus armor points and damage reduction of the item actor properties, so they're overridden through the Default block rather than PostBeginPlay(). It's tidier and cleaner that way.
• The blue armor you get from cheating cannot be upgraded. Though it's not going to be easy to identify or distinguish it.
• It's better to override Inventory.PickupMessage() to return the pickup message for the engine to print instead of handling it the way you did, since that, I believe, isn't multiplayer friendly; all players in-game would see the message when one one picks the item up.

Thanks for the tips.

• I'd make the bonus armor points and damage reduction of the item actor properties, so they're overridden through the Default block rather than PostBeginPlay(). It's tidier and cleaner that way.

I remember messing around with custom properties in the past but couldn't figure out how it works. I'll try again.

• It's better to override  Inventory.PickupMessage()  to return the pickup message for the engine to print instead of handling it the way you did, since that, I believe, isn't multiplayer friendly; all players in-game would see the message when one one picks the item up.

Didn't know that was possible. I'll try to figure out how that works too.
Edit: OK that was easy enough. btw I don't think I'll try to make it work with cheated armor, since it isn't a thing the player picks up and wears.


 

Update: 

• Armor amount and protection are now set in the default block instead of postbeginplay
• Pickup message is now overridden instead of being replaced by console.printfex

Like I mentioned earlier, making it compatible with cheated armor isn't something I'm eager to fix. However if someone can show me a method to distinguish between picked up armor and cheated armor, I'll try to implement it.

Well even though I usually say "screw the cheaters" I looked into this, and turns out it wasn't that hard to figure out. Any armor that was not picked up from the map (ie via cheats) defaults back to BasicArmorPickup, so I added this type to the armortype check. Assuming cheated armor always has max value of 200 like the blue armor, I set its modified max value to 250.

Added