Opened 12 years ago

Closed 12 years ago

#1503 closed defect (fixed)

[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: Patch:

Description

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

Attachments (1)

netmsg_caption.patch​ (932 bytes ) - added by picobyte 12 years ago.

Download all attachments as: .zip

Change History (5)

by picobyte, 12 years ago

Attachment: netmsg_caption.patch​ added

comment:1 by historic_bruno, 12 years ago

Keywords: patch review added
Milestone: Backlog → Alpha 11

comment:2 by Kieran P, 12 years ago

Keywords: handleNetMessage netstatus authenticated connected removed

comment:3 by historic_bruno, 12 years ago

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 by ben, 12 years ago

Owner: set to ben
Resolution: → fixed
Status: new → closed

In 11981:

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

Note: See TracTickets for help on using tickets.