)]}'
{"openstackclient/identity/v3/project.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c924ad14b8d6c7770ca970184e9445d1fcbecdd6","unresolved":false,"context_lines":[{"line_number":338,"context_line":"        if project._info.get(\u0027subtree\u0027):"},{"line_number":339,"context_line":"            project._info[\u0027subtree\u0027] \u003d [str(p[\u0027project\u0027][\u0027id\u0027])"},{"line_number":340,"context_line":"                                        for p in project._info[\u0027subtree\u0027]]"},{"line_number":341,"context_line":"        domain_id \u003d project._info.get(\u0027domain_id\u0027)"},{"line_number":342,"context_line":"        domain \u003d common.find_domain(identity_client,domain_id)"},{"line_number":343,"context_line":"        domain_name \u003d  domain._info.get(\u0027name\u0027)"},{"line_number":344,"context_line":"        project._info[\u0027domain_name\u0027] \u003d domain_name"},{"line_number":345,"context_line":""},{"line_number":346,"context_line":"        project._info.pop(\u0027links\u0027)"},{"line_number":347,"context_line":"        return zip(*sorted(six.iteritems(project._info)))"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a98d1f4_d314a845","line":344,"range":{"start_line":341,"start_character":8,"end_line":344,"end_character":50},"updated":"2016-09-30 06:38:31.000000000","message":"consider wrapping this in a try/except since it\u0027s possible that a non-admin user won\u0027t have the necessary roles to view a domain by ID.","commit_id":"9e3e0d4b6989b0773acae686b9bf2f5e14ba6ead"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"8be68bca260fa0afa6c64b82a5be92c304293896","unresolved":false,"context_lines":[{"line_number":341,"context_line":"        domain_id \u003d project._info.get(\u0027domain_id\u0027)"},{"line_number":342,"context_line":"        domain \u003d common.find_domain(identity_client, domain_id)"},{"line_number":343,"context_line":"        domain_name \u003d domain._info.get(\u0027name\u0027)"},{"line_number":344,"context_line":"        project._info[\u0027domain_name\u0027] \u003d domain_name"},{"line_number":345,"context_line":""},{"line_number":346,"context_line":"        project._info.pop(\u0027links\u0027)"},{"line_number":347,"context_line":"        return zip(*sorted(six.iteritems(project._info)))"}],"source_content_type":"text/x-python","patch_set":2,"id":"da6895a0_c30dbe91","line":344,"updated":"2016-10-25 09:33:02.000000000","message":"Is there a specific reason to do the intermediate assignment to domain_name?","commit_id":"947e2dc7707c06fa4b7dd9518a3143c1d132705e"}]}