Add property UseBankIDsInFile default True re #8974

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>
Test UseBankIDsInFile in unit test re #8974

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>
Re #6564. Remove items applicable to older OS X.

i.e. versions that we no longer support.
Re #6563. Remove installer preflight script.

It was just cleaning up long-gone items so I don't think we
need it anymore. It can't be used with a DragNDrop installer anyway.
Re #6563. Switch to drag and drop install kit.

Replacing PackageMaker.
Re #6563. DragNDrop generator need plain text license only

It can't do anything with the rich text files.
Re #8357. Eliminate valgrind invalid write warning.

By making sure to unregister the observer at the end of the test.
Re #8357. Fix leak in test by breaking cyclic reference.

The BoxController & BoxControllerDummyIO objects were each holding a
shared_ptr back to the other.
Re #8357. Fix memory leaks in tests.

By creating object on the stack instead of unnecessarily on the heap.
Re #8357. Remove leak in test by adding components to instrument.

The fact that you have to call two methods in succession to get this
right is probably not great design. Nor is the fact that you can
'mark' something that isn't in the instrument without any error or warning.
Re #8357. Eliminate memory leaks in tests.

By calling delete where appropriate.
Re #8357. Eliminate memory leak in test.

By removing group from within group to eliminate cyclic shared_ptr
reference. Possibly there's a real potential issue here, but would
anyone really ever put a group within a group?
Re #8357. Update to account for change in TestHelpers class.

The BoxControllerDummyIO constructor was changed to accept (and hold
internally) a bare pointer to avoid a cyclic reference.
Refs #9235 Inserted index sorting
Refs #9325 Added comments
Merge remote branch 'origin/feature/9235_LoadSassena_SortByQVectors'
Refs #5121 Remove debugging statement.
Merge remote-tracking branch 'origin/feature/8974_choose_bank_numbering'
Added description to both algorithms and summary to ViewBOA re #9191

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>
Merge branch 'feature/5121_furyfit_fix_params' of github.com:mantidproject/mantid into feature/5121_furyfit_fix_params