Opened 5 years ago

Closed 5 years ago

#5206 closed defect (fixed)

Playernames of previous match persist in the next gamesetup

Reported by: elexis Owned by: elexis
Priority: Nice to Have Milestone: Alpha 23
Component: UI & Simulation Keywords:
Cc: Patch:

Description (last modified by elexis)

Sometimes we can see playernames of a previous match in the gamesetup.

This error cannot be fixed without deleting the persist-matchsettings or selecting a different maptype.

For years we didn't know how it could occur (so filing this report to have a ticket to find in case we need to look it up again).

But the reason is easy after accidentally finding it in #5199, doubleclicking on Start Game. The first call it will copy over the names, the second call it will save those to the persistmatchsettings file.

There is a different ticket for persist-match-settings saving less information. Playernames should be dropped as well eventually.

Attachments (1)

assignments.jpg (353.3 KB ) - added by elexis 5 years ago.

Download all attachments as: .zip

Change History (5)

by elexis, 5 years ago

Attachment: assignments.jpg added

comment:1 by elexis, 5 years ago

Description: modified (diff)

comment:2 by elexis, 5 years ago

Preventing doubleclicks on startGame is important and will be done in Phab:D1558.

Cutting the problem at the other root, the ability to save values that don't need saving is to be done in #3883.

comment:3 by elexis, 5 years ago

Milestone: BacklogAlpha 23
Priority: Should HaveNice to Have

comment:4 by elexis, 5 years ago

Owner: set to elexis
Resolution: fixed
Status: newclosed

In 21843:

Prevent calling the JS gamestart function twice when doubleclicking on the "Start Game" button.

It triggered an OOS on rejoin prior to rP21836, refs #5199 and
resulted in the playernames of the current match persisting into new gamesetups, fixes #5206.

Differential Revision:
Comments By: wraitii

Note: See TracTickets for help on using tickets.