HIGH: Ledger server-only, clients always see /

[TheCodingDad-TisonK]

Severity: HIGH

File: src/main.lua:430, src/ui/TaxHUD.lua:543-551
Multiplayer: Yes
Fix Complexity: MODERATE
Confidence: 100% (source-verified)

Description

recordExpense enforces getIsServer(), so ledger entries only recorded on server. No writeStream/readStream sync for ledger data.

Impact

MP clients always show "Tracked credits/debits: +$0 / -$0" in HUD regardless of actual server data.

Fix

Add ledger data to the existing NetworkSync broadcast, or create a dedicated TaxLedgerSyncEvent.

---

Source: Consolidated Bug Review 2026-07-26, TM-003

[github-actions]

Hey @TheCodingDad-TisonK, your filing has been received. 🏛️

@TheCodingDad-TisonK has been notified and will review your case as soon as possible. No late fees — we promise.

To help process your report quickly, please confirm:

• You are running the latest version of Tax Mod (visible in the mod manager)
• Your issue includes the relevant [TaxMod] lines from log.txt
  • Log location: Documents/My Games/FarmingSimulator2025/log.txt
  • Search for [TaxMod] and paste the relevant section here
• If this involves a specific tax bracket, deduction, or rate — note the expected value and what the mod actually applied
• You have searched existing issues for a similar report

Note: Issues without a log snippet are difficult to investigate and may be closed. Tax calculation errors are especially hard to reproduce without log data.

[TheCodingDad-TisonK]

Investigation confirmed: src/main.lua:430 —
ecordExpense enforced getIsServer() so ledger entries only recorded on server, with no sync for clients. Fixed in commit 65e16fb by adding ledger data to the NetworkSync broadcast. This has been resolved in the development branch (commit 65e16fb). Awaiting merge to main. Closing is recommended.

[TheCodingDad-TisonK]

Resolved in development branch, merged to main pending PR.