Summary

Fix KeyError: u'message' in SVN commits backend.

Review Request #6361 — Created Sept. 23, 2014 and submitted Sept. 23, 2014, 3:51 p.m.

Information

Owner

david

Repository

Review Board

Branch

release-2.0.x

Bugs

3579

Depends On

Commit

d4ccd17...

Reviewers

Groups

reviewboard

People

Description

Not every commit we get has a message, which can cause exceptions to be thrown
from the repository commits resource. We already had one fix for this sort of
error with the author fields. This change fixes it up to do the same for
message and author everywhere.

Testing Done

Ran unit tests.

Issues

Description	From	Last Updated
What about these instances of commit['message'] and commit['author']?	ST stilor	Sept. 29, 2014, 1:19 p.m.

Tool: Pyflakes
Processed Files:
    reviewboard/scmtools/svn/__init__.py



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

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to release-2.0.x (99fa229)

Fix it!

reviewboard/scmtools/svn/__init__.py (Diff revision 1)
An issue was opened. Show all issues
```
What about these instances of commit['message'] and commit['author']?
```

BTW, this change was checked in with a wrong reference to the issue number (3579 instead of 3570).