Opened 10 years ago

Closed 10 years ago

#9687 closed enhancement (fixed)

dojox.data.GoogleFeedStore should delete the script domn ode after it was loaded

Reported by: virsir Owned by: Shane O'Sullivan
Priority: high Milestone: 1.4
Component: DojoX Data Version: 1.3.2
Keywords: Cc:
Blocked By: Blocking:

Description

If use dojox.data.GoogleFeedStore? to load a lot of feeds, there will be a lot of script dom nodes.
It is better remove this dom node after loaded.

Change History (3)

comment:1 Changed 10 years ago by Jared Jurkiewicz

Owner: changed from Jared Jurkiewicz to Shane O'Sullivan

comment:2 Changed 10 years ago by Shane O'Sullivan

(In [19810]) Refs #9687 Fixes the issue where the GoogleSearchStore? does not delete script elements used for JSONP communication. It now deletes all used script elements, and a test has been added to verify this.

The sort function was moved to get around a check in error that stated that the return type of minus one was of a different type to plus one.

comment:3 Changed 10 years ago by Shane O'Sullivan

Resolution: fixed
Status: newclosed

This issue has been addressed, all script elements are now deleted upon execution.

Note: See TracTickets for help on using tickets.