forked from upstream-mirrors/OpenTracks
Codename: rename methods to FileUtils.getPath and FileUtils.getPathDisplayName
This commit is contained in:
@@ -395,7 +395,7 @@ public class TrackDetailActivity extends AbstractSendToGoogleActivity implements
|
||||
}
|
||||
|
||||
File dir = new File(
|
||||
FileUtils.getDirectoryPath(FileUtils.PICTURES_DIR, Long.toString(trackId)));
|
||||
FileUtils.getPath(FileUtils.PICTURES_DIR, Long.toString(trackId)));
|
||||
FileUtils.ensureDirectoryExists(dir);
|
||||
|
||||
String fileName = SimpleDateFormat.getDateTimeInstance().format(new Date());
|
||||
|
||||
@@ -171,7 +171,7 @@ public class ExportDialogFragment extends DialogFragment {
|
||||
*/
|
||||
private void setExternalStorageOption(RadioButton radioButton, TrackFileFormat trackFileFormat) {
|
||||
radioButton.setText(getString(R.string.export_external_storage_option, trackFileFormat.name(),
|
||||
FileUtils.getDirectoryDisplayName(trackFileFormat.getExtension())));
|
||||
FileUtils.getPathDisplayName(trackFileFormat.getExtension())));
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
@@ -92,7 +92,7 @@ public class FileTypeDialogFragment extends DialogFragment {
|
||||
for (int i = 0; i < choices.length; i++) {
|
||||
TrackFileFormat trackFileFormat = trackFileFormats[i];
|
||||
choices[i] = getString(optionId, trackFileFormat.name(),
|
||||
FileUtils.getDirectoryDisplayName(trackFileFormat.getExtension()));
|
||||
FileUtils.getPathDisplayName(trackFileFormat.getExtension()));
|
||||
}
|
||||
return new AlertDialog.Builder(getActivity()).setNegativeButton(R.string.generic_cancel, null)
|
||||
.setPositiveButton(R.string.generic_ok, new OnClickListener() {
|
||||
|
||||
@@ -86,7 +86,7 @@ class ExternalFileBackup {
|
||||
* @param create whether to try creating the directory if it doesn't exist
|
||||
*/
|
||||
private File getBackupsDirectory(boolean create) {
|
||||
String directoryPath = FileUtils.getDirectoryPath(FileUtils.BACKUPS_DIR);
|
||||
String directoryPath = FileUtils.getPath(FileUtils.BACKUPS_DIR);
|
||||
final File dir = new File(directoryPath);
|
||||
Log.d(TAG, "Dir: " + dir.getAbsolutePath());
|
||||
if (create) {
|
||||
|
||||
@@ -124,7 +124,7 @@ public class RestoreChooserActivity extends Activity {
|
||||
*/
|
||||
private void showNoBackupMessage() {
|
||||
String message = getString(R.string.settings_backup_restore_no_backups,
|
||||
FileUtils.getDirectoryDisplayName(FileUtils.BACKUPS_DIR));
|
||||
FileUtils.getPathDisplayName(FileUtils.BACKUPS_DIR));
|
||||
Toast.makeText(this, message, Toast.LENGTH_LONG).show();
|
||||
}
|
||||
}
|
||||
|
||||
@@ -82,7 +82,7 @@ public class SaveActivity extends Activity {
|
||||
}
|
||||
|
||||
File directory = playTrack ? getCacheDir()
|
||||
: new File(FileUtils.getDirectoryPath(trackFileFormat.getExtension()));
|
||||
: new File(FileUtils.getPath(trackFileFormat.getExtension()));
|
||||
if (!FileUtils.ensureDirectoryExists(directory)) {
|
||||
Toast.makeText(this, R.string.external_storage_not_writable, Toast.LENGTH_LONG).show();
|
||||
finish();
|
||||
@@ -96,7 +96,7 @@ public class SaveActivity extends Activity {
|
||||
}
|
||||
|
||||
directoryDisplayName = playTrack ? getCacheDir().getName()
|
||||
: FileUtils.getDirectoryDisplayName(trackFileFormat.getExtension());
|
||||
: FileUtils.getPathDisplayName(trackFileFormat.getExtension());
|
||||
|
||||
Object retained = getLastNonConfigurationInstance();
|
||||
if (retained instanceof SaveAsyncTask) {
|
||||
|
||||
+1
-1
@@ -470,7 +470,7 @@ abstract class AbstractFileTrackImporter extends DefaultHandler implements Track
|
||||
if (importTrackId == -1L) {
|
||||
return null;
|
||||
}
|
||||
return FileUtils.getDirectoryPath(
|
||||
return FileUtils.getPath(
|
||||
FileUtils.PICTURES_DIR, Long.toString(importTrackId), fileName);
|
||||
}
|
||||
|
||||
|
||||
@@ -89,9 +89,9 @@ public class ImportActivity extends Activity {
|
||||
}
|
||||
String directoryPath;
|
||||
if (importAll) {
|
||||
directoryDisplayName = FileUtils.getDirectoryDisplayName(
|
||||
directoryDisplayName = FileUtils.getPathDisplayName(
|
||||
trackFileFormat.getExtension());
|
||||
directoryPath = FileUtils.getDirectoryPath(trackFileFormat.getExtension());
|
||||
directoryPath = FileUtils.getPath(trackFileFormat.getExtension());
|
||||
if (!FileUtils.isDirectory(new File(directoryPath))) {
|
||||
Toast.makeText(
|
||||
this, getString(R.string.import_no_directory, directoryDisplayName), Toast.LENGTH_LONG)
|
||||
|
||||
+2
-2
@@ -116,7 +116,7 @@ public class KmzTrackImporter implements TrackImporter {
|
||||
|
||||
if (importTrackId != -1L) {
|
||||
File dir = new File(
|
||||
FileUtils.getDirectoryPath(FileUtils.PICTURES_DIR, Long.toString(importTrackId)));
|
||||
FileUtils.getPath(FileUtils.PICTURES_DIR, Long.toString(importTrackId)));
|
||||
if (dir.exists() && dir.isDirectory()) {
|
||||
for (File file : dir.listFiles()) {
|
||||
file.delete();
|
||||
@@ -180,7 +180,7 @@ public class KmzTrackImporter implements TrackImporter {
|
||||
return;
|
||||
}
|
||||
File dir = new File(
|
||||
FileUtils.getDirectoryPath(FileUtils.PICTURES_DIR, Long.toString(importTrackId)));
|
||||
FileUtils.getPath(FileUtils.PICTURES_DIR, Long.toString(importTrackId)));
|
||||
FileUtils.ensureDirectoryExists(dir);
|
||||
|
||||
fileOutputStream = new FileOutputStream(new File(dir, fileName));
|
||||
|
||||
+1
-1
@@ -121,7 +121,7 @@ public class BackupResetSettingsActivity extends AbstractSettingsActivity {
|
||||
switch (id) {
|
||||
case DIALOG_CONFIRM_BACKUP_ID:
|
||||
message = getString(R.string.settings_backup_confirm_message,
|
||||
FileUtils.getDirectoryDisplayName(FileUtils.BACKUPS_DIR));
|
||||
FileUtils.getPathDisplayName(FileUtils.BACKUPS_DIR));
|
||||
dialog = DialogUtils.createConfirmationDialog(this, R.string.settings_backup_confirm_title,
|
||||
message, new DialogInterface.OnClickListener() {
|
||||
@Override
|
||||
|
||||
@@ -88,11 +88,11 @@ public class FileUtils {
|
||||
}
|
||||
|
||||
/**
|
||||
* Gets the directory display name.
|
||||
* Gets the display name for a path on the external storage.
|
||||
*
|
||||
* @param components the components
|
||||
*/
|
||||
public static String getDirectoryDisplayName(String... components) {
|
||||
public static String getPathDisplayName(String... components) {
|
||||
StringBuilder dirNameBuilder = new StringBuilder();
|
||||
dirNameBuilder.append(File.separatorChar);
|
||||
dirNameBuilder.append(SDCARD_TOP_DIR);
|
||||
@@ -103,14 +103,14 @@ public class FileUtils {
|
||||
return dirNameBuilder.toString();
|
||||
}
|
||||
/**
|
||||
* Gets the directory path.
|
||||
* Gets a path on the external storage.
|
||||
*
|
||||
* @param components the components
|
||||
*/
|
||||
public static String getDirectoryPath(String... components) {
|
||||
public static String getPath(String... components) {
|
||||
StringBuilder dirNameBuilder = new StringBuilder();
|
||||
dirNameBuilder.append(Environment.getExternalStorageDirectory());
|
||||
dirNameBuilder.append(getDirectoryDisplayName(components));
|
||||
dirNameBuilder.append(getPathDisplayName(components));
|
||||
return dirNameBuilder.toString();
|
||||
}
|
||||
|
||||
|
||||
@@ -29,13 +29,13 @@ import junit.framework.TestCase;
|
||||
public class FileUtilsTest extends TestCase {
|
||||
|
||||
/**
|
||||
* Tests {@link FileUtils#getDirectoryPath(String...)}.
|
||||
* Tests {@link FileUtils#getPath(String...)}.
|
||||
*/
|
||||
public void testBuildExternalDirectoryPath() {
|
||||
String expectedName = Environment.getExternalStorageDirectory() + File.separator
|
||||
+ FileUtils.SDCARD_TOP_DIR + File.separator + "a" + File.separator + "b" + File.separator
|
||||
+ "c";
|
||||
String dirName = FileUtils.getDirectoryPath("a", "b", "c");
|
||||
String dirName = FileUtils.getPath("a", "b", "c");
|
||||
assertEquals(expectedName, dirName);
|
||||
}
|
||||
|
||||
|
||||
+2
-2
@@ -501,7 +501,7 @@ public class EndToEndTestUtils {
|
||||
* @param trackKind the kind of track
|
||||
*/
|
||||
public static void deleteExportedFiles(String trackKind) {
|
||||
File[] allFiles = (new File(FileUtils.getDirectoryPath(trackKind.toLowerCase()))).listFiles();
|
||||
File[] allFiles = (new File(FileUtils.getPath(trackKind.toLowerCase()))).listFiles();
|
||||
if (allFiles != null) {
|
||||
for (File oneFile : allFiles) {
|
||||
oneFile.delete();
|
||||
@@ -528,7 +528,7 @@ public class EndToEndTestUtils {
|
||||
* @return files array of such kind of exported tracks
|
||||
*/
|
||||
public static File[] getExportedFiles(final String trackKind) {
|
||||
String directoryPath = FileUtils.getDirectoryPath(trackKind);
|
||||
String directoryPath = FileUtils.getPath(trackKind);
|
||||
FileFilter filter = new FileFilter() {
|
||||
@Override
|
||||
public boolean accept(File pathname) {
|
||||
|
||||
+4
-4
@@ -264,20 +264,20 @@ public class ExportAllAndImportAllTest extends ActivityInstrumentationTestCase2<
|
||||
private String getSaveSuccessMessage(int count, String type) {
|
||||
String tracks = activityMyTracks.getResources().getQuantityString(R.plurals.tracks, count,
|
||||
count);
|
||||
String directoryDisplayName = FileUtils.getDirectoryDisplayName(type);
|
||||
String directoryDisplayName = FileUtils.getPathDisplayName(type);
|
||||
return activityMyTracks.getString(R.string.export_external_storage_success, tracks,
|
||||
directoryDisplayName);
|
||||
}
|
||||
|
||||
private String getImportSuccessMessage(int count, String type) {
|
||||
String files = activityMyTracks.getResources().getQuantityString(R.plurals.files, count, count);
|
||||
String directoryDisplayName = FileUtils.getDirectoryDisplayName(type);
|
||||
String directoryDisplayName = FileUtils.getPathDisplayName(type);
|
||||
return activityMyTracks.getString(R.string.import_success, files, directoryDisplayName);
|
||||
}
|
||||
|
||||
private String getImportErrorMessage(int count, int total, String type) {
|
||||
String files = activityMyTracks.getResources().getQuantityString(R.plurals.files, total, total);
|
||||
String directoryDisplayName = FileUtils.getDirectoryDisplayName(type);
|
||||
String directoryDisplayName = FileUtils.getPathDisplayName(type);
|
||||
return activityMyTracks.getString(R.string.import_error, count, files, directoryDisplayName);
|
||||
}
|
||||
|
||||
@@ -481,7 +481,7 @@ public class ExportAllAndImportAllTest extends ActivityInstrumentationTestCase2<
|
||||
+ "</trkseg> " + "</trk> " + "</gpx>";
|
||||
|
||||
try {
|
||||
File file = new File(FileUtils.getDirectoryPath(EndToEndTestUtils.GPX.toLowerCase())
|
||||
File file = new File(FileUtils.getPath(EndToEndTestUtils.GPX.toLowerCase())
|
||||
+ File.separator + fileName);
|
||||
FileOutputStream fop = new FileOutputStream(file);
|
||||
// if file doesnt exists, then create it
|
||||
|
||||
Reference in New Issue
Block a user