{"action":"created","issue":{"url":"https://api.github.com/repos/lodash/lodash/issues/1186","labels_url":"https://api.github.com/repos/lodash/lodash/issues/1186/labels{/name}","comments_url":"https://api.github.com/repos/lodash/lodash/issues/1186/comments","events_url":"https://api.github.com/repos/lodash/lodash/issues/1186/events","html_url":"https://github.com/lodash/lodash/pull/1186","id":73640159,"number":1186,"title":"sortByOrder can sort properties with null values","user":{"login":"ignu","id":40636,"avatar_url":"https://avatars.githubusercontent.com/u/40636?v=3","gravatar_id":"","url":"https://api.github.com/users/ignu","html_url":"https://github.com/ignu","followers_url":"https://api.github.com/users/ignu/followers","following_url":"https://api.github.com/users/ignu/following{/other_user}","gists_url":"https://api.github.com/users/ignu/gists{/gist_id}","starred_url":"https://api.github.com/users/ignu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ignu/subscriptions","organizations_url":"https://api.github.com/users/ignu/orgs","repos_url":"https://api.github.com/users/ignu/repos","events_url":"https://api.github.com/users/ignu/events{/privacy}","received_events_url":"https://api.github.com/users/ignu/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/lodash/lodash/labels/change","name":"change","color":"eb6420"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-05-06T14:29:51Z","updated_at":"2015-05-14T18:56:37Z","closed_at":"2015-05-07T06:40:45Z","pull_request":{"url":"https://api.github.com/repos/lodash/lodash/pulls/1186","html_url":"https://github.com/lodash/lodash/pull/1186","diff_url":"https://github.com/lodash/lodash/pull/1186.diff","patch_url":"https://github.com/lodash/lodash/pull/1186.patch"},"body":"Without this fix, sortByOrder will just silently leave collections will null sorted properties partially sorted. \r\n\r\nSee http://codepen.io/ignu/pen/RPPOwO?editors=001\r\n\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/lodash/lodash/issues/comments/102135303","html_url":"https://github.com/lodash/lodash/pull/1186#issuecomment-102135303","issue_url":"https://api.github.com/repos/lodash/lodash/issues/1186","id":102135303,"user":{"login":"mathieumg","id":1624812,"avatar_url":"https://avatars.githubusercontent.com/u/1624812?v=3","gravatar_id":"","url":"https://api.github.com/users/mathieumg","html_url":"https://github.com/mathieumg","followers_url":"https://api.github.com/users/mathieumg/followers","following_url":"https://api.github.com/users/mathieumg/following{/other_user}","gists_url":"https://api.github.com/users/mathieumg/gists{/gist_id}","starred_url":"https://api.github.com/users/mathieumg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mathieumg/subscriptions","organizations_url":"https://api.github.com/users/mathieumg/orgs","repos_url":"https://api.github.com/users/mathieumg/repos","events_url":"https://api.github.com/users/mathieumg/events{/privacy}","received_events_url":"https://api.github.com/users/mathieumg/received_events","type":"User","site_admin":false},"created_at":"2015-05-14T18:56:37Z","updated_at":"2015-05-14T18:56:37Z","body":"Just to make sure: after this change and when sorting an array of objects using their `myProperty` property  which is a `Number`, should I expect an object that doesn't have `myProperty` (i.e. it is `undefined` for all intents and purposes) to come before `0` or after ̀`Math.Infinity`? \r\n\r\nI think my actual use case is not uncommon, I sort in descending order by a number property and objects who don't have the property in question (most of them) are considered to have a value of 0 for that property, implicitly. They need to go at the end (since it's descending) of everything, and not before the high values. I got it working using a custom iteratee but if this PR means I won't need to do that anymore that's even better! Thank you."}}