Skip to content

Commit 09cda31

Browse files
committed
Move scheme constants to own class
1 parent 6ba4e0b commit 09cda31

File tree

5 files changed

+22
-6
lines changed

5 files changed

+22
-6
lines changed
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
package de.blau.android.contract;
2+
3+
public final class Schemes {
4+
public static final String FILE = "file";
5+
public static final String CONTENT = "content";
6+
7+
/**
8+
* Private default constructor
9+
*/
10+
private Schemes() {
11+
// empty
12+
}
13+
}

src/main/java/de/blau/android/osm/Server.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@
4141
import de.blau.android.PostAsyncActionHandler;
4242
import de.blau.android.R;
4343
import de.blau.android.contract.MimeTypes;
44+
import de.blau.android.contract.Schemes;
4445
import de.blau.android.contract.Urls;
4546
import de.blau.android.dialogs.ErrorAlert;
4647
import de.blau.android.exception.OsmException;
@@ -225,7 +226,7 @@ public Server(@NonNull Context context, @NonNull final API api, @NonNull final S
225226

226227
// if we have a tiled OSM data source, open
227228
Uri readOnlyUri = Uri.parse(getReadOnlyUrl());
228-
if (FileUtil.FILE_SCHEME.equals(readOnlyUri.getScheme())) {
229+
if (Schemes.FILE.equals(readOnlyUri.getScheme())) {
229230
MBTileProviderDataBase tempDB = null;
230231
try {
231232
tempDB = new MBTileProviderDataBase(context, readOnlyUri, 1);

src/main/java/de/blau/android/prefs/PresetEditorActivity.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@
4444
import de.blau.android.App;
4545
import de.blau.android.HelpViewer;
4646
import de.blau.android.R;
47+
import de.blau.android.contract.Schemes;
4748
import de.blau.android.dialogs.Progress;
4849
import de.blau.android.exception.OperationFailedException;
4950
import de.blau.android.osm.Server;
@@ -292,7 +293,7 @@ protected void onPreExecute() {
292293
protected Integer doInBackground(Void... args) {
293294
int loadResult = RESULT_TOTAL_SUCCESS;
294295
Uri uri = Uri.parse(item.value);
295-
if (FileUtil.FILE_SCHEME.equals(uri.getScheme())) {
296+
if (Schemes.FILE.equals(uri.getScheme())) {
296297
loadResult = load(uri, Preset.PRESETXML);
297298
} else {
298299
loadResult = download(item.value, Preset.PRESETXML);

src/main/java/de/blau/android/util/FileUtil.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,11 @@
1515
import androidx.annotation.NonNull;
1616
import androidx.annotation.Nullable;
1717
import de.blau.android.contract.Paths;
18+
import de.blau.android.contract.Schemes;
1819

1920
public abstract class FileUtil {
2021
private static final String DEBUG_TAG = FileUtil.class.getSimpleName();
21-
public static final String FILE_SCHEME = "file";
22-
public static final String FILE_SCHEME_PREFIX = "file:";
22+
public static final String FILE_SCHEME_PREFIX = Schemes.FILE + ":";
2323

2424
/**
2525
* Private constructor to stop instantiation

src/main/java/de/blau/android/util/SelectFile.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
import androidx.annotation.Nullable;
2121
import androidx.fragment.app.FragmentActivity;
2222
import de.blau.android.R;
23+
import de.blau.android.contract.Schemes;
2324
import de.blau.android.prefs.Preferences;
2425

2526
/**
@@ -201,7 +202,7 @@ public static void handleResult(int code, @NonNull Intent data) {
201202
* @param fileUri the file uri
202203
*/
203204
public static void savePref(Preferences prefs, int directoryPrefKey, Uri fileUri) {
204-
if (Build.VERSION.SDK_INT < Build.VERSION_CODES.KITKAT && FileUtil.FILE_SCHEME.equals(fileUri.getScheme())) {
205+
if (Build.VERSION.SDK_INT < Build.VERSION_CODES.KITKAT && Schemes.FILE.equals(fileUri.getScheme())) {
205206
int slash = fileUri.getPath().lastIndexOf('/');
206207
if (slash >= 0) {
207208
String path = fileUri.getPath().substring(0, slash + 1);
@@ -261,7 +262,7 @@ public static String getPath(@NonNull Context context, @NonNull Uri uri) {
261262
Log.i(DEBUG_TAG, "content scheme");
262263
return getDataColumn(context, uri, null, null);
263264
}
264-
} else if (FileUtil.FILE_SCHEME.equalsIgnoreCase(uri.getScheme())) {
265+
} else if (Schemes.FILE.equalsIgnoreCase(uri.getScheme())) {
265266
return uri.getPath();
266267
}
267268
Log.e(DEBUG_TAG, "Unable to determine how to handle Uri " + uri);

0 commit comments

Comments
 (0)