Opened 8 years ago

Closed 8 years ago

#16313 closed defect (fixed)

dojox/mobile/Heading breaks if busy property is set to false before it is ever set to true

Reported by: Colin Snover Owned by: Eric Durocher
Priority: undecided Milestone: 1.9
Component: DojoX Mobile Version: 1.8.1
Keywords: Cc: cjolif
Blocked By: Blocking:

Description (last modified by Colin Snover)

The _setBusyAttr function does not check that prog actually exists in the case that busy is set to false. Fixing #16312 breaks it.

Attachments (1)

patch16313.patch​ (1.2 KB) - added by Adrian Vasiliu 8 years ago.
Now Heading supports setting the busy property to false before being set to true. Added test case to DOH test. - Adrian Vasiliu, IBM, CCLA

Download all attachments as: .zip

Change History (5)

comment:1 Changed 8 years ago by Colin Snover

Description: modified (diff)

comment:2 Changed 8 years ago by cjolif

Cc: cjolif added

comment:3 Changed 8 years ago by cjolif

Milestone: tbd → 1.9

Changed 8 years ago by Adrian Vasiliu

Attachment: patch16313.patch​ added

Now Heading supports setting the busy property to false before being set to true. Added test case to DOH test. - Adrian Vasiliu, IBM, CCLA

comment:4 Changed 8 years ago by cjolif

Resolution: → fixed
Status: new → closed

In [29945]:

fixes #16313. Now Heading supports setting the busy property to false before being set to true. Added test case to DOH test. Thanks Adrian Vasiliu (IBM, CCLA).

Note: See TracTickets for help on using tickets.