)]}'
{"glance/db/sqlalchemy/api.py":[{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"dd118565a296fcfe482998219b538581a3928797","unresolved":false,"context_lines":[{"line_number":510,"context_line":"                crit_attrs.append((attr \u003d\u003d marker_values[j]))"},{"line_number":511,"context_line":""},{"line_number":512,"context_line":"            model_attr \u003d getattr(model, sort_keys[i])"},{"line_number":513,"context_line":"            default \u003d None if isinstance(model_attr.property.columns[0].type,"},{"line_number":514,"context_line":"                                         sqlalchemy.DateTime) else \u0027\u0027"},{"line_number":515,"context_line":"            attr \u003d sa_sql.expression.case([(model_attr !\u003d None,"},{"line_number":516,"context_line":"                                          model_attr), ],"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F%2Bg%2FM%3D","line":513,"updated":"2014-03-26 09:04:37.000000000","message":"Would you mind breaking this like:\n\n  default None\n  if not isinstance(model_attr.property.columns[0].type, sqlalchemy.DateTime):\n      default \u003d \u0027\u0027\n\n(same for the previous one)","commit_id":"33d1840d6a9c1d0b10b94576c7f3f25ffae51bd1"},{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"d91f6a5385940d1ed01c983fd8bad8669f5266e1","unresolved":false,"context_lines":[{"line_number":510,"context_line":"                crit_attrs.append((attr \u003d\u003d marker_values[j]))"},{"line_number":511,"context_line":""},{"line_number":512,"context_line":"            model_attr \u003d getattr(model, sort_keys[i])"},{"line_number":513,"context_line":"            default \u003d None if isinstance(model_attr.property.columns[0].type,"},{"line_number":514,"context_line":"                                         sqlalchemy.DateTime) else \u0027\u0027"},{"line_number":515,"context_line":"            attr \u003d sa_sql.expression.case([(model_attr !\u003d None,"},{"line_number":516,"context_line":"                                          model_attr), ],"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F%2Bg9c%3D","line":513,"in_reply_to":"AAAAWH%2F%2Bg%2FM%3D","updated":"2014-03-26 09:06:57.000000000","message":"ignore this comment, this should have been done in master. I hadn\u0027t realized it is a backport.","commit_id":"33d1840d6a9c1d0b10b94576c7f3f25ffae51bd1"}]}