Merge pull request #3 from Magenerds/bugfix/clear-cron-jobs

Fix cron clear and rename command
Magenerds · Aug 3, 2018 · 3c6088d · 3c6088d
2 parents 8779850 + 57d1781
commit 3c6088d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 17 deletions.
diff --git a/Console/Command/CronProcessCheckCommand.php b/Console/Command/CronProcessCheckCommand.php
@@ -53,8 +53,8 @@ public function __construct(
      */
     public function configure()
     {
-        $this->setName('magenerds:cronprocess:check');
-        $this->setDescription('Check running cronjobs');
+        $this->setName('magenerds:cronprocess:clear');
+        $this->setDescription('Set the status of crashed cronjobs to error');
 
         parent::configure();
     }

diff --git a/Plugin/Model/CronSchedulePlugin.php b/Plugin/Model/CronSchedulePlugin.php
@@ -41,19 +41,4 @@ public function afterTryLockJob(Schedule $subject, $canLock)
 
         return $canLock;
     }
-
-    /**
-     * Remove PID if status is not running
-     *
-     * @param Schedule $subject
-     * @param $key
-     * @param $value
-     * @return void
-     */
-    public function beforeSetData(Schedule $subject, $key, $value = null)
-    {
-        if ($key === 'status' && $value !== Schedule::STATUS_RUNNING) {
-            $subject->setPid(NULL);
-        }
-    }
 }