- Modifies cron router/UI to display proper status message when WP task

scheduler is configured and cron is not running
- Updates sending queue worker and related components to stop (delete)
  cron when all processing is done

assets/js/src/cron.jsx

@@ -53,10 +53,13 @@ define(
       }.bind(this));
     },
     render: function() {
-      if(this.state.status === 'loading') {
-        return(<div>{MailPoet.I18n.t('loadingDaemonStatus')}</div>);
-      }
       switch(this.state.status) {
+        case 'loading':
+          return(
+            <div>
+              {MailPoet.I18n.t('loadingDaemonStatus')}
+            </div>
+          );
         case 'started':
           return(
             <div>
@@ -85,6 +88,20 @@ define(
             </div>
           );
         break;
+        case 'wordpress_task_scheduler_enabled':
+          return(
+            <div>
+              {MailPoet.I18n.t('wordpressTaskSchedulerEnabled')}
+            </div>
+          );
+        break;
+        case false:
+          return(
+            <div>
+              {MailPoet.I18n.t('daemonNotRunning')}
+              </div>
+          );
+        break;
       }
     }
   });

lib/API/Endpoints/Cron.php

@@ -1,6 +1,7 @@
 <?php
 namespace MailPoet\API\Endpoints;
 
+use MailPoet\Config\TaskScheduler;
 use MailPoet\Cron\CronHelper;
 use MailPoet\Cron\Supervisor;
 use MailPoet\Models\Setting;
@@ -27,10 +28,14 @@ class Cron {
   }
 
   function getStatus() {
-    $daemon = Setting::where('name', 'cron_daemon')
-      ->findOne();
-    return ($daemon) ?
-      unserialize($daemon->value) :
+    $task_scheduler = TaskScheduler::getCurrentMethod();
+    $daemon = Setting::getSetting(CronHelper::DAEMON_SETTING);
+    if($daemon) {
+      return $daemon;
+    }
+    $status = ($task_scheduler === TaskScheduler::METHOD_WORDPRESS) ?
+      'wordpress_task_scheduler_enabled' :
       false;
+    return array('status' => $status);
   }
 }

lib/Cron/CronHelper.php

@@ -12,6 +12,7 @@ class CronHelper {
   const DAEMON_EXECUTION_LIMIT = 20;
   const DAEMON_EXECUTION_TIMEOUT = 35;
   const DAEMON_REQUEST_TIMEOUT = 2;
+  const DAEMON_SETTING = 'cron_daemon';
 
   static function createDaemon($token) {
     $daemon = array(
@@ -23,17 +24,27 @@ class CronHelper {
   }
 
   static function getDaemon() {
-    return Setting::getValue('cron_daemon');
+    return Setting::getValue(self::DAEMON_SETTING);
   }
 
   static function saveDaemon($daemon) {
     $daemon['updated_at'] = time();
     return Setting::setValue(
-      'cron_daemon',
+      self::DAEMON_SETTING,
       $daemon
     );
   }
 
+  static function stopDaemon() {
+    $daemon = self::getDaemon();
+    $daemon['status'] = Daemon::STATUS_STOPPED;
+    return self::saveDaemon($daemon);
+  }
+
+  static function deleteDaemon() {
+    return Setting::deleteSetting(self::DAEMON_SETTING);
+  }
+
   static function createToken() {
     return Security::generateRandomString();
   }

lib/Cron/Workers/SendingQueue/Tasks/Newsletter.php

@@ -62,7 +62,7 @@ class Newsletter {
     return $newsletter;
   }
 
-  function render($newsletter) {
+  function render(array $newsletter) {
     $renderer = new Renderer($newsletter);
     $newsletter['rendered_body'] = $renderer->render();
     return $newsletter;
@@ -103,4 +103,12 @@ class Newsletter {
       )
     );
   }
+
+  function markNewsletterAsSent($queue_id) {
+    $newsletter = NewsletterModel::findOne($queue_id);
+    // if it's a standard newsletter, update its status
+    if($newsletter->type === NewsletterModel::TYPE_STANDARD) {
+      $newsletter->setStatus(NewsletterModel::STATUS_SENT);
+    }
+  }
 }

lib/Models/Setting.php

@@ -160,4 +160,13 @@ class Setting extends Model {
 
     return $setting->save();
   }
+
+  public static function getSetting($setting) {
+    return self::where('name', $setting)->findOne();
+  }
+
+  public static function deleteSetting($setting) {
+    $setting = self::where('name', $setting)->findOne();
+    return ($setting) ? $setting->delete() : false;
+  }
 }

views/cron.html

@@ -6,9 +6,11 @@
 
 <% block translations %>
   <%= localize({
+    'daemonNotRunning': __('Daemon is not running'),
+    'wordpressTaskSchedulerEnabled': __('Daemon is not running because WordPress task scheduler is enabled and there are no scheduled or in-progress jobs'),
     'daemonControlError': __('Cron daemon error'),
     'loadingDaemonStatus': __('Loading daemon status...'),
-    'cronDaemonIsRunning': __('Cron daemon is running.'),
+    'cronDaemonIsRunning': __('Cron daemon is running'),
     'stop': __('Stop'),
     'start': __('Start'),
     'cronDaemonState': __('Cron is %$1s')