Do not display map marker if latitude and longitude are not present on an Encounter

[holmbergius]

This PR fixes a small logical flaw in the display logic of map markers, preventing a marker from being displayed at 0,0 on the map if the Encounter object does not have both a latitude and a longitude.

PR fixes #995

Before you Submit!

• Is all the text internationalized?
• If you made a change to the header, did you update the react, jsp, and html?
• Are all depedencies at a locked version?
• Did you adhere to best practices?
• Is there a quick unit test you can add?

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.14%. Comparing base (b798976) to head (6b90c2b).
Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##              main    #996   +/-   ##
=======================================
  Coverage     0.14%   0.14%           
  Complexity      29      29           
=======================================
  Files          565     565           
  Lines        63363   63363           
  Branches     10982   10982           
=======================================
  Hits            89      89           
  Misses       63255   63255           
  Partials        19      19           

Flag	Coverage Δ
backend	0.14% <ø> (ø)
frontend	0.14% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[TanyaStere42]

Just making sure: there's a use case where the location ID has gps rather than the encounter directly, and that still displays, right?

[holmbergius]

@TanyaStere42 Excellent question. I dug through encounter.jsp...and I don't see where default locationID coordinates were ever displayed if the Encounter has no GPS but the location ID does in the JSON. I think this was only ever an option in Encounter Search mapped results.

[TanyaStere42]

That could be a regression? Or it may just be something that has been discussed with a couple of the live platforms for future dev. Not sure which

[naknomum]

the conspiracy to hide Null Island continues!
lgtm