Skip to content

XEH - Disable MagazineUnloaded EH to prevent crash to desktop#1727

Merged
PabstMirror merged 1 commit intomasterfrom
magUnloadBug
Feb 4, 2025
Merged

XEH - Disable MagazineUnloaded EH to prevent crash to desktop#1727
PabstMirror merged 1 commit intomasterfrom
magUnloadBug

Conversation

@PabstMirror
Copy link
Copy Markdown
Contributor

by adding MagazineUnloaded XEH we can cause a CTD due to vanilla bug that may not be fixed until next release

ref https://discord.com/channels/976165959041679380/976224344881655818/1330502064547299328
ref https://discord.com/channels/976165959041679380/976224344881655818/1331186704991784961

Adding the `MagazineUnloaded` EH to the player makes you crash to desktop when you are in the ACE Arsenal.

Repo steps:
1. Run
player addEventHandler ["MagazineUnloaded", {
    params ["_unit", "_weapon", "_muzzle", "_magazine"];
}];
2. Open the ACE Arsenal
3. Select your primary weapon on the left side
4. Select the primary magazine of the primary magazine on the right side
5. Select a new weapon on the left side
6. Crash

ace arsenal is doing something like setWeapItem ["130Rnd_338_Mag", nil, "MMG_02_sand_F"]
We can fix the nil in ace, but cannot make guarantees about all the other code out there
Due to the severity (CTD) I think it's best we skip this new XEH unit the bug is patched in 2.20 (hopefully)

@PabstMirror PabstMirror added this to the 3.18.2 milestone Jan 22, 2025
@PabstMirror PabstMirror merged commit 46a10de into master Feb 4, 2025
@PabstMirror PabstMirror deleted the magUnloadBug branch February 4, 2025 23:20
@PabstMirror PabstMirror added Bug Fix and removed Bug labels Mar 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant