STYLE: updated documentation on selection node

Updated doxygen comments to note that the PlaceNodeClassName
calls replace the AnnotationNodeID calls. 
Fixed a merge error on the enum, fixing the overwriting of event
numbers and moved the UnitModifiedEvent to be last.
Updated doxygen comments on the annotation fiducial node to
note it as deprecated moving the top level description so that
it shows up on the generated pages.



git-svn-id: http://svn.slicer.org/Slicer4/trunk@22480 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: update Markups doxygen documentation

Moved sections so that they'll be visible on the generated pages.
Expanded documentation on the node level versus markup level settings.
Added separate comments for each documented method.



git-svn-id: http://svn.slicer.org/Slicer4/trunk@22481 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: update the CLIs to create new Markup fiduicals from point node selector

The command line module GUI helper hadn't been updated to create Markups
Fiducial nodes by default when creating new nodes was enabled. It was still
set up to create Slicer3 vtkMRMLFiducialListNodes. 
Now by default a vtkMRMLMarkupsFiducialNode will be created when Create New is
selected.
Update the Markups module logic to detect node added events and add a display
node if not in batch processing, import or restore modes.
Updated the simple region growing segmentation help documentation to refer to the
Markups module.

Issue #3417



git-svn-id: http://svn.slicer.org/Slicer4/trunk@22482 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: code clean up to unify setting the active place node

Added a utility function in the logic to set the active place node,
matching the get active place node id method. Call it where the
selection node was being accessed, and make sure that the testing
and setting of the ActivePlaceNodeClassName and ActivePlaceNodeID
are done only when there is a change and with the Reference call
so that events are triggered.
Added a test of the new SetActiveListID method.



git-svn-id: http://svn.slicer.org/Slicer4/trunk@22483 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: Fix loading of MRC files

This commit reverts r22292

Commit r22292 updated ITK external project to explicitly
set Module_ITKReview and also removed the setting of ITK_USE_REVIEW.

Considering that since r22163, Slicer is built against the ITK release
branch and that commits deprecating ITK_USE_REVIEW have not been
back ported to the release branch, commit r22292 had the effect of:
  -> having an ITK built without module review at all.
  -> preventing the MRC IO plugin from being loaded
  -> preventing MRC file from being read into Slicer

Suggested-by: Bradley Lowekamp <blowekamp@mail.nih.gov>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22484 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Fix bug in writing origins to MGH files

Waiting Slicer makes use of the ITK remote module "itkMGHImageIO" (issue #3410),
this commit has been back-ported from Slicer/itkMGHImageIO@13f5e11ef.

It also has been tested by Hans as part of ITK.

Test project /Users/johnsonhj/Dashboard/src/ITK-ccache
Start 2065: MGHIOInDoxygenGroup
1/7 Test #2065: MGHIOInDoxygenGroup .............. Passed 0.09 sec
Start 2066: MGHFactoryCreationTest
2/7 Test #2066: MGHFactoryCreationTest ........... Passed 0.15 sec
Start 2067: MGHReadImagesTest_mgz
3/7 Test #2067: MGHReadImagesTest_mgz ............ Passed 3.77 sec
Start 2068: MGHReadImagesTest_mgh
4/7 Test #2068: MGHReadImagesTest_mgh ............ Passed 0.49 sec
Start 2069: MGHReadImagesTest_mgh.gz
5/7 Test #2069: MGHReadImagesTest_mgh.gz ......... Passed 3.88 sec
Start 2070: itkMGHIOInternalTests
6/7 Test #2070: itkMGHIOInternalTests ............ Passed 0.07 sec
Start 2071: itkMGHIOOriginTest
7/7 Test #2071: itkMGHIOOriginTest ............... Passed 3.58 sec

More details here: http://review.source.kitware.com/#/c/12771/

From: Kent Williams <norman-k-williams@uiowa.edu>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22485 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: Fix compile error introduced when backporting fix for MGH origin issue

The compile issue hasn't been detected earlier because only a visual check
between the original patch (Slicer/itkMGHImageIO@13f5e11ef) and
the back-ported one has been done (r22485).

This commit fixes the build issue and also confirm that the associated
Slicer tests now pass:

// --------------
Start 545: py_nomainwindow_test_slicer_mgh
[...]
test_saveShortCompressedNode (test_slicer_mgh.SlicerUtilLoadSaveMGHTests) ... ok
test_saveUnCompressedNode (test_slicer_mgh.SlicerUtilLoadSaveMGHTests) ... ok
// --------------

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22486 3bd1e089-480b-0410-8dfb-8563597acbee