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
BUG: Explicitly tell the user if extensions can't be installed

In case either the Slicer settings file or, the extensions install
directory are not readable/writable, a nicely formatted message will
be displayed to the user instead of the app-store page,

Note that the associated messages are also logged.

Fixes #3179

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22487 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Update ITKv4 project

$ git shortlog a8429da..97a08e0

Matthew McCormick (3):
      BUG: Enable MRC IO in UseITK whenever the ITKReview module is enabled.
      Merge branch 'MRC-UseITK' into release
      COMP: Fix NIFTI CMake variable separation warning.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22521 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: 3388. Added observer events parameter to the vtkMRMLNode::AddNodeReferenceRole() method, so that the events can be set only once in subclasses. Added observed events to DisplayableNode and TransformableNode. Simplified and unified reference implementation in TransformableNode

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22523 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: 3413. Fixed crash on writing a custom reference.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22524 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Fix "unused-but-set-variable" warning in qMRMLItemDelegate

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22525 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: Fix copy/paste in python console / Update CTK projects

$ git shortlog 0e21d85e..90000c3

Jean-Christophe Fillion-Robin (1):
      Merge pull request #366 from finetjul/294-fix-ctkconsole-copy-paste

Julien Finet (5):
      Fix copy/paste in ctkConsole
      Add private implementation to ctkCoordinatesWidget
      Merge branch '355-ctkcoordinateswidget-pimpl'
      Add ctkCoordinatesWidget_p.h
      Merge branch '355-ctkcoordinateswidget-pimpl'

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22526 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: 3388. Fixed crash in test when referenceRole=0

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22527 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: convert to integer explicitly to avoid ambiguity

Fixes #3420


git-svn-id: http://svn.slicer.org/Slicer4/trunk@22528 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: fix the grey slice viewers labels to be distinctive

Per comment from Jim, see issue 2384


git-svn-id: http://svn.slicer.org/Slicer4/trunk@22529 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: fix the numbering of slice viewer labels

In the situations when Yellow/Green/Red viewers are present, the additional viewers should be numbered starting from 4.


git-svn-id: http://svn.slicer.org/Slicer4/trunk@22530 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: adding 3x3 layout

see issue 2384


git-svn-id: http://svn.slicer.org/Slicer4/trunk@22531 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: reverting previous change as it caused compile error on Win

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22532 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: qMRMLLayoutManagerTest(s) - Fix (Selection|Interaction)Node warnings

This commit will fix the following warnings:

147: Warning: In /home/jchris/Projects/Slicer/Libs/MRML/DisplayableManager/vtkMRMLAbstractDisplayableManager.cxx, line 583
147: vtkMRMLThreeDReformatDisplayableManager (0x2c92750): CreateIfPossible - MRMLScene does NOT contain any InteractionNode
147:
147: Warning: In /home/jchris/Projects/Slicer/Libs/MRML/DisplayableManager/vtkMRMLAbstractDisplayableManager.cxx, line 591
147: vtkMRMLThreeDReformatDisplayableManager (0x2c92750): CreateIfPossible - MRMLScene does NOT contain any SelectionNode

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

Ensure ModelDisplayableManager properly delete resources when
a new scene is set.

Ensure the slicer logic set the scene on layers when a new
scene is set a second time.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22534 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: Re-organize qMRMLLayoutManagerTest(1|2|3|4) for easier readability

Factor out code into convenience functions:
 - checkViewArrangement
 - checkNumberOfItems

Use vtkNew when possible

There is no need to explicitly delete the layout manager since
it was a QWidget instantiated on the stack as a parent.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22535 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: Update "removeSliceView" to be consistent with other remote methods

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22536 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: Update qMRMLLayoutManagerTest2 to check for reference count

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