)]}'
{"nova/db/sqlalchemy/api.py":[{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"2c1ad5cde421471f900286d5b1a9c09b98b3ee31","unresolved":false,"context_lines":[{"line_number":2809,"context_line":"                         filter_by(deleted\u003dFalse).\\"},{"line_number":2810,"context_line":"                         filter_by(parent_group_id\u003dsecurity_group_id).\\"},{"line_number":2811,"context_line":"                         options(joinedload_all(\u0027grantee_group\u0027)).\\"},{"line_number":2812,"context_line":"                         all()"},{"line_number":2813,"context_line":"    return result"},{"line_number":2814,"context_line":""},{"line_number":2815,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAACX%2F%2F%2Bos%3D","side":"PARENT","line":2812,"updated":"2011-09-27 04:36:35.000000000","message":"Do we not need a \u0027if result and result.project_id: authorize_project_context(result.project_id)\u0027 call here?  I notice the removal of the comment.","commit_id":"1c8957abd216d983b56bde0bdf3338ca5a730b15"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"54970d1164c3c95fb168b0916cc644f06ebdae43","unresolved":false,"context_lines":[{"line_number":2809,"context_line":"                         filter_by(deleted\u003dFalse).\\"},{"line_number":2810,"context_line":"                         filter_by(parent_group_id\u003dsecurity_group_id).\\"},{"line_number":2811,"context_line":"                         options(joinedload_all(\u0027grantee_group\u0027)).\\"},{"line_number":2812,"context_line":"                         all()"},{"line_number":2813,"context_line":"    return result"},{"line_number":2814,"context_line":""},{"line_number":2815,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAACX%2F%2F%2Bmk%3D","side":"PARENT","line":2812,"in_reply_to":"AAAACX%2F%2F%2Bms%3D","updated":"2011-09-27 09:31:09.000000000","message":"Alrighty","commit_id":"1c8957abd216d983b56bde0bdf3338ca5a730b15"},{"author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"change_message_id":"9e7f192f787a031afeb82d0a354ada7f3d5aa213","unresolved":false,"context_lines":[{"line_number":2809,"context_line":"                         filter_by(deleted\u003dFalse).\\"},{"line_number":2810,"context_line":"                         filter_by(parent_group_id\u003dsecurity_group_id).\\"},{"line_number":2811,"context_line":"                         options(joinedload_all(\u0027grantee_group\u0027)).\\"},{"line_number":2812,"context_line":"                         all()"},{"line_number":2813,"context_line":"    return result"},{"line_number":2814,"context_line":""},{"line_number":2815,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAACX%2F%2F%2Bms%3D","side":"PARENT","line":2812,"in_reply_to":"AAAACX%2F%2F%2Bos%3D","updated":"2011-09-27 08:27:12.000000000","message":"I removed the comment because I think it was copied from a section above.  I don\u0027t think it is necessary.  You can authorize groups from other projects, so I don\u0027t think there is any reason to filter them out here.","commit_id":"1c8957abd216d983b56bde0bdf3338ca5a730b15"}]}