Labels: Type-Defect Priority-Medium
New issue 781 by marc.bee: missing exception in
What's the URL of the page this enhancement relates to, if any?
The part of the code is used when creating a new review request
Describe the enhancement and the motivation for it.
This is just to avoid a future bug, two cases will delete the review
request instance and a few lines later, the function will try to save it.
to sum-up, the code below may be executed:
review_request.save() # will raise an exception.
You should raise a new exception for instance:
if 'path' in diff_form.errors:
self.errors['diff_path'] = diff_form.errors['path']
raise NewExceptionError # here
elif 'base_diff_path' in diff_form.errors:
raise NewExceptionError # and here
self.errors['diff_path'] = forms.util.ErrorList([
'The selected file does not appear to be a diff.'])
except Exception, e:
self.errors['diff_path'] = forms.util.ErrorList([e])
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
You received this message because you are subscribed to the Google Groups
To post to this group, send email to firstname.lastname@example.org
To unsubscribe from this group, send email to
For more options, visit this group at