)]}'
{"tempest/services/image/v1/json/images_client.py":[{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"231ecdfc2b34a1425d0dea3f458ec6f5bd5ffc73","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":215,"context_line":"        # Added status check for user with admin role"},{"line_number":216,"context_line":"        try:"},{"line_number":217,"context_line":"            if isinstance(self.show_image(id), dict) \u0026\u0026 self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":218,"context_line":"                return True"},{"line_number":219,"context_line":"        except lib_exc.NotFound:"},{"line_number":220,"context_line":"            return True"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba0121b8_592e468c","line":217,"range":{"start_line":217,"start_character":15,"end_line":217,"end_character":52},"updated":"2016-04-01 17:08:32.000000000","message":"I don\u0027t think is necessary to check if the object is a dictionary, if it is not the the except will return True.","commit_id":"8c57859cd13f043d2672a8d8e3a40a4045413920"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"231ecdfc2b34a1425d0dea3f458ec6f5bd5ffc73","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":215,"context_line":"        # Added status check for user with admin role"},{"line_number":216,"context_line":"        try:"},{"line_number":217,"context_line":"            if isinstance(self.show_image(id), dict) \u0026\u0026 self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":218,"context_line":"                return True"},{"line_number":219,"context_line":"        except lib_exc.NotFound:"},{"line_number":220,"context_line":"            return True"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba0121b8_be1ae87a","line":217,"range":{"start_line":217,"start_character":12,"end_line":217,"end_character":108},"updated":"2016-04-01 17:08:32.000000000","message":"Your line is longer than the allowed characters.","commit_id":"8c57859cd13f043d2672a8d8e3a40a4045413920"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"4be4fc4f7b11ba06a119dd212afc7e6312f2efa4","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":215,"context_line":"        # Added status check for user with admin role"},{"line_number":216,"context_line":"        try:"},{"line_number":217,"context_line":"            image_status \u003d self.show_image(id)"},{"line_number":218,"context_line":"            if \u0027image\u0027 in image_status:"},{"line_number":219,"context_line":"                if image_status[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":220,"context_line":"                    return True"},{"line_number":221,"context_line":"        except lib_exc.NotFound:"},{"line_number":222,"context_line":"            return True"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba0121b8_c7fd480a","line":219,"range":{"start_line":217,"start_character":12,"end_line":219,"end_character":64},"updated":"2016-04-01 20:26:06.000000000","message":"All of this can go on a single line identical as the reference fix you talke (296072)... you might have to also update unit test class","commit_id":"afaa03cd7b22cc00fb75619ecf9eea79c82a070c"}]}
