{"action":"created","issue":{"url":"https://api.github.com/repos/jashkenas/backbone/issues/3494","labels_url":"https://api.github.com/repos/jashkenas/backbone/issues/3494/labels{/name}","comments_url":"https://api.github.com/repos/jashkenas/backbone/issues/3494/comments","events_url":"https://api.github.com/repos/jashkenas/backbone/issues/3494/events","html_url":"https://github.com/jashkenas/backbone/issues/3494","id":57660461,"number":3494,"title":"Acceptable values for `models` arg of collection.remove()","user":{"login":"jmm","id":129203,"avatar_url":"https://avatars.githubusercontent.com/u/129203?v=3","gravatar_id":"","url":"https://api.github.com/users/jmm","html_url":"https://github.com/jmm","followers_url":"https://api.github.com/users/jmm/followers","following_url":"https://api.github.com/users/jmm/following{/other_user}","gists_url":"https://api.github.com/users/jmm/gists{/gist_id}","starred_url":"https://api.github.com/users/jmm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmm/subscriptions","organizations_url":"https://api.github.com/users/jmm/orgs","repos_url":"https://api.github.com/users/jmm/repos","events_url":"https://api.github.com/users/jmm/events{/privacy}","received_events_url":"https://api.github.com/users/jmm/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/jashkenas/backbone/labels/change","name":"change","color":"27823f"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-02-13T22:06:26Z","updated_at":"2015-02-16T14:51:52Z","closed_at":null,"body":"The current documentation implies that only model objects are acceptable, but under the hood it uses `collection.get()`, which will take a model object, `id`, or `cid`. Is it supposed to be restricted to just model objects or can it be documented as accepting the same values as `get()`?\r\n"},"comment":{"url":"https://api.github.com/repos/jashkenas/backbone/issues/comments/74519205","html_url":"https://github.com/jashkenas/backbone/issues/3494#issuecomment-74519205","issue_url":"https://api.github.com/repos/jashkenas/backbone/issues/3494","id":74519205,"user":{"login":"jmm","id":129203,"avatar_url":"https://avatars.githubusercontent.com/u/129203?v=3","gravatar_id":"","url":"https://api.github.com/users/jmm","html_url":"https://github.com/jmm","followers_url":"https://api.github.com/users/jmm/followers","following_url":"https://api.github.com/users/jmm/following{/other_user}","gists_url":"https://api.github.com/users/jmm/gists{/gist_id}","starred_url":"https://api.github.com/users/jmm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmm/subscriptions","organizations_url":"https://api.github.com/users/jmm/orgs","repos_url":"https://api.github.com/users/jmm/repos","events_url":"https://api.github.com/users/jmm/events{/privacy}","received_events_url":"https://api.github.com/users/jmm/received_events","type":"User","site_admin":false},"created_at":"2015-02-16T14:51:52Z","updated_at":"2015-02-16T14:51:52Z","body":"> In theory it can contain a combination of each of those, right?\r\n\r\nYou mean?:\r\n\r\n```js\r\nsome_collection.remove([\r\n  // A model `id`\r\n  '123',\r\n  // A model `cid`\r\n  'c123',\r\n  // A model object\r\n  some_model\r\n])\r\n```\r\n\r\nIt currently does that in practice (uses `collection.get()` under the hood). That's my expectation of what it's supposed to do in theory, and how I described it in my [pull request](https://github.com/jashkenas/backbone/pull/3495)."}}