Skip to content

Conversation

Stubbjax
Copy link

@Stubbjax Stubbjax commented Sep 5, 2025

This change fixes an issue where propaganda effects persist for contained units that have a direct PropagandaTowerBehavior module (e.g. Attack Outpost, Assault Troop Crawler).

The contained tower logic was only ever anticipated to be used for the Overlord's tower attachment, which is technically considered as a 'contained' passenger by the Overlord - hence the need for the two container checks. However, as the PropagandaTowerBehavior module can be added directly to units themselves, the container check does not function correctly for singular containers.

In the 1.04 example below, an Assault Troop Crawler enters a tunnel and its propaganda effect persists for units around the tunnel:

image

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

xezon commented Sep 20, 2025

This change correctly identifies the cause of the issue, but the fix is not properly implemented.

The KindOf inside the Contain module is configurable in INI:

  Behavior = HelixContain ModuleTag_09
    Slots                     = 1
    DamagePercentToUnits      = 100%
    AllowInsideKindOf         = PORTABLE_STRUCTURE
    PassengersAllowedToFire   = Yes
  End

Therefore, hardcoding this KindOf is not right.

The correct way to test is in the contain module via ContainModuleInterface::isSpecialOverlordStyleContainer. An example of how this is used can be seen in TransportContain::isPassengerAllowedToFire.

Therefore, I have created new change #1609.

@Stubbjax
Copy link
Author

Oh right, isSpecialOverlordStyleContainer is what I was looking for, though it hardly feels any more generic.

@Stubbjax
Copy link
Author

I believe I have figured out a more comprehensive solution that covers all cases.

PROP.mp4

@xezon xezon changed the title bugfix: Propaganda effects no longer persist for contained units with direct propaganda tower behaviour modules bugfix(module): Propaganda effects no longer persist for contained units with direct propaganda tower behavior modules Oct 16, 2025
@xezon xezon changed the title bugfix(module): Propaganda effects no longer persist for contained units with direct propaganda tower behavior modules bugfix(behavior): Fix unexpected propaganda influence from contained units with direct Propaganda Tower behavior modules Oct 16, 2025
@xezon xezon merged commit 441e1aa into TheSuperHackers:main Oct 16, 2025
18 checks passed
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 Gen Relates to Generals 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.

2 participants