Fix openIssue checkbox set to default when reloading the page with explicitly set value

Review Request #6867 — Created Feb. 1, 2015 and submitted — Latest diff uploaded

Information

Review Board
release-2.0.x
29720ca...

Reviewers

In the "View Diff" page, click one line in diff content to open one comment dialog
and create one comment WITHOUT issue opened. And reload that view diff page, click
the flag of that comment we just created, the "IssueOpened" checkbox is checked in
the dialog. Close it and reopen it again, the checked status is OK.

Since the status setting is missed, we set the issueOpened status and issued status
when creating the draftcomment in the comment block.

Added a unit test, which passes along with all other tests.

Tested with a commentEditor which contains a comment whose 'loaded' attribute and
'issueOpened' attribute are false and the default setting is true. Previously, the
editor's 'openIssue' attribute is true. After, the attribute will set to user's setting
with the patch.

Test for some cases, all passed.

    Loading...