Fix nulls leaking into geo urls

src/components/views/location/LocationPicker.tsx

@@ -289,16 +289,25 @@ export function getGeoUri(position: GeolocationPosition): string {
     const lat = position.coords.latitude;
     const lon = position.coords.longitude;
     const alt = (
-        position.coords.altitude !== undefined
+        isNumeric(position.coords.altitude)
             ? `,${position.coords.altitude}`
             : ""
     );
     const acc = (
-        position.coords.accuracy !== undefined
+        isNumeric(position.coords.accuracy)
             ? `;u=${ position.coords.accuracy }`
             : ""
     );
     return `geo:${lat},${lon}${alt}${acc}`;
 }
 
+function isNumeric(n: number | null | undefined) {
+    return (
+        n !== null &&
+        n !== undefined &&
+        !isNaN(n) &&
+        isFinite(n)
+    );
+}
+
 export default LocationPicker;

test/components/views/location/LocationPicker-test.tsx

@@ -35,6 +35,22 @@ describe("LocationPicker", () => {
             expect(getGeoUri(pos)).toEqual("geo:43.2,12.4");
         });
 
+        it("Nulls in location are not shown in URI", () => {
+            const pos: GeolocationPosition = {
+                coords: {
+                    latitude: 43.2,
+                    longitude: 12.4,
+                    altitude: null,
+                    accuracy: null,
+                    altitudeAccuracy: null,
+                    heading: null,
+                    speed: null,
+                },
+                timestamp: 12334,
+            };
+            expect(getGeoUri(pos)).toEqual("geo:43.2,12.4");
+        });
+
         it("Renders a URI with 3 coords", () => {
             const pos: GeolocationPosition = {
                 coords: {