Move more review request editing responsibilities to the model.

Review Request #4105 — Created May 5, 2013 and submitted — Latest diff uploaded

Information

Review Board
master

Reviewers

Move more review request editing responsibilities to the model.

While moving things into ReviewRequestEditor*, much of the logic needed
to be in the view until we were more or less complete with the transfer.

Now, draft publishing, field setting and validation live in the model.
The view listens for events and calls into the model for more things.

The ReviewRequest model and associated draft are now instantiated with
the content from the database, meaning we can now work with the raw
content in the model instead of processing data in the DOM.

This last part had an impact on load order in reviews.js, so some things
had to be shifted around.
Tested manually setting each field.

Tested that a first edit of a review request still brought up the draft
banner.

Tested change descriptions.

Tested editing and reverting fields.

Tested validation of reviewers, summary and description before publish.

Tested successful publishes.

Tested publishing with some fields open causing the fields to save before
publish.

All unit tests (including new ones) pass.
    Loading...