HIGH: Ledger detail entries not persisted on standalone XML path

[TheCodingDad-TisonK]

Severity: HIGH

File: src/main.lua:114-121 (save), src/main.lua:154-166 (load)
Multiplayer: Yes
Fix Complexity: MODERATE
Confidence: 100% (source-verified)

Description

XML save writes only creditTotal, debitTotal, farmId — entries array (last 10 per farm) not saved. Load recreates as empty {}.

Impact

Ledger detail in HUD and console wiped every save/reload. Only running totals survive. StateLedger path preserves them, but own-file doesn't.

Fix

Serialize the entries array to XML on save, restore on load.

---

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

[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:114-121 (save) and 154-166 (load) — entries array (last 10 per farm) was not serialized to XML, only creditTotal/debitTotal/armId. Fixed in commit 65e16fb by serializing the full entries array. 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.