Location Hierarchy endpoint lineage ids enhancement and refactor

[ndegwamartin]

IMPORTANT: Where possible all PRs must be linked to a Github issue

Resolves #112
Resolves #113

Engineer Checklist

• I have written Unit tests for any new feature(s) and edge cases for
  bug fixes
• I have added documentation for any new feature(s) and configuration
  option(s) on the README.md
• I have run mvn spotless:check to check my code follows the project's
  style guide
• I have run mvn clean test jacoco:report to confirm the coverage report
  was generated at plugins/target/site/jacoco/index.html
• I ran mvn clean package right before creating this pull request.

[codecov]

Codecov Report

Attention: Patch coverage is 49.61240% with 65 lines in your changes missing coverage. Please review.

Project coverage is 62.21%. Comparing base (78290cf) to head (451d081).

Files with missing lines	Patch %	Lines
...teway/plugins/LocationHierarchyEndpointHelper.java	65.26%	26 Missing and 7 partials ⚠️
.../fhir/gateway/plugins/PermissionAccessChecker.java	0.00%	32 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #114      +/-   ##
============================================
- Coverage     62.96%   62.21%   -0.75%     
- Complexity      221      229       +8     
============================================
  Files            16       16              
  Lines          1477     1588     +111     
  Branches        168      194      +26     
============================================
+ Hits            930      988      +58     
- Misses          460      504      +44     
- Partials         87       96       +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ndegwamartin]

@pld @dubdabasoduba for the backward compatibility flag, our default check is for true if the query parameter exists and its value to true. However cant help wondering if it should be an OR there, i.e. filter_mode_lineage Defined or true, since the addition of the query parameter indicates intention to enable this mode.

[pld]

Yea I agree that makes more sense, if it's not passed it is false, if it passed it is true, regardless of what its value