Fix various memory warnings and clang format

Fix datablock test

Fix LoadMuonNexusV2 wrongly being called to load PSI Muon files

The LoadMuonNexusV2 algorithm was wrongly being chosen by the Load algorithm to load PSI Muon files. This occurs as the files are very similar, with both having a /raw_data_entry and a raw_data_1/definition Muon entry.

In order to differentiate these two files, the MuonNexusV2 loader now has an additional confidence metric, which returns 0 if a top level beamline entry is not found, i.e raw_data_1/beamline. This works as the beamline entry in the PSI files is located within raw_data_1/instrument/beamline entry.

Create Muon Nexus V2 loading algorithm

This commit makes TableWorkspace.h and Workspace2D.h use the forward declarations of the shared and unique pointer types defined in the appropriate _fwd.h file.

Was getting too complex and bloated. Refactored out the actual data and
plot changes into another method. toggle_normalisation mostly handles
the extra crap you have to do to stop crashes and weird behaviour.

RE #28381

RE #283811

RE #28381

A ragged workspace with a log colorbar could end up with vmin and vmax
values being negative, which is incompatible with a log scale. The
change_colorbar_axes method already has handling for this, so the
normalisation toggle sets the colorbar to normalised before adjusting
the data and then puts it back afterwards using the handling from the
existing method.

RE #28391

Fixes failure of performance test build that only builds MantidPlot.
f1376efd fixed a bug in an if statement
within `qt/python/CMakeLists.txt` that caused this file to start
being processed when only MantidPlot was enabled.

This commit responds to the code review comments:

- Extract constant strings which are used multiple times into const string variables
- Throw immediately if multi period.
- Remove unclear call to getProperty("OutputWorkspace) in LoadMuonNexusV2
- Remove unnecessary includes and in doing so create forward declarations for TableWorkspace and Workspace2D
- Use std::move for string and workspace copying.
- Remove unnecessary pointer casting, which will be dealt with by the compiler.

Co-authored-by: Conor Finn <47181718+ConorMFinn@users.noreply.github.com>