Refactor DataRegionTable to handle tricky characters better

[labkey-tchad]

Rationale

DataRegionTable doesn't actually handle field names with tricky characters. What it calls field names are actually fieldKeys. This update adopts a similar pattern to the recent app grid test update. I had to replicate some existing non-standard behavior to avoid breaking existing tests (removing spaces from fieldKeys and ignoring spaces when looking up field labels).

Related Pull Requests

• Update test helpers to understand field keys and names #2450
• https://github.com/LabKey/premiumModules/pull/288

Changes

• Update DataRegionTable to use FieldReferenceManager.java to organize and identify columns
• Clean up numerous IntelliJ warnings
• Remove usages of some deprecated DataRegionTable methods

[labkey-chrisj]

Thanks for doing this!