Summary:

Fix keyerror when submitting a clearcase review request containing binary files changes

Review Request #6758 — Created Jan. 12, 2015 and submitted Jan. 14, 2015, 1:39 p.m.

Information
Owner:	delyn
Repository:	Review Board
Branch:	release-2.0.x
Bugs:
Depends On:
Commit:	17420cf...
Reviewers
Groups:	reviewboard
People:

Description

Binary files has no origFile and newFile defined in their header

PS: It was originally reported by Piotr Zwara on github, I have not found any issue in reviewboard bugtracker
https://github.com/reviewboard/reviewboard/commit/2939338493658ce6424d1efc63e010e4b8290bb8#commitcomment-6777854

Testing Done

can upload successfully a clearcase review request containing binary file changes against reviewboard server 2.0.6 + this fix

Expand All
Collapse All

Tool: Pyflakes
Processed Files:
    reviewboard/scmtools/clearcase.py



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/scmtools/clearcase.py

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-2.0.x (e3dd672)