Opened 8 years ago

Closed 6 years ago

Last modified 6 years ago

#14125 closed defect (fixed)

duplicate case statement in dojox.xmpp.xmppSession (1.7rc1)

Reported by: lee Owned by: Dustin Machi
Priority: high Milestone: 1.9
Component: Dojox Version: 1.7.0b1
Keywords: Cc:
Blocked By: Blocking:

Description

Might not be important but I was using an IDE to build an app I'm writing and it was failing validation because in module:

in dojox.xmpp.xmppSession

function:

presenceUpdate()

there's a duplicate case statement for

case : "status"

Change History (5)

comment:1 Changed 8 years ago by Adam Peller

Owner: changed from Adam Peller to Dustin Machi

comment:2 Changed 7 years ago by mm

is anyone going to fix this ? please ?

comment:3 Changed 6 years ago by dylan

Resolution: fixed
Status: newclosed

In [30734]:

fixes #14125, lint clean-up for dojox/xmpp/xmppSession, !strict

comment:4 Changed 6 years ago by dylan

The case:status is intentional... this syntax means that for both status and show, the same block of code will be called.

That said, I found quite a bit of other crufty code in here, so I cleaned-up what I could.

comment:5 Changed 6 years ago by bill

Milestone: tbd1.9
Note: See TracTickets for help on using tickets.