Summary

Fix up several fields in the admin UI for models.

Review Request #4556 — Created Sept. 11, 2013 and discarded Sept. 11, 2013, 12:49 p.m.

Information

Owner

ziis

Repository

Review Board

Branch

Bugs

Depends On

Commit

f560679...

Reviewers

Groups

django-evolution

People

abcd, ziis

Description

While adding some entries to the docs database for the diff file
attachments, I realized we had some missing fields in the admin UI and
some fields that should have been limited to raw IDs. The file
attachment fields on ReviewRequest weren't exposed at all.

This change adds the missing fields and marks up a few to require raw
IDs.

Also, some fields weren't allowed to be blank, even though they should
have been.

Testing Done

This is a review from Review Bot.
Tool: PEP8 Style Checker
Processed Files:
/reviewboard/diffviewer/admin.py
/reviewboard/diffviewer/models.py
/reviewboard/reviews/admin.py
Ignored Files:

This is a review from Review Bot.
Tool: Pyflakes
Processed Files:
/reviewboard/diffviewer/admin.py
/reviewboard/diffviewer/models.py
/reviewboard/reviews/admin.py
Ignored Files:

People:: abcd

People:: ziis

```
check
```

This is a production server. Please do not use it for testing--you just spammed a lot of people.

You can use http://demo.reviewboard.org/ for tests. Username "guest" password "guest"

Status:: Discarded