Skip to content

Commit 442a27c

Browse files
authored
Reip out vector mapfile setting (#437)
It's causing crashes on some phones, and will be replaced with a new mechanism soon, see PR #417.
1 parent 4266e84 commit 442a27c

File tree

3 files changed

+3
-57
lines changed

3 files changed

+3
-57
lines changed

libraries/cyclestreets-fragments/src/main/java/net/cyclestreets/SettingsFragment.kt

Lines changed: 3 additions & 53 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ private const val PREFERENCE_SCREEN_ARG: String = "preferenceScreenArg"
2222
private val SETTINGS_ICONS = mapOf(
2323
"screen-maps-display" to GoogleMaterial.Icon.gmd_map,
2424
"mapstyle" to null,
25-
"mapfile" to null,
2625
"confirm-new-route" to null,
2726
"screen-routing-preferences" to GoogleMaterial.Icon.gmd_directions,
2827
"routetype" to null,
@@ -50,13 +49,11 @@ class SettingsFragment : PreferenceFragmentCompat(), SharedPreferences.OnSharedP
5049
super.onCreate(savedInstance)
5150

5251
setupMapStyles()
53-
setupMapFileList()
5452

5553
setSummary(CycleStreetsPreferences.PREF_ROUTE_TYPE_KEY)
5654
setSummary(CycleStreetsPreferences.PREF_UNITS_KEY)
5755
setSummary(CycleStreetsPreferences.PREF_SPEED_KEY)
5856
setSummary(CycleStreetsPreferences.PREF_MAPSTYLE_KEY)
59-
setSummary(CycleStreetsPreferences.PREF_MAPFILE_KEY)
6057
setSummary(CycleStreetsPreferences.PREF_UPLOAD_SIZE)
6158
setSummary(CycleStreetsPreferences.PREF_NEARING_TURN)
6259
setSummary(CycleStreetsPreferences.PREF_OFFTRACK_DISTANCE)
@@ -117,29 +114,10 @@ class SettingsFragment : PreferenceFragmentCompat(), SharedPreferences.OnSharedP
117114

118115
private fun setupMapStyles() {
119116
findPreference<ListPreference>(CycleStreetsPreferences.PREF_MAPSTYLE_KEY)?.apply {
120-
121-
if (this.value == CycleStreetsPreferences.MAPSTYLE_MAPSFORGE && MapPack.availableMapPacks(context).isEmpty()) {
122-
Log.i(TAG, "Offline Vector Maps were selected, but there are no available map packs; default to OSM")
123-
this.value = CycleStreetsPreferences.MAPSTYLE_OSM
124-
}
125-
126-
TileSource.configurePreference(this)
117+
TileSource.configurePreference(this)
127118
}
128119
}
129120

130-
private fun setupMapFileList() {
131-
findPreference<ListPreference>(CycleStreetsPreferences.PREF_MAPFILE_KEY)?.apply {
132-
populateMapFileList(this)
133-
}
134-
}
135-
136-
private fun populateMapFileList(mapfilePref: ListPreference) {
137-
val names = MapPack.availableMapPacks(context).map { pack: MapPack -> pack.name() }
138-
val files = MapPack.availableMapPacks(context).map { pack: MapPack -> pack.path() }
139-
mapfilePref.entries = names.toTypedArray()
140-
mapfilePref.entryValues = files.toTypedArray()
141-
}
142-
143121
override fun onResume() {
144122
super.onResume()
145123

@@ -165,37 +143,9 @@ class SettingsFragment : PreferenceFragmentCompat(), SharedPreferences.OnSharedP
165143
prefUI.summary = prefUI.entry
166144
if (prefUI is EditTextPreference)
167145
prefUI.summary = prefUI.text
146+
}
168147

169-
if (CycleStreetsPreferences.PREF_MAPSTYLE_KEY == key)
170-
setMapFileSummary((prefUI as ListPreference).value)
171-
}
172-
173-
private fun setMapFileSummary(style: String) {
174-
val pref = findPreference<Preference>(CycleStreetsPreferences.PREF_MAPFILE_KEY) ?: return
175-
val mapfilePref = pref as ListPreference
176-
177-
val enabled = style == CycleStreetsPreferences.MAPSTYLE_MAPSFORGE
178-
mapfilePref.isEnabled = enabled
179-
180-
if (!enabled)
181-
return
182-
183-
if (mapfilePref.entryValues.isEmpty()) {
184-
mapfilePref.isEnabled = false
185-
MessageBox.YesNo(requireView(), R.string.settings_no_map_packs) { _, _ -> MapPack.searchGooglePlay(requireContext()) }
186-
return
187-
}
188-
189-
val mapfile = CycleStreetsPreferences.mapfile()
190-
var index = mapfilePref.findIndexOfValue(mapfile)
191-
if (index == -1)
192-
index = 0 // default to something
193-
194-
mapfilePref.setValueIndex(index)
195-
mapfilePref.summary = mapfilePref.entries[index]
196-
}
197-
198-
private fun setAccountSummary() {
148+
private fun setAccountSummary() {
199149
val pref = findPreference<Preference>(CycleStreetsPreferences.PREF_ACCOUNT_KEY) ?: return
200150
val account = pref as PreferenceScreen
201151

libraries/cyclestreets-view/src/main/java/net/cyclestreets/tiles/TileSource.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,6 @@ private static void addBuiltInSources(final Context context) {
197197
addTileSource("OpenCycleMap (shows hills)", OPENCYCLEMAP);
198198
addTileSource("OpenStreetMap default style", OPENSTREETMAP);
199199
addTileSource("Ordnance Survey OpenData", OSMAP);
200-
addTileSource("Offline Vector Maps", MAPSFORGE);
201200

202201
builtInsAdded_ = true;
203202
}

libraries/cyclestreets-view/src/main/res/xml/prefs.xml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,6 @@
99
android:title="Map style"
1010
android:summary="OpenStreetMap"
1111
android:defaultValue="CycleStreets-OSM" />
12-
<ListPreference
13-
android:key="mapfile"
14-
android:title="Vector mapfile" />
1512
<CheckBoxPreference
1613
android:key="confirm-new-route"
1714
android:title="Confirm start new route"

0 commit comments

Comments
 (0)