From b1677f4781811f67e2d7cc9539538c75710940b2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jaime=20Guti=C3=A9rrez=20Alfaro?= Date: Wed, 11 Feb 2026 10:06:02 -0600 Subject: [PATCH] Fix: Add fallback when pref value is not set --- .../java/net/osmtracker/activity/Preferences.java | 11 +++++++++-- app/src/main/res/values/strings-preferences.xml | 2 ++ 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/app/src/main/java/net/osmtracker/activity/Preferences.java b/app/src/main/java/net/osmtracker/activity/Preferences.java index 9e637d2a..d1b2cd54 100644 --- a/app/src/main/java/net/osmtracker/activity/Preferences.java +++ b/app/src/main/java/net/osmtracker/activity/Preferences.java @@ -349,8 +349,15 @@ private void setupListPreference(String preferenceKey, String staticSummary) { listPref.setSummaryProvider(preference -> { ListPreference lp = (ListPreference) preference; CharSequence entry = lp.getEntry(); - // Null check: entry might be null if no value is selected - String displayValue = Objects.requireNonNull(entry).toString(); + + // Handle cases where no value has been selected yet. (backwards compatibility) + String displayValue; + if (entry == null || TextUtils.isEmpty(entry)) { + // Fallback text if no value is set. + displayValue = getString(R.string.prefs_not_set); + } else { + displayValue = entry.toString(); + } return displayValue + ".\n" + staticSummary; }); } diff --git a/app/src/main/res/values/strings-preferences.xml b/app/src/main/res/values/strings-preferences.xml index 5b11dda5..88861e3c 100644 --- a/app/src/main/res/values/strings-preferences.xml +++ b/app/src/main/res/values/strings-preferences.xml @@ -117,4 +117,6 @@ Export compass heading Defines if and how the compass data should be exported to the GPX file Reset default value + Not set +