Skip to main content

Bug Tracker

Side navigation

#6345 closed enhancement (invalid)

Opened March 24, 2010 05:27PM UTC

Closed November 18, 2010 09:08PM UTC

Last modified January 31, 2011 05:54PM UTC

Allow .data() to accept a function for its setter argument

Reported by: kswedberg Owned by: kswedberg
Priority: undecided Milestone: 1.5
Component: data Version: 1.5
Keywords: Cc:
Blocked by: Blocking:
Description

This would be nice, as it aligns .data() with .css(), .attr(), and all the domManip methods that take a function setter.

I've committed a patch to my fork on Github, along with tests:

http://github.com/kswedberg/jquery/commit/1b5f3836fd63522bdf0ec41d21aa859db05904f0

Attachments (0)
Change History (6)

Changed November 10, 2010 01:25AM UTC by dmethvin comment:1

#5896 is a duplicate of this ticket.

Changed November 10, 2010 04:28PM UTC by SlexAxton comment:2

milestone: 1.4.31.4.5
owner: → kswedberg
priority: → undecided
status: newpending

That patch seems to be gone, karl, any chance we can see it again?

Changed November 18, 2010 07:52PM UTC by kswedberg comment:3

status: pendingassigned

Hey Alex, Sorry about that. I think I annihilated the fork/branch that had that commit. I'll try to get something committed again.

Changed November 18, 2010 09:08PM UTC by kswedberg comment:4

resolution: → invalid
status: assignedclosed

ajpiano pointed out that .data() should be able to store functions just as it stores any other type. so treating functions differently ( by calling them ) wouldn't be a good idea. Closing this ticket. :'(

Changed January 31, 2011 05:53PM UTC by john comment:5

version: 1.4.21.5

There wasn't a 1.4.5 release, was actually 1.5.

Changed January 31, 2011 05:54PM UTC by john comment:6

milestone: 1.4.51.5

There was no 1.4.5 release, was actually 1.5.