diff options
author | ByteHamster <info@bytehamster.com> | 2021-03-27 22:31:10 +0100 |
---|---|---|
committer | ByteHamster <info@bytehamster.com> | 2021-03-28 22:32:40 +0200 |
commit | 998535515bae8ec5bcd2bd95bc0cf4f34416dff7 (patch) | |
tree | 0e069520f4ef706b428e587cccfd94c0434f3600 /app/src/main/java/de/danoeh/antennapod/asynctask | |
parent | b1e15e8704f459014d76332014e81c3619bbc51c (diff) | |
download | AntennaPod-998535515bae8ec5bcd2bd95bc0cf4f34416dff7.zip |
Make AntennaPod pass the medium SpotBugs findings
Diffstat (limited to 'app/src/main/java/de/danoeh/antennapod/asynctask')
-rw-r--r-- | app/src/main/java/de/danoeh/antennapod/asynctask/DocumentFileExportWorker.java | 4 | ||||
-rw-r--r-- | app/src/main/java/de/danoeh/antennapod/asynctask/ExportWorker.java | 4 |
2 files changed, 2 insertions, 6 deletions
diff --git a/app/src/main/java/de/danoeh/antennapod/asynctask/DocumentFileExportWorker.java b/app/src/main/java/de/danoeh/antennapod/asynctask/DocumentFileExportWorker.java index 906d50c61..73e19c746 100644 --- a/app/src/main/java/de/danoeh/antennapod/asynctask/DocumentFileExportWorker.java +++ b/app/src/main/java/de/danoeh/antennapod/asynctask/DocumentFileExportWorker.java @@ -5,7 +5,6 @@ import android.net.Uri; import androidx.annotation.NonNull; import androidx.documentfile.provider.DocumentFile; -import java.io.FileNotFoundException; import java.io.IOException; import java.io.OutputStream; import java.io.OutputStreamWriter; @@ -37,9 +36,6 @@ public class DocumentFileExportWorker { OutputStreamWriter writer = null; try { Uri uri = output.getUri(); - if (uri == null) { - throw new FileNotFoundException("Export file not found."); - } outputStream = context.getContentResolver().openOutputStream(uri); if (outputStream == null) { throw new IOException(); diff --git a/app/src/main/java/de/danoeh/antennapod/asynctask/ExportWorker.java b/app/src/main/java/de/danoeh/antennapod/asynctask/ExportWorker.java index 0930b59eb..af9f041af 100644 --- a/app/src/main/java/de/danoeh/antennapod/asynctask/ExportWorker.java +++ b/app/src/main/java/de/danoeh/antennapod/asynctask/ExportWorker.java @@ -41,8 +41,8 @@ public class ExportWorker { public Observable<File> exportObservable() { if (output.exists()) { - Log.w(TAG, "Overwriting previously exported file."); - output.delete(); + boolean success = output.delete(); + Log.w(TAG, "Overwriting previously exported file: " + success); } return Observable.create(subscriber -> { OutputStreamWriter writer = null; |