ZDoom

Short answer: Yes, this is by design.

Long answer: You have some mistaken assumptions about how BasicArmor works. A player can only ever have one BasicArmor class, and that's the BasicArmor class the player started with. The default BasicArmor class can be defined in MAPINFO: [url]https://zdoom.org/wiki/MAPINFO/GameInfo_definition#BasicArmorClass[/url]

Further, the only time a player can carry BasicArmorPickups or BasicArmorBonuses is if they are intended to be used with the Use Inventory key to add armor stats to BasicArmor. Otherwise, whenever BasicArmorPickup or BasicArmorBonus items are collected BasicArmor conditionally absorbs all of their stats. BasicArmor basically acts as a be-all end-all global armor tracker. To use ModifyDamage or AbsorbDamage virtuals on it effectively you can check for the armor type it currently has and apply special effects based on that. Or you can forgo BasicArmor completely to engineer armor that understands when it is worn and when it is not. Just some suggestions.

version "4.12"

class MySuperArmor : BasicArmor
{

	override void ModifyDamage(int damage, Name damageType, out int newdamage, bool passive, Actor inflictor, Actor source, int flags)
	{
		Console.printf("foo");
		if (passive && damage > 0)
		{
			Console.printf("bar!");
			newdamage = 0;
		}
	}
}

Hello there, with this simple item
[code]version "4.12"

class MySuperArmor : BasicArmor
{

override void ModifyDamage(int damage, Name damageType, out int newdamage, bool passive, Actor inflictor, Actor source, int flags)
{
Console.printf("foo");
if (passive && damage > 0)
{
Console.printf("bar!");
newdamage = 0;
}
}
}[/code]

I've noticed if the item is derived from item such function is called and it works, but if I derive my class from BasicArmor is not called.
It this by design?
I've tested with the GIT version.