)]}'
{"cinderclient/utils.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"e681ed10e8c3999f36540ac172a603bb09a790ee","unresolved":false,"context_lines":[{"line_number":135,"context_line":"                    field_name \u003d field.replace(\u0027 \u0027, \u0027_\u0027)"},{"line_number":136,"context_line":"                else:"},{"line_number":137,"context_line":"                    field_name \u003d field.lower().replace(\u0027 \u0027, \u0027_\u0027)"},{"line_number":138,"context_line":"                if type(o) \u003d\u003d dict and field in o:"},{"line_number":139,"context_line":"                    data \u003d o[field]"},{"line_number":140,"context_line":"                else:"},{"line_number":141,"context_line":"                    data \u003d getattr(o, field_name, \u0027\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a80dd14_b31d4313","line":138,"updated":"2015-03-16 19:52:44.000000000","message":"Do we actually need this one?  I think we can just use data \u003d getattr... and skip the second if/else here.\n\nRegardless, the enhancement here is nice, I\u0027ll approve it, just wanted to note above incase somebody wants to come back and fix it up.","commit_id":"0f73c5fb8ab593da7e332ade273e7da31ea8e303"}]}