{"comment":{"url":"https://api.github.com/repos/umbraco/OurUmbraco/comments/10019999","html_url":"https://github.com/umbraco/OurUmbraco/commit/18380b824a53be75339dde5d51b89650d588e42f#commitcomment-10019999","id":10019999,"user":{"login":"Shazwazza","id":1742685,"avatar_url":"https://avatars.githubusercontent.com/u/1742685?v=3","gravatar_id":"","url":"https://api.github.com/users/Shazwazza","html_url":"https://github.com/Shazwazza","followers_url":"https://api.github.com/users/Shazwazza/followers","following_url":"https://api.github.com/users/Shazwazza/following{/other_user}","gists_url":"https://api.github.com/users/Shazwazza/gists{/gist_id}","starred_url":"https://api.github.com/users/Shazwazza/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Shazwazza/subscriptions","organizations_url":"https://api.github.com/users/Shazwazza/orgs","repos_url":"https://api.github.com/users/Shazwazza/repos","events_url":"https://api.github.com/users/Shazwazza/events{/privacy}","received_events_url":"https://api.github.com/users/Shazwazza/received_events","type":"User","site_admin":false},"position":20,"line":20,"path":"our.umbraco.org/Examine/ProjectsIndexDataService/ProjectNodeIndexDataService.cs","commit_id":"18380b824a53be75339dde5d51b89650d588e42f","created_at":"2015-03-04T00:38:08Z","updated_at":"2015-03-04T00:38:08Z","body":"Argh! more null checking please!!! You can't just assume something isn't going to be null. For example because of this the project index cannot rebuild with new data because :\r\n\r\nproject.GetProperty(\"description\").Value \r\n\r\nreturns null and then you do a ToString() on a null and we get 'object reference' errors. Really you also need to be checking if this is null too:\r\n\r\nproject.GetProperty(\"description\") \r\n\r\nsince if that is null, then doing a .Value will also give you 'object reference' errors."}}