Refactor folder sync

appinfo/routes.php

@@ -20,54 +20,77 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>
  *
  */
-
 $app = new \OCA\Mail\AppInfo\Application();
 $app->registerRoutes($this,
 	[
-		'routes' => [
-			['name' => 'page#index', 'url' => '/', 'verb' => 'GET'],
-			['name' => 'page#compose', 'url' => '/compose', 'verb' => 'GET'],
-			['name' => 'accounts#send', 'url' => '/accounts/{accountId}/send', 'verb' => 'POST'],
-			['name' => 'accounts#draft', 'url' => '/accounts/{accountId}/draft', 'verb' => 'POST'],
-			[
-				'name' => 'messages#downloadAttachment',
-				'url' => '/accounts/{accountId}/folders/{folderId}/messages/{messageId}/attachment/{attachmentId}',
-				'verb' => 'GET'],
-			[
-				'name' => 'messages#saveAttachment',
-				'url' => '/accounts/{accountId}/folders/{folderId}/messages/{messageId}/attachment/{attachmentId}',
-				'verb' => 'POST'],
-			[
-				'name' => 'messages#getHtmlBody',
-				'url' => '/accounts/{accountId}/folders/{folderId}/messages/{messageId}/html',
-				'verb' => 'GET'],
-			[
-				'name' => 'messages#setFlags',
-				'url' => '/accounts/{accountId}/folders/{folderId}/messages/{messageId}/flags',
-				'verb' => 'PUT'],
-			[
-				'name' => 'messages#move',
-				'url' => '/accounts/{accountId}/folders/{folderId}/messages/{id}/move',
-				'verb' => 'POST'],
-			[
-				'name' => 'proxy#redirect',
-				'url' => '/redirect',
-				'verb' => 'GET'],
-			[
-				'name' => 'proxy#proxy',
-				'url' => '/proxy',
-				'verb' => 'GET'],
-			[
-				'name' => 'folders#detectChanges',
-				'url' => '/accounts/{accountId}/folders/detectChanges',
-				'verb' => 'POST'],
+	'routes' => [
+		[
+			'name' => 'page#index',
+			'url' => '/',
+			'verb' => 'GET'
+		],
+		[
+			'name' => 'page#compose',
+			'url' => '/compose',
+			'verb' => 'GET'
+		],
+		[
+			'name' => 'accounts#send',
+			'url' => '/accounts/{accountId}/send',
+			'verb' => 'POST'
+		],
+		[
+			'name' => 'accounts#draft',
+			'url' => '/accounts/{accountId}/draft',
+			'verb' => 'POST'
+		],
+		[
+			'name' => 'folders#sync',
+			'url' => '/accounts/{accountId}/folders/{folderId}/sync',
+			'verb' => 'GET'
+		],
+		[
+			'name' => 'messages#downloadAttachment',
+			'url' => '/accounts/{accountId}/folders/{folderId}/messages/{messageId}/attachment/{attachmentId}',
+			'verb' => 'GET'
+		],
+		[
+			'name' => 'messages#saveAttachment',
+			'url' => '/accounts/{accountId}/folders/{folderId}/messages/{messageId}/attachment/{attachmentId}',
+			'verb' => 'POST'
+		],
+		[
+			'name' => 'messages#getHtmlBody',
+			'url' => '/accounts/{accountId}/folders/{folderId}/messages/{messageId}/html',
+			'verb' => 'GET'
+		],
+		[
+			'name' => 'messages#setFlags',
+			'url' => '/accounts/{accountId}/folders/{folderId}/messages/{messageId}/flags',
+			'verb' => 'PUT'
+		],
+		[
+			'name' => 'messages#move',
+			'url' => '/accounts/{accountId}/folders/{folderId}/messages/{id}/move',
+			'verb' => 'POST'
+		],
+		[
+			'name' => 'proxy#redirect',
+			'url' => '/redirect',
+			'verb' => 'GET'
+		],
+		[
+			'name' => 'proxy#proxy',
+			'url' => '/proxy',
+			'verb' => 'GET'
 		],
-		'resources' => [
-			'autoComplete' => ['url' => '/autoComplete'],
+	],
+	'resources' => [
+		'autoComplete' => ['url' => '/autoComplete'],
 			'localAttachments' => ['url' => '/attachments'],
-			'accounts' => ['url' => '/accounts'],
-			'folders' => ['url' => '/accounts/{accountId}/folders'],
-			'messages' => ['url' => '/accounts/{accountId}/folders/{folderId}/messages'],
-			'aliases' => ['url' => '/accounts/{accountId}/aliases'],
-		]
-	]);
+		'accounts' => ['url' => '/accounts'],
+		'folders' => ['url' => '/accounts/{accountId}/folders'],
+		'messages' => ['url' => '/accounts/{accountId}/folders/{folderId}/messages'],
+		'aliases' => ['url' => '/accounts/{accountId}/aliases'],
+	]
+]);

js/app.js

@@ -43,6 +43,7 @@ define(function(require) {
 	require('service/attachmentservice');
 	require('service/davservice');
 	require('service/folderservice');
+	require('service/foldersyncservice');
 	require('service/messageservice');
 	require('service/aliasesservice');
 	require('util/notificationhandler');

js/controller/accountcontroller.js

@@ -15,13 +15,6 @@ define(function(require) {
 
 	var FolderController = require('controller/foldercontroller');
 	var Radio = require('radio');
-	var UPDATE_INTERVAL = 5 * 60 * 1000; // 5 minutes
-
-	function startBackgroundChecks(accounts) {
-		setInterval(function() {
-			require('background').checkForNotifications(accounts);
-		}, UPDATE_INTERVAL);
-	}
 
 	/**
 	 * Load all accounts
@@ -44,18 +37,15 @@ define(function(require) {
 			})).then(function() {
 				return accounts;
 			});
-		}).then(function(accounts) {
-			startBackgroundChecks(accounts);
-			return accounts;
-		}, function(e) {
-			console.error(e);
-			Radio.ui.trigger('error:show', t('mail', 'Error while loading the accounts.'));
 		}).then(function(accounts) {
 			// Show accounts regardless of the result of
 			// loading the folders
 			Radio.ui.trigger('sidebar:accounts');
 
 			return accounts;
+		}, function(e) {
+			console.error(e);
+			Radio.ui.trigger('error:show', t('mail', 'Error while loading the accounts.'));
 		});
 	}