Hi, yes, it's me again!
In some demos (TurretIntro.zip), the list of server classes reported by the DataTables message inexplicably contain duplicates, while the DataTables themselves do not. This causes the class-to-property mapping to desync.
This can be fixed by matching classes by data table name, not index.
So here:
|
SendTable table = _dtRef.Tables[serverClass.DataTableId]; |
... a valid fix would be to instead search
_dtRef.Tables for an entry where
dataTable.Name == serverClass.DataTableName.
Sorry for the issue spam. I'd love to PR these things myself, but as I've said in a prior issue, I have zero experience with C# or its environment. Hopefully my explanations of the proposed fixes make it clear enough what needs to be done.
Hi, yes, it's me again!
In some demos (TurretIntro.zip), the list of server classes reported by the DataTables message inexplicably contain duplicates, while the DataTables themselves do not. This causes the class-to-property mapping to desync.
This can be fixed by matching classes by data table name, not index.
So here:
UntitledParser/DemoParser/src/Parser/GameState/DataTablesManager.cs
Line 40 in de6dffd
... a valid fix would be to instead search
_dtRef.Tablesfor an entry wheredataTable.Name == serverClass.DataTableName.Sorry for the issue spam. I'd love to PR these things myself, but as I've said in a prior issue, I have zero experience with C# or its environment. Hopefully my explanations of the proposed fixes make it clear enough what needs to be done.