Allow removing multiple data keys at once with $.fn.removeData

[[email protected]…]

I just saw some code in Wijmo that had a string of chained removeData() calls, and it occurred to me there should probably be a way to remove multiple data keys at once. Not sure whether this is a worthwhile addition (I know there are better ways they could have done this to avoid needing to remove one key at a time), but if so, I'd suggest an API that accepts either a string containing a single key (which is what currently works) or an array of keys.

[SlexAxton]

Seems fair. Any desire to write a patch? :D

[anonymous]

Yes :) Didn't want to submit one unless it was something people liked.

[rmurphey]

Replying to anonymous:

Yes :) Didn't want to submit one unless it was something people liked.

Sorry, that was me, I wasn't logged in.

[rmurphey]

Pull request 77 submitted, if you want it.

[snover]

​pull request

[john]

We should consider this for 1.7.

[john]

Nominating ticket for 1.7 discussion.

[Rick Waldron]

+1, Should be consistent with existing API

[jaubourg]

+1, makes sense

[rmurphey]

Per the discussion on the pull request (​https://github.com/jquery/jquery/pull/77) there needs to be a decision as to the API. If we go with a single string, which is consistent with existing APIs ...

$('#foo').removeData('a b c');

... then we make it impossible to remove a key with a space in it. If we go with an array, as I did in the pull request ...

$('#foo').removeData([ 'a', 'b', 'c' ]);

... then we can allow the removal of any valid key, either one or many at a time.

I have no horse in this race; I can see the merits of both solutions.

[timmywil]

+1, In favor, but I don't think space-separated by itself is enough.

[ajpiano]

+1, I don't know if we want to break space-separated keys for 1.7 - I like the array, even thought it's "inconsistent" with other aspects of the API

[john]

+1, Let's do this.

[scottgonzalez]

+1, space delimited string, like everything else

[Rick Waldron]

Scott, keep in mind that data properties can be any valid string

​https://github.com/jquery/jquery/pull/77#issuecomment-665309

illustrated here:

​http://jsfiddle.net/rwaldron/Bt4nK/

But then again, we could always just say that strings with spaces are prohibited

[jzaefferer]

+0

[Corey Frang]

Landing pull request 512. 1.7 - removeData now takes space separated lists and arrays of keys - Fixes #7323.

More Details:

• ​https://github.com/jquery/jquery/pull/512
• ​http://bugs.jquery.com/ticket/7323 Changeset: 9b3768b968bc99d3422355e69e2c7c35bedf681e