{"comment":{"url":"https://api.github.com/repos/umbraco/OurUmbraco/comments/9949481","html_url":"https://github.com/umbraco/OurUmbraco/commit/038e58632c1900f873b67af536f888f527209047#commitcomment-9949481","id":9949481,"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":51,"line":62,"path":"our.umbraco.org/Examine/ProjectsIndexDataService/ProjectNodeIndexDataService.cs","commit_id":"038e58632c1900f873b67af536f888f527209047","created_at":"2015-02-27T01:29:43Z","updated_at":"2015-02-27T01:29:43Z","body":"Never ever iterate over an enumerable this way and use ElementAt(i). Why has this been done? This means that for every single iteration, the entire collection will be re-enumerated.\r\n\r\nIn any case, this entire logic needs to be fixed, there is severe N+1 queries going on here. I'll fix it."}}