Only retry jobs if we cn do it

Signed-off-by: Mario Danic <mario@lovelyhq.com>
This commit is contained in:
Mario Danic 2017-12-11 00:08:46 +01:00 committed by tobiasKaminsky
parent 34143f4e0f
commit ed93595112
No known key found for this signature in database
GPG key ID: 0E00D4D47D0C5AF7

View file

@ -71,6 +71,7 @@ import com.owncloud.android.operations.UploadFileOperation;
import com.owncloud.android.ui.activity.FileActivity;
import com.owncloud.android.ui.activity.UploadListActivity;
import com.owncloud.android.ui.notifications.NotificationUtils;
import com.owncloud.android.utils.ConnectivityUtils;
import com.owncloud.android.utils.ErrorMessageAdapter;
import com.owncloud.android.utils.PowerUtils;
import com.owncloud.android.utils.ThemeUtils;
@ -355,6 +356,34 @@ public class FileUploader extends Service
}
}
private boolean checkIfUploadCanBeRetried(OCUpload ocUpload, boolean gotWifi, boolean isCharging) {
boolean needsWifi = ocUpload.isUseWifiOnly();
boolean needsCharging = ocUpload.isWhileChargingOnly();
boolean works = false;
if (needsCharging) {
if (isCharging) {
works = true;
} else {
return false;
}
}
if (needsWifi) {
if (gotWifi) {
works = true;
} else {
return false;
}
}
if (!new File(ocUpload.getLocalPath()).exists()) {
works = false;
}
return works;
}
/**
* Retry a subset of all the stored failed uploads.
*
@ -370,6 +399,13 @@ public class FileUploader extends Service
Account currentAccount = null;
boolean resultMatch;
boolean accountMatch;
boolean gotNetwork = !Device.getNetworkType(context).equals(JobRequest.NetworkType.ANY) &&
!ConnectivityUtils.isInternetWalled(context);
boolean gotWifi = gotNetwork && Device.getNetworkType(context).equals(JobRequest.NetworkType.UNMETERED);
boolean charging = Device.getBatteryStatus(context).isCharging();
boolean isPowerSaving = PowerUtils.isPowerSaveMode(context);
for ( OCUpload failedUpload: failedUploads) {
accountMatch = (account == null || account.name.equals(failedUpload.getAccountName()));
resultMatch = ((uploadResult == null || uploadResult.equals(failedUpload.getLastResult())));
@ -378,10 +414,13 @@ public class FileUploader extends Service
!currentAccount.name.equals(failedUpload.getAccountName())) {
currentAccount = failedUpload.getAccount(context);
}
if (!isPowerSaving && gotNetwork && checkIfUploadCanBeRetried(failedUpload, gotWifi, charging)) {
retry(context, currentAccount, failedUpload);
}
}
}
}
/**
* Private implementation of retry.