Do no run SyncJob in cron

Fixes #7184

The SyncJob can be very resource intensive. Since it requests all users
on the system to create the system addressbook. In order to do this it
creates a vcard for every user and updates the addressbook.

There is no need for this job since the proper signals are emitted and
handled in the carddav backend to update the addressbook live.

Worst comes to worst there is always the occ command to bring the
address book in sync again.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

diff --git a/apps/dav/appinfo/info.xml b/apps/dav/appinfo/info.xml
index 2d9f73b3f43eafc097be14a96fe731d8ac59c6a8..b4e136f039b331775f0b4294b3c06c778039a5f1 100644 (file)
--- a/apps/dav/appinfo/info.xml
+++ b/apps/dav/appinfo/info.xml
@@ -5,7 +5,7 @@
        <description>WebDAV endpoint</description>
        <licence>AGPL</licence>
        <author>owncloud.org</author>
-       <version>1.3.0</version>
+       <version>1.3.1</version>
        <default_enable/>
        <types>
                <filesystem/>
@@ -17,9 +17,6 @@
        <dependencies>
                <nextcloud min-version="12" max-version="12" />
        </dependencies>
-       <background-jobs>
-               <job>OCA\DAV\CardDAV\SyncJob</job>
-       </background-jobs>
        <repair-steps>
                <post-migration>
                        <step>OCA\DAV\Migration\FixBirthdayCalendarComponent</step>

diff --git a/apps/dav/lib/CardDAV/SyncJob.php b/apps/dav/lib/CardDAV/SyncJob.php
deleted file mode 100644 (file)
index f0f8d51..0000000
--- a/apps/dav/lib/CardDAV/SyncJob.php
+++ /dev/null
@@ -1,42 +0,0 @@
-<?php
-/**
- * @copyright Copyright (c) 2016, ownCloud, Inc.
- *
- * @author Joas Schilling <coding@schilljs.com>
- * @author Thomas Müller <thomas.mueller@tmit.eu>
- *
- * @license AGPL-3.0
- *
- * This code is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Affero General Public License, version 3,
- * as published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU Affero General Public License for more details.
- *
- * You should have received a copy of the GNU Affero General Public License, version 3,
- * along with this program.  If not, see <http://www.gnu.org/licenses/>
- *
- */
-
-namespace OCA\DAV\CardDAV;
-
-use OC\BackgroundJob\TimedJob;
-use OCA\DAV\AppInfo\Application;
-
-class SyncJob extends TimedJob {
-
-       public function __construct() {
-               // Run once a day
-               $this->setInterval(24 * 60 * 60);
-       }
-
-       protected function run($argument) {
-               $app = new Application();
-               /** @var SyncService $ss */
-               $ss = $app->getSyncService();
-               $ss->syncInstance();
-       }
-}