Skip to content

Commit 0f9e1ce

Browse files
author
Julian Kast
committed
Reverted breaking change to make sure when the overwrite error is hit, we return true
1 parent ecd6bea commit 0f9e1ce

3 files changed

Lines changed: 3 additions & 4 deletions

File tree

android/sdl_android/src/androidTest/java/com/smartdevicelink/managers/file/FileManagerTests.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -876,8 +876,7 @@ public void onComplete(final Map<String, String> errors) {
876876
assertOnMainThread(new Runnable() {
877877
@Override
878878
public void run() {
879-
assertTrue(errors != null);
880-
assertTrue(errors.containsValue(fileManager.fileManagerCannotOverwriteError));
879+
verify(internalInterface, times(1)).sendRPC(any(PutFile.class));
881880
}
882881
});
883882
}

base/src/main/java/com/smartdevicelink/managers/file/BaseFileManager.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -417,7 +417,7 @@ public void onComplete(boolean success, int bytesAvailable, Collection<String> f
417417
if (!file.isPersistent()) {
418418
BaseFileManager.this.uploadedEphemeralFileNames.add(fileClone.getName());
419419
}
420-
} else if (!fileManagerCannotOverwriteError.equals(errorMessage)) {
420+
} else {
421421
incrementFailedUploadCountForFileName(fileClone.getName(), BaseFileManager.this.failedFileUploadsCount);
422422

423423
int maxUploadCount = fileClone instanceof SdlArtwork ? maxArtworkUploadAttempts : maxFileUploadAttempts;

base/src/main/java/com/smartdevicelink/managers/file/UploadFileOperation.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ private void start() {
9696
if (!file.getOverwrite() && fileManager.get().mutableRemoteFileNames.contains(file.getName())) {
9797
DebugTool.logWarning(TAG, fileManager.get().fileManagerCannotOverwriteError);
9898
if (this.fileWrapper.getCompletionListener() != null) {
99-
this.fileWrapper.getCompletionListener().onComplete(false, bytesAvailable, null, fileManager.get().fileManagerCannotOverwriteError);
99+
this.fileWrapper.getCompletionListener().onComplete(true, bytesAvailable, null, fileManager.get().fileManagerCannotOverwriteError);
100100
}
101101
onFinished();
102102
return;

0 commit comments

Comments
 (0)