From 1d2dcedabe29bbafcf4674dfc77653778e971bf0 Mon Sep 17 00:00:00 2001
From: Pengyi Zhang <pengyi@agileware.com.au>
Date: Fri, 13 Dec 2019 15:47:40 +1100
Subject: [PATCH] CIVICHIMP-6 remove jobs

---
 CRM/Mailchimp/Upgrader.php | 14 ++++++++++++++
 mailchimp.php              | 26 --------------------------
 2 files changed, 14 insertions(+), 26 deletions(-)

diff --git a/CRM/Mailchimp/Upgrader.php b/CRM/Mailchimp/Upgrader.php
index 6002b86..aa6bfb8 100644
--- a/CRM/Mailchimp/Upgrader.php
+++ b/CRM/Mailchimp/Upgrader.php
@@ -227,6 +227,20 @@ public function upgrade_21() {
 
     return TRUE;
   }
+
+	/**
+	 * Remove unused jobs
+	 * @throws \CiviCRM_API3_Exception
+	 */
+	public function upgrade_22() {
+		$result = civicrm_api3('Job', 'get', [
+			'sequential' => 1,
+			'name' => ['IN' => ["Mailchimp Push Sync", "Mailchimp Pull Sync"]],
+			'api.Job.delete' => ['id' => "\$value.id"],
+		]);
+
+		return TRUE;
+	}
   /**
    * Example: Run an external SQL script
    *
diff --git a/mailchimp.php b/mailchimp.php
index 03550f5..22b53cb 100644
--- a/mailchimp.php
+++ b/mailchimp.php
@@ -34,32 +34,6 @@ function mailchimp_civicrm_xmlMenu(&$files) {
  * @link http://wiki.civicrm.org/confluence/display/CRMDOC/hook_civicrm_install
  */
 function mailchimp_civicrm_install() {
-
-  // Create a cron job to do sync data between CiviCRM and MailChimp.
-  $params = array(
-    'sequential' => 1,
-    'name'          => 'Mailchimp Push Sync',
-    'description'   => 'Sync contacts between CiviCRM and MailChimp, assuming CiviCRM to be correct. Please understand the implications before using this.',
-    'run_frequency' => 'Daily',
-    'api_entity'    => 'Mailchimp',
-    'api_action'    => 'pushsync',
-    'is_active'     => 0,
-  );
-  $result = civicrm_api3('job', 'create', $params);
-  
-
-  // Create Pull Sync job.
-  $params = array(
-    'sequential' => 1,
-    'name'          => 'Mailchimp Pull Sync',
-    'description'   => 'Sync contacts between CiviCRM and MailChimp, assuming Mailchimp to be correct. Please understand the implications before using this.',
-    'run_frequency' => 'Daily',
-    'api_entity'    => 'Mailchimp',
-    'api_action'    => 'pullsync',
-    'is_active'     => 0,
-  );
-  $result = civicrm_api3('job', 'create', $params);
-
   return _mailchimp_civix_civicrm_install();
 }