From 583aba1b442a1b41a11442d51dc31c5d1f18617e Mon Sep 17 00:00:00 2001 From: Daniel Gultsch Date: Thu, 2 Jun 2016 21:37:52 +0200 Subject: [PATCH] print specific toast when download failed because of write error --- .../conversations/http/HttpDownloadConnection.java | 14 ++++++++++++-- src/main/res/values/strings.xml | 1 + 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/src/main/java/eu/siacs/conversations/http/HttpDownloadConnection.java b/src/main/java/eu/siacs/conversations/http/HttpDownloadConnection.java index d23cb71a3..cffed30ae 100644 --- a/src/main/java/eu/siacs/conversations/http/HttpDownloadConnection.java +++ b/src/main/java/eu/siacs/conversations/http/HttpDownloadConnection.java @@ -140,12 +140,18 @@ public class HttpDownloadConnection implements Transferable { mXmppConnectionService.updateConversationUi(); } + private class WriteException extends IOException { + + } + private void showToastForException(Exception e) { e.printStackTrace(); if (e instanceof java.net.UnknownHostException) { mXmppConnectionService.showErrorToastInUi(R.string.download_failed_server_not_found); } else if (e instanceof java.net.ConnectException) { mXmppConnectionService.showErrorToastInUi(R.string.download_failed_could_not_connect); + } else if (e instanceof WriteException) { + mXmppConnectionService.showErrorToastInUi(R.string.download_failed_could_not_write_file); } else if (!(e instanceof CancellationException)) { mXmppConnectionService.showErrorToastInUi(R.string.download_failed_file_not_found); } @@ -284,11 +290,15 @@ public class HttpDownloadConnection implements Transferable { file.createNewFile(); os = AbstractConnectionManager.createOutputStream(file, true); } - int count = -1; + int count; byte[] buffer = new byte[1024]; while ((count = is.read(buffer)) != -1) { transmitted += count; - os.write(buffer, 0, count); + try { + os.write(buffer, 0, count); + } catch (IOException e) { + throw new WriteException(); + } updateProgress((int) ((((double) transmitted) / expected) * 100)); if (canceled) { throw new CancellationException(); diff --git a/src/main/res/values/strings.xml b/src/main/res/values/strings.xml index 51c658ab0..c2810639a 100644 --- a/src/main/res/values/strings.xml +++ b/src/main/res/values/strings.xml @@ -539,6 +539,7 @@ Download failed: Server not found Download failed: File not found Download failed: Could not connect to host + Download failed: Could not write file Use white background Show received messages as black text on a white background Tor network unavailable