)]}'
{"keystone/assignment/controllers.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"e8ef1f2b5b34ab49094b0b44d5e7782e504c3464","unresolved":false,"context_lines":[{"line_number":685,"context_line":"                # log a warning"},{"line_number":686,"context_line":"                target \u003d \u0027Unknown\u0027"},{"line_number":687,"context_line":"                # Should always be a domain or project, but since to get"},{"line_number":688,"context_line":"                # here things have gone astray, let\u0027s be cautious."},{"line_number":689,"context_line":"                if \u0027scope\u0027 in ref:"},{"line_number":690,"context_line":"                    if \u0027domain\u0027 in ref[\u0027scope\u0027]:"},{"line_number":691,"context_line":"                        dom_id \u003d ref[\u0027scope\u0027][\u0027domain\u0027].get(\u0027id\u0027, \u0027Unknown\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_33f049c9","line":688,"updated":"2014-08-25 17:21:44.000000000","message":"I suppose this online comment still belongs above the target definition above","commit_id":"e15b105ed7d04e0cf7fe2720c2ed6634f0e29730"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"a8c604d3038f5d343109909c82b2957de060a156","unresolved":false,"context_lines":[{"line_number":686,"context_line":"                target \u003d \u0027Unknown\u0027"},{"line_number":687,"context_line":"                # Should always be a domain or project, but since to get"},{"line_number":688,"context_line":"                # here things have gone astray, let\u0027s be cautious."},{"line_number":689,"context_line":"                if \u0027scope\u0027 in ref:"},{"line_number":690,"context_line":"                    if \u0027domain\u0027 in ref[\u0027scope\u0027]:"},{"line_number":691,"context_line":"                        dom_id \u003d ref[\u0027scope\u0027][\u0027domain\u0027].get(\u0027id\u0027, \u0027Unknown\u0027)"},{"line_number":692,"context_line":"                        target \u003d \u0027Domain: %s\u0027 % dom_id"}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_6f572827","line":689,"updated":"2014-08-26 01:47:05.000000000","message":"using scope \u003d ref.get(\u0027scope\u0027) would remove a lot of duplication throughout here.","commit_id":"e15b105ed7d04e0cf7fe2720c2ed6634f0e29730"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"a8c604d3038f5d343109909c82b2957de060a156","unresolved":false,"context_lines":[{"line_number":695,"context_line":"                        target \u003d \u0027Project: %s\u0027 % proj_id"},{"line_number":696,"context_line":"                role_id \u003d \u0027Unknown\u0027"},{"line_number":697,"context_line":"                if \u0027role\u0027 in ref and \u0027id\u0027 in ref[\u0027role\u0027]:"},{"line_number":698,"context_line":"                    role_id \u003d ref[\u0027role\u0027][\u0027id\u0027]"},{"line_number":699,"context_line":"                LOG.warning("},{"line_number":700,"context_line":"                    _(\u0027Group %(group)s not found for role-assignment - \u0027"},{"line_number":701,"context_line":"                      \u0027%(target)s with Role: %(role)s\u0027), {"}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_2f7610d9","line":698,"updated":"2014-08-26 01:47:05.000000000","message":"lines 696-698 could be\n\n role_id \u003d ref.get(\u0027role\u0027, {\u0027id\u0027: \u0027Unknown\u0027).get(\u0027id\u0027, \u0027Unknown\u0027)","commit_id":"e15b105ed7d04e0cf7fe2720c2ed6634f0e29730"}]}