diff options
| author | Samuel Tardieu <sam@rfc1149.net> | 2013-01-27 00:04:24 +0100 |
|---|---|---|
| committer | Samuel Tardieu <sam@rfc1149.net> | 2013-01-27 01:02:15 +0100 |
| commit | 2d802377e0da124e2dcb32758180ec313a8d9388 (patch) | |
| tree | ca7a420ed768c6f6fb58fc393aab4a77d906e323 /main/src/cgeo/geocaching/SettingsActivity.java | |
| parent | 1139fc83197316f0ababb5cd365cbb440b632e3b (diff) | |
| download | cgeo-2d802377e0da124e2dcb32758180ec313a8d9388.zip cgeo-2d802377e0da124e2dcb32758180ec313a8d9388.tar.gz cgeo-2d802377e0da124e2dcb32758180ec313a8d9388.tar.bz2 | |
Isolate intent extras into a new class
Diffstat (limited to 'main/src/cgeo/geocaching/SettingsActivity.java')
| -rw-r--r-- | main/src/cgeo/geocaching/SettingsActivity.java | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/main/src/cgeo/geocaching/SettingsActivity.java b/main/src/cgeo/geocaching/SettingsActivity.java index 64a086d..6669fdd 100644 --- a/main/src/cgeo/geocaching/SettingsActivity.java +++ b/main/src/cgeo/geocaching/SettingsActivity.java @@ -973,8 +973,8 @@ public class SettingsActivity extends AbstractActivity { switch (requestCode) { case SELECT_MAPFILE_REQUEST: - if (data.hasExtra("mapfile")) { - final String mapFile = data.getStringExtra("mapfile"); + if (data.hasExtra(Intents.EXTRA_MAP_FILE)) { + final String mapFile = data.getStringExtra(Intents.EXTRA_MAP_FILE); Settings.setMapFile(mapFile); if (!Settings.isValidMapFile(Settings.getMapFile())) { showToast(res.getString(R.string.warn_invalid_mapfile)); @@ -1032,9 +1032,8 @@ public class SettingsActivity extends AbstractActivity { } private void selectDirectory(String startDirectory, int directoryKind) { - Intent dirChooser; try { - dirChooser = new Intent(FileManagerIntents.ACTION_PICK_DIRECTORY); + final Intent dirChooser = new Intent(FileManagerIntents.ACTION_PICK_DIRECTORY); if (StringUtils.isNotBlank(startDirectory)) { dirChooser.setData(Uri.parse("file://" + new File(startDirectory).getAbsolutePath())); } @@ -1043,8 +1042,8 @@ public class SettingsActivity extends AbstractActivity { startActivityForResult(dirChooser, directoryKind); } catch (android.content.ActivityNotFoundException ex) { // OI file manager not available - dirChooser = new Intent(this, SimpleDirChooser.class); - dirChooser.putExtra(SimpleDirChooser.START_DIR, startDirectory); + final Intent dirChooser = new Intent(this, SimpleDirChooser.class); + dirChooser.putExtra(Intents.EXTRA_START_DIR, startDirectory); startActivityForResult(dirChooser, directoryKind); } } |
