Apply forensics when loading with an ammo box #32280
Open
+3
−2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
About the PR
Your glove fibers or fingerprints will now appear on each bullet loaded with an ammo box.
Fixes #21555
Why / Balance
Loading with an ammo box is just an abstraction of loading the bullets by hand, in order to save the pain of clicking one by one. Logically, forensics are meant to be applied to each bullet as you load it.
Technical details
We now call the SharedInteractionSystem.InteractUsing() instead of directly raising the event. This ensures that the contact events are raised for forensics. (We skip interaction checks since those are done already by that point.)
(If preferred, I can also directly raise a contact event here, but it seems better to reuse code and keep things centralized when possible.)
Media
ammo-box-fingerprint-demo.mp4
Requirements
Changelog
🆑