Allow an ignore list on constructing SpectrumDetectorMapping.

Enables filtering of given IDs when the list is first constructed.
Refs #7890
Fix spectrum numbering for ISIS event files.

Uses new ignore id feature on SpectrumDetectorMapping and also
sets the spectrum numbers on the resized EventWorkspace to those from
the file.
The default numbering scheme of starting at 1 is not sufficient as
a monitor block may be start of the table.
Refs #7890
Revert "Fix spectrum numbering for ISIS event files."

This reverts commit da7c0c809f4ad082b899fb0f461eefacceea6433.
Correct fix for the spectrum numbering in ISIS event files.

Refs #7890
Correction of SANS adding data

revert changes made in #7600 that was build over a bug in
loading event data.

re #7890
Restore selection after dock updates. Re #8032
Track renaming in the ADS to keep selection in the dock. Re #8032
Correct SANSadd2 script

Correct the way it gets the binning from the monitors.

re #8049
Re #6198. Just return the maps by value.

In C++11, map has a move constructor so returning by value isn't expensive.
Actually, even on the non-C++11 Mac tests show that the RVO saves us.

Now to fix all the client code...
Re #6198. Fix usages of mapping methods for value return.

Notes:
- 4 classes weren't deleting the returned pointer
- more were not exception safe
- some changes rely on the move assignment operator for efficiency.
Re #6198. Fix clients of mapping methods in Algorithms package.

(At least 5 more places that weren't deleting the returned map.)
Re #6198. Fix clients of mapping methods in Crystal package.
Re #6198. Fix last use of mapping method in Framework.

We're relying on the move assignment operator here for efficiency, so
things will be a little slower on the Mac until it gets C++11 libs.
But we're not leaking the thing like we were before.
Fix previously-leaking usage of getSpectrumToWorkspaceIndexMap.

This is the one that the Coverity scan highlighted. Re #6198.
Re #6198. Update InstrumentView usage of mapping method.

I restored the documented exception throwing behaviour after checking that
every place that calls the method catches the exception. Not sure when/why
that behaviour was changed, but this avoids the risk of accessing an
invalid iterator.
Re #6198. Fix usages of mapping methods in tests.
Re #6198. Fix for crashing unit test on Mac.
Merge remote-tracking branch 'origin/bugfix/8049_sans2d_add_many_files'
Merge remote-tracking branch 'origin/bugfix/8032_dock_loses_selection'
Merge remote-tracking branch 'origin/bugfix/7890_isis_event_monitors'