Bug Tracker

Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#11435 closed bug (fixed)

Obsolete test code to remove toJSON from .data's return value

Reported by: amonat@… Owned by:
Priority: low Milestone: 1.8
Component: data Version: git
Keywords: Cc:
Blocked by: Blocking:

Description

The ".data()" test in test/unit/data.js has this code (in 2 places):

	// TODO: Remove this hack which was introduced for 1.5.1
	delete dataObj.toJSON;

Commit d5f144a7 changed the way jQuery.data() works so that the .toJSON isn't included, so I think this code is no longer necessary. I removed it and the test still passed.

Change History (4)

comment:1 Changed 6 years ago by addyosmani

Priority: undecidedlow
Status: newopen

This was just mentioned in -meeting and it looks like we're going to try addressing it. Marking as valid.

comment:2 Changed 5 years ago by Andy Monat

Resolution: fixed
Status: openclosed

Fix #11435. Remove obsolete test code.

Changeset: dc337f628c0d2a4f6d691851eb95a622599e38b2

comment:3 Changed 5 years ago by dmethvin

Milestone: None1.8

comment:4 Changed 5 years ago by dmethvin

Component: unfileddata
Note: See TracTickets for help on using tickets.