flywithu/ownCloud-SMS-App
1
0
Fork 0
mirror of https://github.com/nerzhul/ownCloud-SMS-App.git synced 2025-10-30 18:07:51 +00:00
Code Issues Releases Wiki Activity

Use foreach loops to improve a little bit performance.

Browse Source 
This also permit to fix a bug on periodic sync fetches
This commit is contained in:
Loic Blot 2015-11-17 20:33:44 +01:00
parent facdd5032f
commit ac239f1f71
1 changed files with 5 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/main/java/fr/unix_experience/owncloud_sms/activities/GeneralSettingsActivity.java
View File

@ -97,14 +97,12 @@ public class GeneralSettingsActivity extends NrzSettingsActivity {
long syncFreq = Long.parseLong(value);
// Get ownCloud SMS account list
for (int i = 0; i < myAccountList.length; i++) {
for (Account acct: myAccountList) {
// And get all authorities for this account
List<PeriodicSync> syncList = ContentResolver.getPeriodicSyncs(myAccountList[i], GeneralSettingsActivity._accountAuthority);
List<PeriodicSync> syncList = ContentResolver.getPeriodicSyncs(acct, GeneralSettingsActivity._accountAuthority);
boolean foundSameSyncCycle = false;
for (int j = 0; j < syncList.size(); j++) {
PeriodicSync ps = syncList.get(i);
for (PeriodicSync ps: syncList) {
if ((ps.period == syncFreq) && (ps.extras.getInt("synctype") == 1)) {
foundSameSyncCycle = true;
}
@ -114,11 +112,9 @@ public class GeneralSettingsActivity extends NrzSettingsActivity {
Bundle b = new Bundle();
b.putInt("synctype", 1);
ContentResolver.removePeriodicSync(myAccountList[i],
GeneralSettingsActivity._accountAuthority, b);
ContentResolver.removePeriodicSync(acct, GeneralSettingsActivity._accountAuthority, b);
if (syncFreq > 0) {
ContentResolver.addPeriodicSync(myAccountList[i],
GeneralSettingsActivity._accountAuthority, b, syncFreq * 60);
ContentResolver.addPeriodicSync(acct, GeneralSettingsActivity._accountAuthority, b, syncFreq * 60);
}
}