@@ -136,7 +136,7 @@ discard block |
||
| 136 | 136 | /** |
| 137 | 137 | * Set nextRun |
| 138 | 138 | * |
| 139 | - * @param datetime $nextRun |
|
| 139 | + * @param \DateTime $nextRun |
|
| 140 | 140 | */ |
| 141 | 141 | public function setNextRun($nextRun) |
| 142 | 142 | { |
@@ -156,7 +156,7 @@ discard block |
||
| 156 | 156 | /** |
| 157 | 157 | * Add results |
| 158 | 158 | * |
| 159 | - * @param Alpixel\Bundle\CronBundle\Entity\CronJobResult $results |
|
| 159 | + * @param CronJobResult $results |
|
| 160 | 160 | */ |
| 161 | 161 | public function addCronJobResult(\Alpixel\Bundle\CronBundle\Entity\CronJobResult $results) |
| 162 | 162 | { |
@@ -176,7 +176,7 @@ discard block |
||
| 176 | 176 | /** |
| 177 | 177 | * Set mostRecentRun |
| 178 | 178 | * |
| 179 | - * @param Alpixel\Bundle\CronBundle\Entity\CronJobResult $mostRecentRun |
|
| 179 | + * @param CronJobResult $mostRecentRun |
|
| 180 | 180 | */ |
| 181 | 181 | public function setMostRecentRun(\Alpixel\Bundle\CronBundle\Entity\CronJobResult $mostRecentRun) |
| 182 | 182 | { |
@@ -186,7 +186,7 @@ discard block |
||
| 186 | 186 | /** |
| 187 | 187 | * Get mostRecentRun |
| 188 | 188 | * |
| 189 | - * @return Alpixel\Bundle\CronBundle\Entity\CronJobResult |
|
| 189 | + * @return CronJobResult |
|
| 190 | 190 | */ |
| 191 | 191 | public function getMostRecentRun() |
| 192 | 192 | { |
@@ -150,7 +150,7 @@ discard block |
||
| 150 | 150 | /** |
| 151 | 151 | * Set job |
| 152 | 152 | * |
| 153 | - * @param Alpixel\Bundle\CronBundle\Entity\CronJob $job |
|
| 153 | + * @param CronJob $job |
|
| 154 | 154 | */ |
| 155 | 155 | public function setJob(\Alpixel\Bundle\CronBundle\Entity\CronJob $job) |
| 156 | 156 | { |
@@ -160,7 +160,7 @@ discard block |
||
| 160 | 160 | /** |
| 161 | 161 | * Get job |
| 162 | 162 | * |
| 163 | - * @return Alpixel\Bundle\CronBundle\Entity\CronJob |
|
| 163 | + * @return CronJob |
|
| 164 | 164 | */ |
| 165 | 165 | public function getJob() |
| 166 | 166 | { |
@@ -53,7 +53,7 @@ discard block |
||
| 53 | 53 | $em->flush(); |
| 54 | 54 | |
| 55 | 55 | $end = microtime(true); |
| 56 | - $duration = sprintf('%0.2f', $end - $start); |
|
| 56 | + $duration = sprintf('%0.2f', $end-$start); |
|
| 57 | 57 | $output->writeln("Cron run completed in $duration seconds"); |
| 58 | 58 | } |
| 59 | 59 | |
@@ -100,11 +100,11 @@ discard block |
||
| 100 | 100 | $statusStr = 'failed'; |
| 101 | 101 | } |
| 102 | 102 | |
| 103 | - $durationStr = sprintf('%0.2f', $jobEnd - $jobStart); |
|
| 103 | + $durationStr = sprintf('%0.2f', $jobEnd-$jobStart); |
|
| 104 | 104 | $output->writeln("$statusStr in $durationStr seconds"); |
| 105 | 105 | |
| 106 | 106 | // Record the result |
| 107 | - $this->recordJobResult($em, $job, $jobEnd - $jobStart, $jobOutput->getOutput(), $returnCode); |
|
| 107 | + $this->recordJobResult($em, $job, $jobEnd-$jobStart, $jobOutput->getOutput(), $returnCode); |
|
| 108 | 108 | |
| 109 | 109 | // And update the job with it's next scheduled time |
| 110 | 110 | $newTime = new \DateTime(); |
@@ -114,6 +114,10 @@ |
||
| 114 | 114 | $job->setNextRun($newTime); |
| 115 | 115 | } |
| 116 | 116 | |
| 117 | + /** |
|
| 118 | + * @param CronJob $timeTaken |
|
| 119 | + * @param string $resultCode |
|
| 120 | + */ |
|
| 117 | 121 | protected function recordJobResult(CronJob $job, $timeTaken, $output, $resultCode) |
| 118 | 122 | { |
| 119 | 123 | |