mnalis/apps-android-commons
1
0
Fork 0
mirror of https://github.com/commons-app/apps-android-commons.git synced 2025-10-26 20:33:53 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

fix: remove unnecessary latLng variable and return null directly

Browse source
This commit is contained in:
Rsedp8 2025-05-24 17:04:54 +02:00
parent e81abaa1e4
commit 10479a01b7
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
app/src/main/java/fr/free/nrw/commons/utils/LocationUtils.kt
View file

@ -16,12 +16,12 @@ object LocationUtils {
val indexOfPrefix = customQuery.indexOf("Point(") val indexOfPrefix = customQuery.indexOf("Point(")
if (indexOfPrefix == -1) { if (indexOfPrefix == -1) {
Timber.e("Invalid prefix index - Seems like user has entered an invalid query") Timber.e("Invalid prefix index - Seems like user has entered an invalid query")
return latLng return null
} }
val indexOfSuffix = customQuery.indexOf(")\"", indexOfPrefix) val indexOfSuffix = customQuery.indexOf(")\"", indexOfPrefix)
if (indexOfSuffix == -1) { if (indexOfSuffix == -1) {
Timber.e("Invalid suffix index - Seems like user has entered an invalid query") Timber.e("Invalid suffix index - Seems like user has entered an invalid query")
return latLng return null
} }
val latLngString = customQuery.substring(indexOfPrefix + "Point(".length, indexOfSuffix) val latLngString = customQuery.substring(indexOfPrefix + "Point(".length, indexOfSuffix)
if (latLngString.isEmpty()) { if (latLngString.isEmpty()) {