{"comment":{"url":"https://api.github.com/repos/biplovbhandari/eden/comments/11968211","html_url":"https://github.com/biplovbhandari/eden/commit/3ba23ece9743a92d7692e81ce0831dce859d37de#commitcomment-11968211","id":11968211,"user":{"login":"nursix","id":1257183,"avatar_url":"https://avatars.githubusercontent.com/u/1257183?v=3","gravatar_id":"","url":"https://api.github.com/users/nursix","html_url":"https://github.com/nursix","followers_url":"https://api.github.com/users/nursix/followers","following_url":"https://api.github.com/users/nursix/following{/other_user}","gists_url":"https://api.github.com/users/nursix/gists{/gist_id}","starred_url":"https://api.github.com/users/nursix/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nursix/subscriptions","organizations_url":"https://api.github.com/users/nursix/orgs","repos_url":"https://api.github.com/users/nursix/repos","events_url":"https://api.github.com/users/nursix/events{/privacy}","received_events_url":"https://api.github.com/users/nursix/received_events","type":"User","site_admin":false},"position":277,"line":1990,"path":"modules/s3db/cap.py","commit_id":"3ba23ece9743a92d7692e81ce0831dce859d37de","created_at":"2015-07-02T06:29:18Z","updated_at":"2015-07-02T06:29:18Z","body":"This whole block doesn't make sense to me? resource.components is a dict, and you iterate over it's keys in order to find a key? Who does that?\r\n\r\nAnd then, very ugly, you try/except the case where this loop hasn't found the key, and hence not re-assigned the same variable, and hence you access a non-existent \"link\" attribute of it?\r\n\r\nSorry, but this is not only sub-standard, this is a completely unacceptable coding style."}}