Skip to content

Conversation

@Stubbjax
Copy link

@Stubbjax Stubbjax commented Oct 3, 2025

This change allows the Sneak Attack to remain selected when it transitions from its spawning object.

A new TransferSelection field can now be added to the CreateObjectDie module to enable this behaviour.

Note: No other object uses the TransferPreviousHealth field, so this field could technically be used to control the reselection behaviour (initially?) and thus be retail compatible. However, it would not make logical sense and would confound the behaviour.

Object GLASneakAttackTunnelNetworkStart
  Behavior             = CreateObjectDie ModuleTag_11
    CreationList  = OCL_CreateSneakAttackTunnel
    TransferPreviousHealth = Yes
+   TransferSelection = Yes
  End
End

Feature or bugfix?

@Stubbjax Stubbjax self-assigned this Oct 3, 2025
@Stubbjax Stubbjax added Bug Something is not working right, typically is user facing Enhancement Is new feature or request Minor Severity: Minor < Major < Critical < Blocker ZH Relates to Zero Hour labels Oct 3, 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.

Feature or bugfix?

Is half bugfix, half feature. Given that this is mainly undesired behavior with the Sneak Attack, I would say we can label it bugfix.

Generals also has this Module. Replicate the change there?

@Stubbjax
Copy link
Author

Stubbjax commented Oct 7, 2025

Generals also has this Module. Replicate the change there?

I was going to, but was stopped by TransferPreviousHealth not existing in Generals, and thought it would be better to migrate both that and TransferSelection in a separate consolidation change. I can add it here if desired, but then TransferPreviousHealth will either need to come with it or be added in a separate change anyway.

@xezon
Copy link

xezon commented Oct 7, 2025

Can be separate change, replicating these fields to Generals.

@xezon xezon added the GLA Affects GLA faction label Oct 7, 2025
@xezon xezon changed the title tweak: Sneak Attack remains selected after spawning bugfix(module): Fix GLA Sneak Attack not staying selected after fully emerging Oct 7, 2025
@xezon xezon changed the title bugfix(module): Fix GLA Sneak Attack not staying selected after fully emerging bugfix(module): Add 'TransferSelection' field to CreateObjectDie INI module to allow transfer selection on object creation Oct 7, 2025
@xezon xezon changed the title bugfix(module): Add 'TransferSelection' field to CreateObjectDie INI module to allow transfer selection on object creation tweak(module): Add 'TransferSelection' field to CreateObjectDie INI module to allow transfer selection on object creation Oct 7, 2025
@xezon xezon merged commit dbdcd89 into TheSuperHackers:main Oct 7, 2025
15 checks passed
@Stubbjax Stubbjax deleted the sneak-attack-selection branch October 14, 2025 22:58
fbraz3 pushed a commit to fbraz3/GeneralsX that referenced this pull request Nov 10, 2025
…odule to allow transfer selection on object creation (TheSuperHackers#1665)

This change allows to configure the GLA Sneak Attack in INI to remain selected after it transitioned to the fully emerged object.

Object GLASneakAttackTunnelNetworkStart
  Behavior = CreateObjectDie ModuleTag
    ...
    TransferSelection = Yes ; The tunnel remains selected after emerging
  End
End
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 Enhancement Is new feature or request GLA Affects GLA faction Minor Severity: Minor < Major < Critical < Blocker ZH Relates to Zero Hour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants