Fabio/brouter
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Report errors from DownloadWorker

Browse source
This commit is contained in:
Manuel Fuhr 2022-05-27 07:04:48 +02:00
parent 9445361f28
commit eae79d3425
2 changed files with 16 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
brouter-routing-app/src/main/java/btools/routingapp/BInstallerActivity.java
View file

@ -181,18 +181,23 @@ public class BInstallerActivity extends AppCompatActivity {
String result; String result;
switch (workInfo.getState()) { switch (workInfo.getState()) {
case FAILED: case FAILED:
result = "failed."; result = "Download failed";
break; break;
case CANCELLED: case CANCELLED:
result = "cancelled"; result = "Download cancelled";
break; break;
case SUCCEEDED: case SUCCEEDED:
result = "succeeded"; result = "Download succeeded";
break; break;
default: default:
result = ""; result = "";
} }
Toast.makeText(this, "Download " + result + ".", Toast.LENGTH_SHORT).show(); if (workInfo.getState() != WorkInfo.State.FAILED) {
Toast.makeText(this, result, Toast.LENGTH_SHORT).show();
} else {
String error = workInfo.getOutputData().getString(DownloadWorker.KEY_OUTPUT_ERROR);
Toast.makeText(this, result + ": " + error, Toast.LENGTH_LONG).show();
}
mProgressIndicator.hide(); mProgressIndicator.hide();
scanExistingFiles(); scanExistingFiles();
} }

10
brouter-routing-app/src/main/java/btools/routingapp/DownloadWorker.java
View file

@ -32,6 +32,7 @@ import btools.util.ProgressListener;
public class DownloadWorker extends Worker { public class DownloadWorker extends Worker {
public static final String KEY_INPUT_SEGMENT_NAMES = "SEGMENT_NAMES"; public static final String KEY_INPUT_SEGMENT_NAMES = "SEGMENT_NAMES";
public static final String KEY_OUTPUT_ERROR = "ERROR";
public static final String PROGRESS_SEGMENT_NAME = "PROGRESS_SEGMENT_NAME"; public static final String PROGRESS_SEGMENT_NAME = "PROGRESS_SEGMENT_NAME";
public static final String PROGRESS_SEGMENT_PERCENT = "PROGRESS_SEGMENT_PERCENT"; public static final String PROGRESS_SEGMENT_PERCENT = "PROGRESS_SEGMENT_PERCENT";
@ -128,6 +129,7 @@ public class DownloadWorker extends Worker {
@Override @Override
public Result doWork() { public Result doWork() {
Data inputData = getInputData(); Data inputData = getInputData();
Data.Builder output = new Data.Builder();
String[] segmentNames = inputData.getStringArray(KEY_INPUT_SEGMENT_NAMES); String[] segmentNames = inputData.getStringArray(KEY_INPUT_SEGMENT_NAMES);
if (segmentNames == null) { if (segmentNames == null) {
if (DEBUG) Log.d(LOG_TAG, "Failure: no segmentNames"); if (DEBUG) Log.d(LOG_TAG, "Failure: no segmentNames");
@ -147,10 +149,12 @@ public class DownloadWorker extends Worker {
} }
} catch (IOException e) { } catch (IOException e) {
Log.w(LOG_TAG, e); Log.w(LOG_TAG, e);
return Result.failure(); output.putString(KEY_OUTPUT_ERROR, e.toString());
return Result.failure(output.build());
} catch (InterruptedException e) { } catch (InterruptedException e) {
Log.w(LOG_TAG, e); Log.w(LOG_TAG, e);
return Result.failure(); output.putString(KEY_OUTPUT_ERROR, e.toString());
return Result.failure(output.build());
} }
if (DEBUG) Log.d(LOG_TAG, "doWork finished"); if (DEBUG) Log.d(LOG_TAG, "doWork finished");
return Result.success(); return Result.success();
@ -242,7 +246,7 @@ public class DownloadWorker extends Worker {
connection.connect(); connection.connect();
if (connection.getResponseCode() != HttpURLConnection.HTTP_OK) { if (connection.getResponseCode() != HttpURLConnection.HTTP_OK) {
throw new IOException("HTTP Request failed"); throw new IOException("HTTP Request failed: " + downloadUrl + " returned " + connection.getResponseCode());
} }
int fileLength = connection.getContentLength(); int fileLength = connection.getContentLength();
try ( try (