Status: New Owner: ---- Labels: Type-Enhancement Priority-Medium New issue 1217 by madcoder: Enhanced error detection when uploading diff http://code.google.com/p/reviewboard/issues/detail?id=1217
What version are you running? ReviewBoard-1.0alpha4 What's the URL of the page this enhancement relates to, if any? Selecting "Update Diff" on an existing review Describe the enhancement and the motivation for it. Occasionally, if there's a problem with a subsequent diff when trying to update an existing review, the diff file will be accepted and stored with no error checking, then trying to view that diff will result in a python exception. You can then update the review again with a new, correct diff, and viewing that revision of the review will work correctly, however that "bad" revision is permanently in the review, and will break every time you try to view it. If the "Update Diff" functionality had a preview, the uploader could choose to either publish or discard that updated diff. To that end, reviewboard could attempt the shell `patch` call, and upon exception, automatically display an error for the new diff instead of accepting the diff as a permanent part of the review. I'll try to work up an example that shows this behavior and respond with an attachment. What operating system are you using? What browser? Mac OS X 10.5.7; Firefox 3.0.11 Please provide any additional information below. -- 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: http://code.google.com/hosting/settings --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/reviewboard-issues?hl=en -~----------~----~----~----~------~----~------~--~---
