)]}'
{"cinder/api/contrib/quota_classes.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"43ffb10b14f42971d3fba9ec2a8db2f8ef56d365","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        quota_class \u003d id"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        for key, value in body[\u0027quota_class_set\u0027].items():"},{"line_number":64,"context_line":"            try:"},{"line_number":65,"context_line":"                db.quota_class_update(context, quota_class, key, value)"},{"line_number":66,"context_line":"            except exception.QuotaClassNotFound:"},{"line_number":67,"context_line":"                db.quota_class_create(context, quota_class, key, value)"},{"line_number":68,"context_line":"            except exception.AdminRequired:"},{"line_number":69,"context_line":"                raise webob.exc.HTTPForbidden()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        quota_set \u003d QUOTAS.get_class_quotas(context, quota_class)"},{"line_number":72,"context_line":"        group_quota_set \u003d GROUP_QUOTAS.get_class_quotas(context, quota_class)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f6a8fd7_b8afcce1","line":69,"range":{"start_line":63,"start_character":7,"end_line":69,"end_character":47},"updated":"2018-04-19 12:01:30.000000000","message":"It would be good to place try/except block outside of for statement.","commit_id":"3da328be9d892922176d2d11c3f09ea73aaf3611"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"9ea15b41eaa1c3faf669783a3fa4531c5fccb133","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        quota_class \u003d id"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        for key, value in body[\u0027quota_class_set\u0027].items():"},{"line_number":64,"context_line":"            try:"},{"line_number":65,"context_line":"                db.quota_class_update(context, quota_class, key, value)"},{"line_number":66,"context_line":"            except exception.QuotaClassNotFound:"},{"line_number":67,"context_line":"                db.quota_class_create(context, quota_class, key, value)"},{"line_number":68,"context_line":"            except exception.AdminRequired:"},{"line_number":69,"context_line":"                raise webob.exc.HTTPForbidden()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        quota_set \u003d QUOTAS.get_class_quotas(context, quota_class)"},{"line_number":72,"context_line":"        group_quota_set \u003d GROUP_QUOTAS.get_class_quotas(context, quota_class)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f6a8fd7_70e21bd7","line":69,"range":{"start_line":63,"start_character":7,"end_line":69,"end_character":47},"in_reply_to":"9f6a8fd7_b8afcce1","updated":"2018-04-26 08:50:18.000000000","message":"Obviously, we can\u0027t place try/except block outside of loop statement:(","commit_id":"3da328be9d892922176d2d11c3f09ea73aaf3611"}]}