Fix the New Review Request page with missing commit data.
Review Request #7446 — Created June 22, 2015 and submitted
The New Review Request page's commit list was expecting a valid date for
each commit, and crashed if one wasn't provided. This would occur if dealing
with an inaccessible commit on Subversion.We now set a flag if the commit is inaccessible, and change our presentation
for that commit. The commit will say "Revision XYZ (not accessible on this
repository)", and won't be available to post for review.
Hard-coded all the data for a commit to being blank, and tried viewing the
commit list. Saw that the data, while useless, didn't look flat-out broken.Tried posting inaccessible commits for review, and couldn't. I could still
post accessible commits.
| Description | From | Last Updated |
|---|---|---|
|
Should this be gettext-ed? |
 brennie
|
-
-
reviewboard/static/rb/js/newReviewRequest/views/commitView.js (Diff revision 1) Should this be
gettext-ed?
Change Summary:
Gettextized some text.
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 2 (+8 -5) |
-
Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/newReviewRequest/views/commitView.js reviewboard/static/rb/js/resources/models/repositoryCommitModel.js Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/newReviewRequest/views/commitView.js reviewboard/static/rb/js/resources/models/repositoryCommitModel.js
Change Summary:
- Inaccessible commits now appear greyed out and say "Revision XYZ (not accessible on this repository)".
- They also can no longer be posted for review, and do not show links to an associated review request.
Description: |
|
|||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Testing Done: |
|
|||||||||||||||||||||||||||||||||
Commit: |
|
|||||||||||||||||||||||||||||||||
Diff: |
Revision 3 (+55 -17) |
-
Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/newReviewRequest/views/commitView.js reviewboard/static/rb/css/newReviewRequest.less reviewboard/static/rb/js/resources/models/repositoryCommitModel.js Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/newReviewRequest/views/commitView.js reviewboard/static/rb/css/newReviewRequest.less reviewboard/static/rb/js/resources/models/repositoryCommitModel.js
Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/newReviewRequest/views/commitView.js reviewboard/static/rb/js/resources/models/repositoryCommitModel.js Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/newReviewRequest/views/commitView.js reviewboard/static/rb/js/resources/models/repositoryCommitModel.js