LUDevNet / ParadoxServer

Server-side component of LU-Explorer
https://lu-dev.net/lu-explorer
4 stars 3 forks source link

Columns that are missing in old clients #25

Open Xiphoseer opened 1 year ago

Xiphoseer commented 1 year ago

We use paradox-typed-db based on 1.10.64, but we want ParadoxServer to work with old clients ideally. This issue is to track columns that were added recently that we should ignore completely if missing:

aronwk-aaron commented 1 year ago
Xiphoseer commented 1 year ago

Missions:isRandom

Turns out this wasn't the same sort of bug, but an entirely different class that should be fixed now

aronwk-aaron commented 1 year ago

I think this issue can be closed as we now have the oldest client in our possession up and running with lu-explorer

Xiphoseer commented 1 year ago

Are there still warnings in the logs? The current code just doesn't crash under any circumstances, but that doesn't mean it doesn't try to read those columns anymore.

(ie paradox-typed-db is more forgiving now, but will still complain the server is calling it wrong)

aronwk-aaron commented 1 year ago

oh right! I'll grab those logs and document what it is complaining about

aronwk-aaron commented 1 year ago

These are the ones I see in logs

 WARN  paradox_typed_db::rows         > Missing column gate_version in ZoneTable
 WARN  paradox_typed_db::rows         > Missing column gate_version in SpeedchatMenu
 WARN  paradox_typed_db::rows         > Missing column gate_version in UgBehaviorSounds
 WARN  paradox_typed_db::rows         > Missing column gate_version in SkillBehavior