nasa/mct — pull request #267 by VWoeltjen (2014-03-13 21:21:54+00:00)
Title: [Save] Obey policy from View Modified dialog

Addresses nasa/mct#243 by refactoring View Modified dialog:

* Consolidate view modified dialog code (which can be initiated from the inspector, the standard housing, and the change view action) into one location. This avoids duplication, and permits the resolution of the issue in one place.
* Utilize Save action to complete Save when chosen by the user from this dialog. Previously, Save was effectively reimplemented across these dialogs, which permitted the omission of policy considerations. Utilizing the actual Save action means that any such considerations addressed there will also be accounted for by the View Modified dialog.

As a result, choosing Save when there are nested modified objects with mixed ownership permits only owned components to be actually persisted, with appropriate notification to the user about this change.

```
Wrote or re-wrote unit tests (Y/N): Y*
Ran entire suite of unit tests successfully (Y/N): Y
Verified bug fix with the described replication steps or verified the implementation meets the requirement cited (Y/N): Y
Code coverage requirements satisfied (new code=80%, changed code=at least as high as existing) (Y/N): Y
Code review date and attendees: Self-review by pull request
Describe potential impact on other areas of the code: View switching, window closing
Describe any test prerequisites for this fix (e.g., restart the database): N/A
For bugs marked "Regression - Yes", describe the cause of the regression: N/A
Did you modify any pom.xml file? If yes, did you thoroughly document your changes and reasons? N/A
```

New tests for new class not needed, as coverage in this class was ensured by existing tests for the various situations in which a View Modified dialog might be launch.