Skip to content
This repository has been archived by the owner on Apr 17, 2022. It is now read-only.

desink with host t3 map #4784

Closed
wzdev-ci opened this issue May 20, 2018 · 11 comments
Closed

desink with host t3 map #4784

wzdev-ci opened this issue May 20, 2018 · 11 comments

Comments

@wzdev-ci
Copy link
Contributor

resolution_fixed type_bug | by andrvaut


If host T3 map, after starting the room appears 2 + 2 = 5


Issue migrated from trac:4784 at 2022-04-16 13:06:06 -0700

@wzdev-ci
Copy link
Contributor Author

andrvaut uploaded file t3.zip (294.1 KiB)

all log

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Jun 5, 2018

Berserk Cyborg changed blocking which not transferred by tractive

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Jun 5, 2018

Berserk Cyborg changed blockedby which not transferred by tractive

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Jun 5, 2018

Berserk Cyborg commented


Do you know when this started to happen? And T2 is not affected?

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Jun 5, 2018

andrvaut commented


I do not remember when the last time I played T2 / T3 in the master version.
Start with T2 did not check.

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Aug 8, 2018

Berserk Cyborg uploaded file 0001-Fix-instant-T2-T3-multiplayer-desync.patch (2.9 KiB)

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Aug 9, 2018

andrvaut commented


I checked the patch. There is no desynk

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Aug 9, 2018

Berserk Cyborg changed status from new to closed

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Aug 9, 2018

Berserk Cyborg changed owner from `` to Berserk Cyborg

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Aug 9, 2018

Berserk Cyborg changed resolution from `` to fixed

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Aug 9, 2018

Berserk Cyborg committed [647]


In Warzone2100/warzone2100@647ffd2:

#CommitTicketReference repository="" revision="647ffd2da1b75bfef2ce4649d4e613705b5e8b82"
Fix instant T2/T3 multiplayer desync.

getMultiTechLevel() would return a different result for the host
and clients since current_tech is only set when the user selects a
new tech level via menu. Thus, in rules.js, only the host would be
able to research T2/T3 tech.

fixes ticket 4784.

@wzdev-ci wzdev-ci closed this as completed Aug 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant