From 2c887ff4fbd0180c5c94c7102ae9ab8ef2547642 Mon Sep 17 00:00:00 2001 From: Daniel Schaal Date: Tue, 29 Sep 2015 06:43:54 +0200 Subject: Convert OwnCloud_Reader to a singleton --- .../services/OwnCloudSyncService.java | 29 +++++++++------------- .../services/SyncItemStateService.java | 9 +++---- 2 files changed, 15 insertions(+), 23 deletions(-) (limited to 'News-Android-App/src/main/java/de/luhmer/owncloudnewsreader/services') diff --git a/News-Android-App/src/main/java/de/luhmer/owncloudnewsreader/services/OwnCloudSyncService.java b/News-Android-App/src/main/java/de/luhmer/owncloudnewsreader/services/OwnCloudSyncService.java index 139538ce..9167db86 100644 --- a/News-Android-App/src/main/java/de/luhmer/owncloudnewsreader/services/OwnCloudSyncService.java +++ b/News-Android-App/src/main/java/de/luhmer/owncloudnewsreader/services/OwnCloudSyncService.java @@ -72,11 +72,11 @@ public class OwnCloudSyncService extends Service { public void startSync() throws RemoteException { if(!isSyncRunning()) { // Only check for API version once - if(_Reader.getApi() == null) { - _Reader.Start_AsyncTask_GetVersion(OwnCloudSyncService.this, onAsyncTask_GetVersionFinished); + if(OwnCloud_Reader.getInstance().getApi() == null) { + OwnCloud_Reader.getInstance().Start_AsyncTask_GetVersion(OwnCloudSyncService.this, onAsyncTask_GetVersionFinished); startedSync(SYNC_TYPES.SYNC_TYPE__GET_API); } else { - _Reader.Start_AsyncTask_PerformItemStateChange(OwnCloudSyncService.this, onAsyncTask_PerformTagExecute); + OwnCloud_Reader.getInstance().Start_AsyncTask_PerformItemStateChange(OwnCloudSyncService.this, onAsyncTask_PerformTagExecute); startedSync(SYNC_TYPES.SYNC_TYPE__ITEM_STATES); } } @@ -84,25 +84,20 @@ public class OwnCloudSyncService extends Service { @Override public boolean isSyncRunning() throws RemoteException { - return _Reader.isSyncRunning(); + return OwnCloud_Reader.getInstance().isSyncRunning(); } }; - - - static OwnCloud_Reader _Reader; @Override public void onCreate() { super.onCreate(); - if(_Reader == null) - _Reader = new OwnCloud_Reader(); Log.d(TAG, "onCreate() called"); } @Override public boolean onUnbind(Intent intent) { //Destroy service if no sync is running - if(!_Reader.isSyncRunning()) { + if(!OwnCloud_Reader.getInstance().isSyncRunning()) { Log.v(TAG, "Stopping service because of inactivity"); stopSelf(); } @@ -123,9 +118,9 @@ public class OwnCloudSyncService extends Service { SharedPreferences mPrefs = PreferenceManager.getDefaultSharedPreferences(OwnCloudSyncService.this); String baseUrl = mPrefs.getString(SettingsActivity.EDT_OWNCLOUDROOTPATH_STRING, ""); API api = API.GetRightApiForVersion(appVersion, baseUrl); - _Reader.setApi(api); - - _Reader.Start_AsyncTask_PerformItemStateChange(OwnCloudSyncService.this, onAsyncTask_PerformTagExecute); + OwnCloud_Reader.getInstance().setApi(api); + + OwnCloud_Reader.getInstance().Start_AsyncTask_PerformItemStateChange(OwnCloudSyncService.this, onAsyncTask_PerformTagExecute); startedSync(SYNC_TYPES.SYNC_TYPE__ITEM_STATES); } @@ -146,13 +141,13 @@ public class OwnCloudSyncService extends Service { if((Boolean) task_result) { if(task_id == Constants.TaskID_PerformStateChange) { - _Reader.Start_AsyncTask_GetFolder(OwnCloudSyncService.this, onAsyncTask_GetFolder); + OwnCloud_Reader.getInstance().Start_AsyncTask_GetFolder(OwnCloudSyncService.this, onAsyncTask_GetFolder); startedSync(SYNC_TYPES.SYNC_TYPE__FOLDER); } else - _Reader.setSyncRunning(true); + OwnCloud_Reader.getInstance().setSyncRunning(true); } } } @@ -168,7 +163,7 @@ public class OwnCloudSyncService extends Service { if(task_result != null) ThrowException((Exception) task_result); else { - _Reader.Start_AsyncTask_GetFeeds(OwnCloudSyncService.this, onAsyncTask_GetFeed); + OwnCloud_Reader.getInstance().Start_AsyncTask_GetFeeds(OwnCloudSyncService.this, onAsyncTask_GetFeed); startedSync(SYNC_TYPES.SYNC_TYPE__FEEDS); } @@ -188,7 +183,7 @@ public class OwnCloudSyncService extends Service { if(task_result != null) ThrowException((Exception) task_result); else { - _Reader.Start_AsyncTask_GetItems(OwnCloudSyncService.this, onAsyncTask_GetItems, FeedItemTags.ALL);//Recieve all unread Items + OwnCloud_Reader.getInstance().Start_AsyncTask_GetItems(OwnCloudSyncService.this, onAsyncTask_GetItems, FeedItemTags.ALL);//Recieve all unread Items startedSync(SYNC_TYPES.SYNC_TYPE__ITEMS); } diff --git a/News-Android-App/src/main/java/de/luhmer/owncloudnewsreader/services/SyncItemStateService.java b/News-Android-App/src/main/java/de/luhmer/owncloudnewsreader/services/SyncItemStateService.java index 2b5d5974..44994d0d 100644 --- a/News-Android-App/src/main/java/de/luhmer/owncloudnewsreader/services/SyncItemStateService.java +++ b/News-Android-App/src/main/java/de/luhmer/owncloudnewsreader/services/SyncItemStateService.java @@ -36,9 +36,6 @@ import de.luhmer.owncloudnewsreader.reader.owncloud.API; import de.luhmer.owncloudnewsreader.reader.owncloud.OwnCloud_Reader; public class SyncItemStateService extends IntentService { - - OwnCloud_Reader _Reader = new OwnCloud_Reader(); - public SyncItemStateService() { super(null); } @@ -49,7 +46,7 @@ public class SyncItemStateService extends IntentService { @Override protected void onHandleIntent(Intent intent) { - _Reader.Start_AsyncTask_GetVersion(this, onAsyncTask_GetVersionFinished); + OwnCloud_Reader.getInstance().Start_AsyncTask_GetVersion(this, onAsyncTask_GetVersionFinished); } OnAsyncTaskCompletedListener onAsyncTask_GetVersionFinished = new OnAsyncTaskCompletedListener() { @@ -64,9 +61,9 @@ public class SyncItemStateService extends IntentService { String baseUrl = mPrefs.getString(SettingsActivity.EDT_OWNCLOUDROOTPATH_STRING, ""); API api = API.GetRightApiForVersion(appVersion, baseUrl); - _Reader.setApi(api); + OwnCloud_Reader.getInstance().setApi(api); - _Reader.Start_AsyncTask_PerformItemStateChange(SyncItemStateService.this, null); + OwnCloud_Reader.getInstance().Start_AsyncTask_PerformItemStateChange(SyncItemStateService.this, null); } } }; -- cgit v1.2.3