)]}'
{"nova/db/sqlalchemy/api.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"357629f4884f0cfa2ccafeefaee0bb1a94620fb5","unresolved":false,"context_lines":[{"line_number":4943,"context_line":"    query \u003d model_query(context, models.Migration)"},{"line_number":4944,"context_line":"    if \"status\" in filters:"},{"line_number":4945,"context_line":"        status \u003d filters[\"status\"]"},{"line_number":4946,"context_line":"        status \u003d [status] if isinstance(status, six.string_types) else status"},{"line_number":4947,"context_line":"        query \u003d query.filter(models.Migration.status.in_(status))"},{"line_number":4948,"context_line":"    if \"host\" in filters:"},{"line_number":4949,"context_line":"        host \u003d filters[\"host\"]"}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_084a7283","line":4946,"updated":"2016-05-13 11:08:15.000000000","message":"nit: status \u003d status if isinstance(status, list) else [status] \n\nmight be slightly more clear and less prone to silly python string type gorps.\n\nmakes me wonder if sending ?host\u003dfoo\u0026host\u003dbar doesn\u0027t explode this?","commit_id":"11babbbb939f546a981f9faf429051722393e802"}]}