Skip to content

Commit c612d7a

Browse files
Merge pull request nextcloud#12316 from nextcloud/missleading_error_toast_in_uploads_tab
Fix misleading toast in uploads tab
2 parents e0304d2 + cd3bdde commit c612d7a

3 files changed

Lines changed: 13 additions & 11 deletions

File tree

app/src/main/java/com/owncloud/android/ui/activity/UploadListActivity.java

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -196,19 +196,22 @@ private void loadItems() {
196196
private void refresh() {
197197
backgroundJobManager.startImmediateFilesSyncJob(false, true);
198198

199-
// retry failed uploads
200-
new Thread(() -> FileUploader.retryFailedUploads(
201-
this,
202-
uploadsStorageManager,
203-
connectivityService,
204-
userAccountManager,
205-
powerManagementService))
206-
.start();
199+
if(uploadsStorageManager.getFailedUploads().length > 0){
200+
// retry failed uploads
201+
new Thread(() -> FileUploader.retryFailedUploads(
202+
this,
203+
uploadsStorageManager,
204+
connectivityService,
205+
userAccountManager,
206+
powerManagementService))
207+
.start();
208+
DisplayUtils.showSnackMessage(this, R.string.uploader_local_files_uploaded);
209+
}
210+
207211

208212
// update UI
209213
uploadListAdapter.loadUploadItemsFromDb();
210214
swipeListRefreshLayout.setRefreshing(false);
211-
DisplayUtils.showSnackMessage(this, R.string.uploader_local_files_uploaded);
212215
}
213216

214217
@Override

app/src/main/java/com/owncloud/android/utils/FilesUploadHelper.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,6 @@ class FilesUploadHelper {
151151
val boundListener = mBoundListeners[key]
152152

153153
boundListener?.onTransferProgress(progressRate, totalTransferredSoFar, totalToTransfer, fileName)
154-
Log_OC.d("TAG", "Hello")
155154
}
156155
}
157156
}

app/src/main/res/values/strings.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -916,7 +916,7 @@
916916
<string name="failed_to_start_editor">Failed to start editor</string>
917917
<string name="create_rich_workspace">Add folder info</string>
918918
<string name="creates_rich_workspace">creates folder info</string>
919-
<string name="uploader_local_files_uploaded">Try to upload local files again</string>
919+
<string name="uploader_local_files_uploaded">Retry to upload failed local files</string>
920920
<string name="uploader_file_not_found_on_server_message">We couldnt locate the file on server. Another user may have deleted the file</string>
921921
<string name="uploader_file_not_found_message">File not found. Are you sure that this file exists or has a previous conflict not been resolved?</string>
922922
<string name="uploader_upload_failed_sync_conflict_error">File upload conflict</string>

0 commit comments

Comments
 (0)