Skip to content

fix: stop Random World Events cab log flood and skip live action re-register - #37

Merged
TheCodingDad-TisonK merged 1 commit into
Realistic-Farming:developmentfrom
WizardlyPayload:wizard/playtest-failfix-20260816
Aug 16, 2026
Merged

fix: stop Random World Events cab log flood and skip live action re-register#37
TheCodingDad-TisonK merged 1 commit into
Realistic-Farming:developmentfrom
WizardlyPayload:wizard/playtest-failfix-20260816

Conversation

@WizardlyPayload

Copy link
Copy Markdown
Contributor

Summary

  • If all three vehicle action ids are live, return before re-register and before the re-entrancy flag.
  • Drop per-call registered in VEHICLE context info logs. Version 2.2.0.1.

Test plan

  • Sit in a cab: no repeating registered in VEHICLE context lines
  • RWE HUD / settings / drag keys still work in cab and on foot

The vehicle hook ran every engine rebuild in a cab and logged each register, flooding the client log.

@sasha-rf sasha-rf left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved. The vehicle hook fires on every endActionEventsModification rebuild in a cab. When all three action IDs are already live, the early return skips the remove+re-register cycle and the info logs that came with it. Partial sets still take the full path. The return is before the re-entrancy flag, which is correct: the skip cannot wedge the hook shut. Version bump to 2.2.0.1.

@sasha-rf

Copy link
Copy Markdown

@TheCodingDad-TisonK verified and approved, ready to merge.

@TheCodingDad-TisonK
TheCodingDad-TisonK merged commit 2529592 into Realistic-Farming:development Aug 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants