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 comment:1
Changed November 10, 2010 04:28PM UTC by comment:2
| milestone: | 1.4.3 → 1.4.5 |
|---|---|
| owner: | → kswedberg |
| priority: | → undecided |
| status: | new → pending |
That patch seems to be gone, karl, any chance we can see it again?
Changed November 18, 2010 07:52PM UTC by comment:3
| status: | pending → assigned |
|---|
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 comment:4
| resolution: | → invalid |
|---|---|
| status: | assigned → closed |
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 comment:5
| version: | 1.4.2 → 1.5 |
|---|
There wasn't a 1.4.5 release, was actually 1.5.
Changed January 31, 2011 05:54PM UTC by comment:6
| milestone: | 1.4.5 → 1.5 |
|---|
There was no 1.4.5 release, was actually 1.5.
#5896 is a duplicate of this ticket.