Skip to content

Conversation

@Stubbjax
Copy link

Fixes #135

This change fixes the damage calculation for determining the Battle Bus's initial death, meaning it no longer prematurely dies when receiving damage via radiation or poison fields.

The issue is most notable when receiving damage via radiation or poison fields due to HazardFieldCoreWeapon - an invisible 200-damage hazard cleanup weapon that typically accompanies poison and radiation fields to stop them from stacking. As no armour damage modifiers are applied to ignore the hazard cleanup damage, the Battle Bus dies when receiving damage from this source if health is <= 200. Even an Ambulance can kill a Battle Bus by exploiting this logic.

GOOD_BUS.mp4

@Stubbjax Stubbjax self-assigned this Sep 19, 2025
@Stubbjax Stubbjax added Bug Something is not working right, typically is user facing GLA Affects GLA faction Minor Severity: Minor < Major < Critical < Blocker ZH Relates to Zero Hour NoRetail This fix or change is not applicable with Retail game compatibility labels Sep 19, 2025
Copy link

@xezon xezon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice.


if( damageInfo->in.m_damageType != DAMAGE_UNRESISTABLE
&& !m_isSecondLife
#if RETAIL_COMPATIBLE_CRC
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think better use RETAIL_COMPATIBLE_BUG here

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As in this is a desirable bug to be kept? If some players strangely desire this behaviour, they can surely just tweak the Battle Bus's HAZARD_CLEANUP damage modifier to deal damage.

Armor BattleBusTruckArmor
+  Armor = HAZARD_CLEANUP      50%    ;Now consistently harmed by cleaning weapons
-  Armor = HAZARD_CLEANUP       0%    ;Not harmed by cleaning weapons
End

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

@xezon xezon added the Controversial Is controversial label Sep 21, 2025
@xezon xezon changed the title bugfix: Battle Bus now correctly applies damage modifiers when determining initial death bugfix(module): Fix damage calculation for determining the initial death of GLA Battle Bus Sep 23, 2025
@xezon xezon merged commit a2cebd8 into TheSuperHackers:main Sep 23, 2025
18 checks passed
@Stubbjax Stubbjax deleted the fix-battle-bus-premature-death branch September 24, 2025 03:28
fbraz3 pushed a commit to fbraz3/GeneralsX that referenced this pull request Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something is not working right, typically is user facing Controversial Is controversial GLA Affects GLA faction Minor Severity: Minor < Major < Critical < Blocker NoRetail This fix or change is not applicable with Retail game compatibility ZH Relates to Zero Hour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GLA Battle Bus will enter bunkered mode at 50% health left when shot with Toxin and Radiation weapons

2 participants