Ticket #6345 (closed enhancement: invalid)
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: | ||
| Blocking: | Blocked by: |
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
Change History
comment:2 Changed 3 years ago by SlexAxton
- Owner set to kswedberg
- Priority set to undecided
- Status changed from new to pending
- Milestone changed from 1.4.3 to 1.4.5
That patch seems to be gone, karl, any chance we can see it again?
comment:3 Changed 3 years ago by kswedberg
- Status changed from pending to 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.
comment:4 Changed 3 years ago by kswedberg
- Status changed from assigned to closed
- Resolution set to invalid
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. :'(
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
#5896 is a duplicate of this ticket.