PawletOS/lineageos_updater
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use single return statement in onStartCommand()

Browse Source 
This should prevent future changes from accidentally turning a
sticky service into a not sticky one.

Change-Id: I89f172626a98635149441a63536e0aa5a27f8280
This commit is contained in:
Gabriele M
2018-03-24 16:17:20 +01:00
parent 2ec8fc6cec
commit 3aabbc7ed5
1 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/org/lineageos/updater/controller/UpdaterService.java
View File

@@ -167,9 +167,7 @@ public class UpdaterService extends Service {
// The service is being restarted.
ABUpdateInstaller installer = ABUpdateInstaller.getInstance(this,
mUpdaterController);
if (installer.reconnect()) {
return START_STICKY;
}
installer.reconnect();
}
} else if (ACTION_DOWNLOAD_CONTROL.equals(intent.getAction())) {
String downloadId = intent.getStringExtra(EXTRA_DOWNLOAD_ID);
@@ -191,9 +189,7 @@ public class UpdaterService extends Service {
if (Utils.isABUpdate(update.getFile())) {
ABUpdateInstaller installer = ABUpdateInstaller.getInstance(this,
mUpdaterController);
if (installer.install(downloadId)) {
return START_STICKY;
}
installer.install(downloadId);
} else {
UpdateInstaller installer = new UpdateInstaller(this, mUpdaterController);
installer.install(downloadId);
@@ -215,7 +211,7 @@ public class UpdaterService extends Service {
installer.cancel();
}
}
return START_NOT_STICKY;
return ABUpdateInstaller.isInstallingUpdate(this) ? START_STICKY : START_NOT_STICKY;
}
public Controller getUpdaterController() {