Get the folder just once instead of for each exported file

ycs0405 · Apr 15, 2017 · 2a6c4c8 · 2a6c4c8
1 parent dc811ad
commit 2a6c4c8
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/app/src/main/java/org/gnucash/android/export/ExportAsyncTask.java b/app/src/main/java/org/gnucash/android/export/ExportAsyncTask.java
@@ -271,6 +271,9 @@ private void moveExportToGoogleDrive() throws Exporter.ExporterException {
         final GoogleApiClient googleApiClient = BackupPreferenceFragment.getGoogleApiClient(GnuCashApplication.getAppContext());
         googleApiClient.blockingConnect();
 
+        SharedPreferences sharedPreferences = PreferenceManager.getDefaultSharedPreferences(mContext);
+        String folderId = sharedPreferences.getString(mContext.getString(R.string.key_google_drive_app_folder_id), "");
+        DriveFolder folder = DriveId.decodeFromString(folderId).asDriveFolder();
         try {
             for (String exportedFilePath : mExportedFiles) {
                 DriveApi.DriveContentsResult driveContentsResult =
@@ -297,10 +300,6 @@ private void moveExportToGoogleDrive() throws Exporter.ExporterException {
                         .setTitle(exportedFile.getName())
                         .setMimeType(mExporter.getExportMimeType())
                         .build();
-
-                SharedPreferences sharedPreferences = PreferenceManager.getDefaultSharedPreferences(mContext);
-                String folderId = sharedPreferences.getString(mContext.getString(R.string.key_google_drive_app_folder_id), "");
-                DriveFolder folder = DriveId.decodeFromString(folderId).asDriveFolder();
                 // create a file on root folder
                 DriveFolder.DriveFileResult driveFileResult =
                         folder.createFile(googleApiClient, changeSet, driveContents)