wesnoth/wesnoth

Missing updates for empty team labels in multiplayer

Aperta

#7768 aperta il 4 lug 2023

Vedi su GitHubGuida contributor
 (13 commenti) (0 reazioni) (0 assegnatari)C++ (1215 fork)auto 404
BugGood first issueMPUI

Metriche repository

Star
 (6798 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Game and System Information

  • Version: 1.16.9
  • Downloaded from: Steam
  • Build info: -
  • OS: Windows 10

Description of the bug

When you update an existing map label with an empty string. Only the local label is updated and not the labels of all team members.

Steps to reproduce the behavior

  1. Start a multiplay Game
  2. Create a Map label via Right click -> Set Label -> a non-empty arbitrary text
  3. Set the Label to empty via Right click -> Set Label -> empty String

Now only the player who set the label has an empty label. All other players have the non-empty arbitrary text.

Expected behavior

The expected behavior would be that all players have an empty string for that label.

Additional context

Probably the problem lies in this method https://github.com/wesnoth/wesnoth/blob/ade6e5803a6273cabd17e88628b6ad2cb469e0d6/src/map/label.cpp#L147 Since the updates are only carried out when the labels are not empty

Guida contributor

Direzione di ricerca
Esamina il metodo di aggiornamento dell'etichetta in src/map/label.cpp alla riga 147. Verifica perché gli aggiornamenti vengono saltati quando la nuova etichetta è vuota e assicurati che la logica di sincronizzazione di rete invii anche stringhe vuote. Cerca eventuali condizioni che filtrano le etichette vuote nel protocollo di sincronizzazione.
Tech stack
ccpp
Dominio
game devbackendnetworking
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++GitGame development basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40