Ticket #1503 (closed defect: fixed)

Opened 11 months ago

Last modified 11 months ago

[PATCH] messages.js: handleNetMessage() obj.caption overwritten

Reported by: picobyte Owned by: ben
Priority: Should Have Milestone: Alpha 11
Component: UI & Simulation Keywords: patch
Cc:

Description

There are missing break statements in messages.js: handleNetMessage() causing obj.caption to be overwritten.

Attachments

netmsg_caption.patch​ (932 bytes) - added by picobyte 11 months ago.

Change History

Changed 11 months ago by picobyte

comment:1 Changed 11 months ago by historic_bruno

  • Keywords patch review added
  • Milestone changed from Backlog to Alpha 11

comment:2 Changed 11 months ago by k776

  • Keywords handleNetMessage netstatus authenticated connected removed

comment:3 Changed 11 months ago by historic_bruno

  • Keywords review removed

Oops, apparently it was me who added that, I really don't remember it at all, probably I just forgot the break statements. Thanks for noticing :)

comment:4 Changed 11 months ago by ben

  • Owner set to ben
  • Status changed from new to closed
  • Resolution set to fixed

In 11981:

Adds missing break statements, thanks to picobyte. Fixes #1503

Note: See TracTickets for help on using tickets.