Skip to content

Commit 3806edd

Browse files
author
Andrea Barbasso
committed
[UXP-155] trim coordinates
1 parent 703920b commit 3806edd

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

src/app/core/services/location.service.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -123,13 +123,13 @@ export class LocationService {
123123
/**
124124
* Check if a string contains valid coordinateString in the format `latitude,longitude`
125125
* @param coordinateString the string to be checked
126-
* @returns {boolean} whether the string is valid and it contains valid coordinateString
126+
* @returns {boolean} whether the string is valid, and it contains valid coordinateString
127127
*/
128128
public isValidCoordinateString(coordinateString: string): boolean {
129129
if (this.isCoordinateString(coordinateString)) {
130130
const coordinateArray = coordinateString.split(',');
131-
const latitude = parseFloat(coordinateArray[0]);
132-
const longitude = parseFloat(coordinateArray[1]);
131+
const latitude = parseFloat(coordinateArray[0]?.trim());
132+
const longitude = parseFloat(coordinateArray[1]?.trim());
133133
return !isNaN(latitude) && !isNaN(longitude) && this.isValidCoordinatePair(latitude, longitude);
134134
} else {
135135
return false;
@@ -144,8 +144,8 @@ export class LocationService {
144144
public parseCoordinates(coordinates: string): LocationCoordinates {
145145
const coordinateArr = coordinates.split(',');
146146
if (this.isValidCoordinateString(coordinates)) {
147-
const latitude = parseFloat(coordinateArr[0]);
148-
const longitude = parseFloat(coordinateArr[1]);
147+
const latitude = parseFloat(coordinateArr[0]?.trim());
148+
const longitude = parseFloat(coordinateArr[1]?.trim());
149149
return { latitude, longitude };
150150
} else {
151151
console.warn('Location service', `Invalid coordinates "${coordinates}"`);

0 commit comments

Comments
 (0)