-
Notifications
You must be signed in to change notification settings - Fork 105
bugfix(behavior): Fix unexpected propaganda influence from contained units with direct Propaganda Tower behavior modules #1545
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bugfix(behavior): Fix unexpected propaganda influence from contained units with direct Propaganda Tower behavior modules #1545
Conversation
Generals/Code/GameEngine/Source/GameLogic/Object/Behavior/PropagandaTowerBehavior.cpp
Outdated
Show resolved
Hide resolved
Generals/Code/GameEngine/Source/GameLogic/Object/Behavior/PropagandaTowerBehavior.cpp
Outdated
Show resolved
Hide resolved
… direct propaganda tower behaviour modules
ad11b89
to
198e8c8
Compare
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:
Therefore, hardcoding this KindOf is not right. The correct way to test is in the contain module via Therefore, I have created new change #1609. |
Oh right, |
I believe I have figured out a more comprehensive solution that covers all cases. PROP.mp4 |
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: