COMP: Ensure custom qt style plugins are fixed-up on MacOSX

This is a follow-up of commit r22238 ensuring the style plugins
are considered as candidate by the fixup script.

See #3253

Signed-off-by: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22684 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: Fixed volume rendering presets issue (presets were not displayed)

The scene model RowCache was temporarily outdated during node insertion. If the cache was accessed during node insertion, the qMRMLSceneModel::indexFromNode method indicated that the node doesn't exist yet.
Fixed by adding an entry to the cache with the node-to-be-added, which results in not using the cache to determine if the node exists or not but uses the traditional way of iterating through all the items.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22685 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: Avoid confusion creating enum for custom SaveDataDialog role.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22686 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: Properly support extension

Revert commit r22148 that was setting the path to saved MRB as MRMLScene URL.
The MRML scene URL knows remains associated with the mrml file after either
a read or save to mrml, or even export (MRB, Kiwi.zip, ...) operation.

To be able to properly switch between different available file formats
while supporting filename containing dots. The extension of a filename
needs to be removed based on a list of known extensions. For that purpose,
the following two functions have been introduced:
  - extractKnownExtension
  - stripKnownExtension

These functions rely on the existing IOManager function "fileWriterExtensions"
able to return a list of name filters associated with either a storage node
or a mrml scene.

For any fileName item, in order to get the associated storage node or mrml scene,
a new role named "UIDRole" has been introduced. For a given fileName item,
if a nodeID is set with the "UIDRole", it is understood that it will
be the nodeID of the corresponding storage node. If it is not set, it means
the fileName item is associated the scene.

See #2948

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22687 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: In Slicer session, keep track of Scene file format used to save

Within a given a Slicer session, the selected Scene file format will
be remembered if the scene could be successfully saved/exported.
Note that the corresponding value won't be saved into the settings.

See #2948

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22688 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: dicom header viewer added to DICOMLib using ctkDICOMObjectListWidget

ENH: update DICOM module for Table View

New interface to support better searching and header display.

Now plugins are selectable to avoid wasting time on unneeded plugins.

Plugins are now evaluated on-demand, to simplify the workflow.

http://www.commontk.org/index.php/CTK-Hackfest-May-2013#DICOM_Database_and_Networking

https://github.com/commontk/CTK/pull/361

ENH: update to latest CTK for header widget

Also fix lingering rebase issue (stray variable).

BUG: Give header popup independent geometry setting

Also fix incorrect end of line character

ENH: examine button diabled until needed

STYLE: remove tabs in code

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22689 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: MultiVolumeExplorer - Anticipate logic headers reorganization.

Soon, as a compilation speedup, the base class "vtkMRMLAbstractLogic"
will forward declared classes in its definition file.

This commit fix compilation issue that would have occurred following
integration of the topic "finetjul/Slicer@vtkmrmlabstractlogic-clean-includes-trunk"

Fixes #3494

Co-authored-by: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>

From: Andrey Fedorov <fedorov@bwh.harvard.edu>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22693 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: avoid python attribute error on dicom context menu

TODO: the elements are currently disabled since we don't have
corresponding implementations for the new dicom widget.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22694 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: Support vtkStringArray as model field array

Closes #3344

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22695 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: minor edits for 4.3 documentation

Fix formatting errors, attributions, phrasing.



git-svn-id: http://svn.slicer.org/Slicer4/trunk@22697 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: Added test for MRML node reference serialization and parsing
Issue #3416

From: Csaba Pinter <csaba.pinter@queensu.ca>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22700 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: Slicer 4.3.1-1

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22702 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Revert  tweak release 4.3.1-1 commit done on trunk

This commit revert the previous commit r22702. This commit should be
done on the Slicer-4-3-1-1 branch instead of the trunk.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22703 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: updating docs for 4.3

Added description of including fiber bundles, scroll to, and show/hide all.



git-svn-id: http://svn.slicer.org/Slicer4/trunk@22705 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Propagate cmake variable Slicer_QTSCRIPTEDMODULES_DISABLED

to the Slicer inner build configuration.

It originally was a typo: eq_superbuild_extra_args instead of
ep_superbuild_extra_args which was latter replaced into
EXTERNAL_PROJECT_OPTIONAL_ARGS

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22706 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: Dynamically populate top-level module categories

Closes #3399

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22707 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Remove commented/unused CMake code from External_ITKv4.cmake

See #1739

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22709 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Remove "-fopenmp" hack from External_ITKv4

This hack was first introduced while depending on ITK v4.0a07 (as
specified in Slicer r17596 comment). Following v4.0a07, ITK
build system has been updated to properly deal with the
issue (See kitware/ITK@8a135bcb).

Considering that
  (1) Slicer now depends on a version of ITK including
commit kitware/ITK@8a135bcb
  (2) variables "ITK_CMAKE_C_FLAGS" and "ITK_CMAKE_CXX_FLAGS" have
never been used to configure ITKv4 project
  (3) variable "CMAKE_CXX_FLAGS" was improperly referenced as "CMAKE_CX_FLAGS"

the corresponding hack is removed from the external project.

See #1739

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22710 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Explicitly set CTKDICOMCore as dependency of qSlicerBaseQTCore

Since qSlicerCoreApplication.h include ctkDICOMDatabase.h, this commit
add an explicit dependency between the two libraries.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22711 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: OpenIGTLinkIF - Anticipate logic headers reorganization.

Soon, as a compilation speedup, the base class "vtkMRMLAbstractLogic"
will forward declared classes in its definition file.

This commit fix compilation issue that would have occurred following
integration of the topic "finetjul/Slicer@vtkmrmlabstractlogic-clean-includes-trunk"

Fixes #3493

Co-authored-by: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>

From: Junichi Tokuda <tokuda@bwh.harvard.edu>

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