)]}'
{"contrib/designate-dashboard/designatedashboard/dashboards/project/dns_domains/forms.py":[{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"ac113b8589ca19e35cb6237c210993617fe2d044","unresolved":false,"context_lines":[{"line_number":108,"context_line":"                    self.api_error("},{"line_number":109,"context_line":"                        _(\u0027Error Type: %(type)s\u0027)"},{"line_number":110,"context_line":"                        % {"},{"line_number":111,"context_line":"                            \"message\": ex.type"},{"line_number":112,"context_line":"                        })"},{"line_number":113,"context_line":" "},{"line_number":114,"context_line":"            return False"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba7be1f8_d8e28000","line":111,"updated":"2015-02-23 17:26:19.000000000","message":"this needs to change to \"type\" eg:\n\n  \"type\": ex.type","commit_id":"d8a366989c77d2face94cf894b3b5abd3cbcf639"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"ac113b8589ca19e35cb6237c210993617fe2d044","unresolved":false,"context_lines":[{"line_number":199,"context_line":"                    self.api_error("},{"line_number":200,"context_line":"                        _(\u0027Error Type: %(type)s\u0027)"},{"line_number":201,"context_line":"                        % {"},{"line_number":202,"context_line":"                            \"message\": ex.type"},{"line_number":203,"context_line":"                        })"},{"line_number":204,"context_line":" "},{"line_number":205,"context_line":"            return False"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba7be1f8_18edf8ef","line":202,"updated":"2015-02-23 17:26:19.000000000","message":"as above","commit_id":"d8a366989c77d2face94cf894b3b5abd3cbcf639"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"ac113b8589ca19e35cb6237c210993617fe2d044","unresolved":false,"context_lines":[{"line_number":463,"context_line":"                        })"},{"line_number":464,"context_line":"                else:"},{"line_number":465,"context_line":"                    self.api_error("},{"line_number":466,"context_line":"                        _(\u0027Error Type: %(type)s\u0027)"},{"line_number":467,"context_line":"                        % {"},{"line_number":468,"context_line":"                            \"message\": ex.type"},{"line_number":469,"context_line":"                        })"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba7be1f8_d8c9a07a","line":466,"updated":"2015-02-23 17:26:19.000000000","message":"as above","commit_id":"d8a366989c77d2face94cf894b3b5abd3cbcf639"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"ac113b8589ca19e35cb6237c210993617fe2d044","unresolved":false,"context_lines":[{"line_number":537,"context_line":"                    self.api_error("},{"line_number":538,"context_line":"                        _(\u0027Error Type: %(type)s\u0027)"},{"line_number":539,"context_line":"                        % {"},{"line_number":540,"context_line":"                            \"message\": ex.type"},{"line_number":541,"context_line":"                        })"},{"line_number":542,"context_line":"            return False"},{"line_number":543,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ba7be1f8_38a2bcae","line":540,"updated":"2015-02-23 17:26:19.000000000","message":"as above","commit_id":"d8a366989c77d2face94cf894b3b5abd3cbcf639"},{"author":{"_account_id":8094,"name":"Vinod Mangalpally","email":"mrvinod@outlook.com","username":"vinodmr"},"change_message_id":"cd0e40b844be1f4bc84245572962ac620152dc59","unresolved":false,"context_lines":[{"line_number":95,"context_line":"                    _(\u0027Error: %(message)s (Request ID: %(request_id)s)\u0027)"},{"line_number":96,"context_line":"                    % {"},{"line_number":97,"context_line":"                        \"request_id\": ex.request_id,"},{"line_number":98,"context_line":"                        \"message\": ex.message"},{"line_number":99,"context_line":"                    })"},{"line_number":100,"context_line":"            else:"},{"line_number":101,"context_line":"                if ex.message is not None:"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_9a366eb3","line":98,"updated":"2015-02-27 15:39:45.000000000","message":"How about here and other similar places below - would ex.message always be valid?","commit_id":"8c9fa2964463b1f9bc01a4258cd8552c52a20c1b"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"026043c8dcfdcfd39227c4c015ba89dc7d8f7feb","unresolved":false,"context_lines":[{"line_number":95,"context_line":"                    _(\u0027Error: %(message)s (Request ID: %(request_id)s)\u0027)"},{"line_number":96,"context_line":"                    % {"},{"line_number":97,"context_line":"                        \"request_id\": ex.request_id,"},{"line_number":98,"context_line":"                        \"message\": ex.message"},{"line_number":99,"context_line":"                    })"},{"line_number":100,"context_line":"            else:"},{"line_number":101,"context_line":"                if ex.message is not None:"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_3a78222b","line":98,"in_reply_to":"ba7be1f8_9a366eb3","updated":"2015-02-27 15:43:23.000000000","message":"nope - good catch. \nWe have really bad error handling currently, and I am not sure there is a 500 error that has a message.\n\nEndre: the if else block used below should be replicated for the HTTP 500 errors as well","commit_id":"8c9fa2964463b1f9bc01a4258cd8552c52a20c1b"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"8686de8e3b65c7ba91bae4ec271fb19eac81460d","unresolved":false,"context_lines":[{"line_number":51,"context_line":"                data[\"type\"] \u003d ex.type"},{"line_number":52,"context_line":"                msg +\u003d \"Error Type: %(type)s\""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"            msg +\u003d \" (Request ID: %(request_id)s\""},{"line_number":55,"context_line":"            data[\"request_id\"] \u003d ex.request_id"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"            form.api_error(_(msg) % data)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a80dd14_eae11ed1","line":54,"updated":"2015-03-05 22:16:49.000000000","message":"We should be only showing the Req ID on errors over 500","commit_id":"add041df0f473007f1984c61128eec7adb19fe92"}]}