mirror of
https://github.com/Cockatrice/Cockatrice.git
synced 2026-09-27 16:32:20 -07:00
030e4f5871
Fix duplicate attach messages in the game log caused by a redundant child arrow in CardItem::drawAttachArrow. Unlike the sibling drawArrow, drawAttachArrow omitted the ``card == this`` guard when iterating selectedItems(). Because right-clicking a card to open the attach menu selects that card, it was always present in selectedItems(), producing a second arrow for the same source card. On release both arrows sent an identical Command_AttachCard, so the server broadcast two Event_AttachCard messages and the log rendered "attaches to" twice. Mirror the drawArrow skip condition so the active card is excluded and only one attach command is sent. Co-authored-by: Lukas Brübach <Bruebach.Lukas@bdosecurity.de> |
||
|---|---|---|
| .. | ||
| abstract_card_drag_item.cpp | ||
| abstract_card_drag_item.h | ||
| abstract_card_item.cpp | ||
| abstract_card_item.h | ||
| abstract_counter.cpp | ||
| abstract_counter.h | ||
| abstract_graphics_item.cpp | ||
| abstract_graphics_item.h | ||
| arrow_item.cpp | ||
| arrow_item.h | ||
| arrow_target.cpp | ||
| arrow_target.h | ||
| card_drag_item.cpp | ||
| card_drag_item.h | ||
| card_item.cpp | ||
| card_item.h | ||
| counter_general.cpp | ||
| counter_general.h | ||
| graphics_item_type.h | ||
| translate_counter_name.cpp | ||
| translate_counter_name.h | ||