Skip to content

Wardrobe - Fix: ACRE radios reset#11148

Merged
PabstMirror merged 22 commits intoacemod:masterfrom
OverlordZorn:wardrobe-acre-fix
Nov 27, 2025
Merged

Wardrobe - Fix: ACRE radios reset#11148
PabstMirror merged 22 commits intoacemod:masterfrom
OverlordZorn:wardrobe-acre-fix

Conversation

@OverlordZorn
Copy link
Copy Markdown
Contributor

When merged this pull request will:

use sqf command instead of cba fnc to retain unfiltered classnames

@OverlordZorn
Copy link
Copy Markdown
Contributor Author

@Drofseh

@jonpas
Copy link
Copy Markdown
Member

jonpas commented Nov 25, 2025

CBA_fnc_setLoadout uses setUnitLoadout but ACRE supports it in a way that forces new IDs to avoid issues. I am not sure what the best way to approach this is, maybe what you did is fine for this use case. @BrettMayson ?

@BrettMayson
Copy link
Copy Markdown
Member

Yeah the CBA functions are more for preserving details when saving/ transferring to another unit. For doing swaps here, which shouldn't affect anything another mod is doing, using setUnitLoadout directly is better

Copy link
Copy Markdown
Contributor

@PabstMirror PabstMirror left a comment

Choose a reason for hiding this comment

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

this will break gunbag because none of the extendedInfo is saved

@BrettMayson
Copy link
Copy Markdown
Member

BrettMayson commented Nov 25, 2025

Does wardrobe not preserve variables on the backpack? Doing that should handle this right?

@PabstMirror
Copy link
Copy Markdown
Contributor

I think the combo of this and #11139 should work

@PabstMirror PabstMirror added this to the 3.20.2 milestone Nov 27, 2025
@PabstMirror PabstMirror added the kind/bug-fix Release Notes: **FIXED:** label Nov 27, 2025
@PabstMirror PabstMirror merged commit 5740af1 into acemod:master Nov 27, 2025
3 checks passed
@OverlordZorn OverlordZorn deleted the wardrobe-acre-fix branch February 8, 2026 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug-fix Release Notes: **FIXED:**

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wardrobe - Switching containers (uniforms, etc) resets ACRE radios.

4 participants