Summary

Load the richText field in several resource models.

Review Request #4800 — Created Oct. 18, 2013 and submitted Oct. 21, 2013, 1:54 p.m.

Information

Owner

chipx86

Repository

Review Board

Branch

master

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

Load the richText field in several resource models.

Now that the richText field is added, load it in when parsing payloads
for the appropriate models. Currently, this is not used for anything
but information.

Unit tests were added for these fields.

Testing Done

Unit tests pass.

Used these in an upcoming change.

This is a review from Review Bot.
Tool: PEP8 Style Checker
Processed Files:
Ignored Files:
reviewboard/static/rb/js/resources/models/tests/screenshotCommentModelTests.js
reviewboard/static/rb/js/resources/models/reviewReplyModel.js
reviewboard/static/rb/js/resources/models/tests/reviewReplyModelTests.js
reviewboard/static/rb/js/resources/models/tests/fileAttachmentCommentModelTests.js
reviewboard/static/rb/js/resources/models/baseCommentModel.js
reviewboard/static/rb/js/resources/models/draftReviewRequestModel.js
reviewboard/static/rb/js/resources/models/tests/reviewModelTests.js
reviewboard/static/rb/js/resources/models/reviewRequestModel.js
reviewboard/static/rb/js/resources/models/baseCommentReplyModel.js
reviewboard/static/rb/js/resources/models/reviewModel.js
reviewboard/static/rb/js/resources/models/tests/diffCommentModelTests.js

This is a review from Review Bot.
Tool: Pyflakes
Processed Files:
Ignored Files:
reviewboard/static/rb/js/resources/models/tests/screenshotCommentModelTests.js
reviewboard/static/rb/js/resources/models/reviewReplyModel.js
reviewboard/static/rb/js/resources/models/tests/reviewReplyModelTests.js
reviewboard/static/rb/js/resources/models/tests/fileAttachmentCommentModelTests.js
reviewboard/static/rb/js/resources/models/baseCommentModel.js
reviewboard/static/rb/js/resources/models/draftReviewRequestModel.js
reviewboard/static/rb/js/resources/models/tests/reviewModelTests.js
reviewboard/static/rb/js/resources/models/reviewRequestModel.js
reviewboard/static/rb/js/resources/models/baseCommentReplyModel.js
reviewboard/static/rb/js/resources/models/reviewModel.js
reviewboard/static/rb/js/resources/models/tests/diffCommentModelTests.js

Change Summary:

Missed a couple.

Diff:

Revision 2 (+36 -5)

Show changes

	reviewboard/static/rb/js/resources/models/baseCommentModel.js
	reviewboard/static/rb/js/resources/models/baseCommentReplyModel.js
	reviewboard/static/rb/js/resources/models/draftReviewRequestModel.js
	reviewboard/static/rb/js/resources/models/reviewModel.js
	reviewboard/static/rb/js/resources/models/reviewReplyModel.js
	reviewboard/static/rb/js/resources/models/reviewRequestModel.js
	reviewboard/static/rb/js/resources/models/tests/baseCommentReplyModelTests.js
	reviewboard/static/rb/js/resources/models/tests/diffCommentModelTests.js
	5 more

This is a review from Review Bot.
Tool: PEP8 Style Checker
Processed Files:
Ignored Files:
reviewboard/static/rb/js/resources/models/tests/draftReviewRequestModelTests.js
reviewboard/static/rb/js/resources/models/tests/screenshotCommentModelTests.js
reviewboard/static/rb/js/resources/models/reviewReplyModel.js
reviewboard/static/rb/js/resources/models/tests/reviewReplyModelTests.js
reviewboard/static/rb/js/resources/models/tests/fileAttachmentCommentModelTests.js
reviewboard/static/rb/js/resources/models/baseCommentModel.js
reviewboard/static/rb/js/resources/models/draftReviewRequestModel.js
reviewboard/static/rb/js/resources/models/tests/reviewModelTests.js
reviewboard/static/rb/js/resources/models/reviewRequestModel.js
reviewboard/static/rb/js/resources/models/baseCommentReplyModel.js
reviewboard/static/rb/js/resources/models/tests/baseCommentReplyModelTests.js
reviewboard/static/rb/js/resources/models/reviewModel.js
reviewboard/static/rb/js/resources/models/tests/diffCommentModelTests.js

This is a review from Review Bot.
Tool: Pyflakes
Processed Files:
Ignored Files:
reviewboard/static/rb/js/resources/models/tests/draftReviewRequestModelTests.js
reviewboard/static/rb/js/resources/models/tests/screenshotCommentModelTests.js
reviewboard/static/rb/js/resources/models/reviewReplyModel.js
reviewboard/static/rb/js/resources/models/tests/reviewReplyModelTests.js
reviewboard/static/rb/js/resources/models/tests/fileAttachmentCommentModelTests.js
reviewboard/static/rb/js/resources/models/baseCommentModel.js
reviewboard/static/rb/js/resources/models/draftReviewRequestModel.js
reviewboard/static/rb/js/resources/models/tests/reviewModelTests.js
reviewboard/static/rb/js/resources/models/reviewRequestModel.js
reviewboard/static/rb/js/resources/models/baseCommentReplyModel.js
reviewboard/static/rb/js/resources/models/tests/baseCommentReplyModelTests.js
reviewboard/static/rb/js/resources/models/reviewModel.js
reviewboard/static/rb/js/resources/models/tests/diffCommentModelTests.js

Ship it!

```
Ship It!
```

Status:: Completed