Re #6191. Pass by reference, not value.
Re #6191. Add a method to set all the detector IDs at once.

Without having to call clear followed by add to make sure there was not
anything already there.
Re #6191. Don't use the SpectraDetectorMap.
Merge branch 'feature/6191_remove_spectradetectormap' into develop