[rbseverity] Fix the severity extension to properly support general comments

Review Request #9205 — Created Sept. 23, 2017 and submitted — Latest diff uploaded

Information

rb-extension-pack
master
09e348b...

Reviewers

The severity of a problem was not being displayed on the review page, if
the severity was added using the add comment button on the review dialog
page.

The drop downs menus for severity options on the edit review page
were not saving the model with the selected option. The model not saving
the selected option caused the data not be saved in the database when,
submitting the form.

The patch fixes the problem by saving the model anytime the severity
changes in the drop down. When the severity is saved it is properly added
to the database, and this fixes the problem on the review page.

I added a comment of each severity using the add comment button on the
reviewable page. On the review dialog page I added another comment with a
severity and published it from this page. I repeated the process adding
multiple comments with varying degrees of severity, and different
amounts of comments.

I ran the js tests with the extension enabled and all expected tests
passed. I had 2 failures, they seemed unrelated and other people were
experiencing them, and they are being filed separately.

All tickets had proper severity in the database and displayed it on the
reviewable page.

    Loading...