Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#8338 closed defect (fixed)

[dojox.atom] destroy usage + cleanups

Reported by: dante Owned by: Jared Jurkiewicz
Priority: high Milestone: 1.3
Component: General Version: 1.2.3
Keywords: Cc:
Blocked By: Blocking:

Description

Migrating to use dojo.destroy (#7975) in dojox.atom.Feed* I noticed some chances for serious code reduction. Attaching as separate patch here for your consideration.

Attachments (2)

atom.patch​ (4.3 KB) - added by dante 10 years ago.
repetition reduction and dojo.destroy
atom.2.patch​ (4.4 KB) - added by Jared Jurkiewicz 10 years ago.
Fixing an error made in the code reduction. Passed tests.

Download all attachments as: .zip

Change History (4)

Changed 10 years ago by dante

Attachment: atom.patch​ added

repetition reduction and dojo.destroy

Changed 10 years ago by Jared Jurkiewicz

Attachment: atom.2.patch​ added

Fixing an error made in the code reduction. Passed tests.

comment:1 Changed 10 years ago by Jared Jurkiewicz

Resolution: → fixed
Status: new → closed

(In [16221]) Cleanup to use dojo.destroy, + some code reduction. fixes #8338

comment:2 Changed 10 years ago by Adam Peller

Milestone: tbd → 1.3

batch move of tickets marked 'tbd' fixed in the 1.3 timeframe

Note: See TracTickets for help on using tickets.