@@ -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 |      { | 
                                                        
@@ -9,8 +9,8 @@  | 
                                                    ||
| 9 | 9 | public function getKnownJobs()  | 
                                                        
| 10 | 10 |      { | 
                                                        
| 11 | 11 | $data = $this->getEntityManager()  | 
                                                        
| 12 | -                     ->createQuery('SELECT job.command FROM CronBundle:CronJob job') | 
                                                        |
| 13 | - ->getScalarResult();  | 
                                                        |
| 12 | +                        ->createQuery('SELECT job.command FROM CronBundle:CronJob job') | 
                                                        |
| 13 | + ->getScalarResult();  | 
                                                        |
| 14 | 14 | $toRet = [];  | 
                                                        
| 15 | 15 |          foreach ($data as $datum) { | 
                                                        
| 16 | 16 | $toRet[] = $datum['command'];  | 
                                                        
@@ -12,8 +12,8 @@  | 
                                                    ||
| 12 | 12 | protected function configure()  | 
                                                        
| 13 | 13 |      { | 
                                                        
| 14 | 14 |          $this->setName('cron:enable-job') | 
                                                        
| 15 | -             ->setDescription('Enables a cron job') | 
                                                        |
| 16 | -             ->addArgument('job', InputArgument::REQUIRED, 'Name of the job to enable'); | 
                                                        |
| 15 | +                ->setDescription('Enables a cron job') | 
                                                        |
| 16 | +                ->addArgument('job', InputArgument::REQUIRED, 'Name of the job to enable'); | 
                                                        |
| 17 | 17 | }  | 
                                                        
| 18 | 18 | |
| 19 | 19 | protected function execute(InputInterface $input, OutputInterface $output)  | 
                                                        
@@ -13,8 +13,8 @@  | 
                                                    ||
| 13 | 13 | protected function configure()  | 
                                                        
| 14 | 14 |      { | 
                                                        
| 15 | 15 |          $this->setName('cron:pruneLogs') | 
                                                        
| 16 | -             ->setDescription('Prunes the logs for each cron job, leaving only recent failures and the most recent success') | 
                                                        |
| 17 | -             ->addArgument('job', InputArgument::OPTIONAL, 'Operate only on this job'); | 
                                                        |
| 16 | +                ->setDescription('Prunes the logs for each cron job, leaving only recent failures and the most recent success') | 
                                                        |
| 17 | +                ->addArgument('job', InputArgument::OPTIONAL, 'Operate only on this job'); | 
                                                        |
| 18 | 18 | }  | 
                                                        
| 19 | 19 | |
| 20 | 20 | protected function execute(InputInterface $input, OutputInterface $output)  | 
                                                        
@@ -13,8 +13,8 @@  | 
                                                    ||
| 13 | 13 | protected function configure()  | 
                                                        
| 14 | 14 |      { | 
                                                        
| 15 | 15 |          $this->setName('cron:disable-job') | 
                                                        
| 16 | -             ->setDescription('Disables a cron job') | 
                                                        |
| 17 | -             ->addArgument('job', InputArgument::REQUIRED, 'Name of the job to disable'); | 
                                                        |
| 16 | +                ->setDescription('Disables a cron job') | 
                                                        |
| 17 | +                ->addArgument('job', InputArgument::REQUIRED, 'Name of the job to disable'); | 
                                                        |
| 18 | 18 | }  | 
                                                        
| 19 | 19 | |
| 20 | 20 | protected function execute(InputInterface $input, OutputInterface $output)  | 
                                                        
@@ -14,8 +14,8 @@  | 
                                                    ||
| 14 | 14 | protected function configure()  | 
                                                        
| 15 | 15 |      { | 
                                                        
| 16 | 16 |          $this->setName('cron:status') | 
                                                        
| 17 | -             ->setDescription('Displays the current status of cron jobs') | 
                                                        |
| 18 | -             ->addArgument('job', InputArgument::OPTIONAL, 'Show information for only this job'); | 
                                                        |
| 17 | +                ->setDescription('Displays the current status of cron jobs') | 
                                                        |
| 18 | +                ->addArgument('job', InputArgument::OPTIONAL, 'Show information for only this job'); | 
                                                        |
| 19 | 19 | }  | 
                                                        
| 20 | 20 | |
| 21 | 21 | protected function execute(InputInterface $input, OutputInterface $output)  | 
                                                        
@@ -16,8 +16,8 @@  | 
                                                    ||
| 16 | 16 | protected function configure()  | 
                                                        
| 17 | 17 |      { | 
                                                        
| 18 | 18 |          $this->setName('cron:run') | 
                                                        
| 19 | -             ->setDescription('Runs any currently schedule cron jobs') | 
                                                        |
| 20 | -             ->addArgument('job', InputArgument::OPTIONAL, 'Run only this job (if enabled)'); | 
                                                        |
| 19 | +                ->setDescription('Runs any currently schedule cron jobs') | 
                                                        |
| 20 | +                ->addArgument('job', InputArgument::OPTIONAL, 'Run only this job (if enabled)'); | 
                                                        |
| 21 | 21 | }  | 
                                                        
| 22 | 22 | |
| 23 | 23 | protected function execute(InputInterface $input, OutputInterface $output)  | 
                                                        
@@ -52,7 +52,7 @@ discard block  | 
                                                    ||
| 52 | 52 | $em->flush();  | 
                                                        
| 53 | 53 | |
| 54 | 54 | $end = microtime(true);  | 
                                                        
| 55 | -        $duration = sprintf('%0.2f', $end - $start); | 
                                                        |
| 55 | +        $duration = sprintf('%0.2f', $end-$start); | 
                                                        |
| 56 | 56 |          $output->writeln("Cron run completed in $duration seconds"); | 
                                                        
| 57 | 57 | }  | 
                                                        
| 58 | 58 | |
@@ -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($job, $jobEnd - $jobStart, $jobOutput->getOutput(), $returnCode);  | 
                                                        |
| 107 | + $this->recordJobResult($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();  | 
                                                        
@@ -16,9 +16,9 @@  | 
                                                    ||
| 16 | 16 | protected function configure()  | 
                                                        
| 17 | 17 |      { | 
                                                        
| 18 | 18 |          $this->setName('cron:scan') | 
                                                        
| 19 | -             ->setDescription('Scans for any new or deleted cron jobs') | 
                                                        |
| 20 | -             ->addOption('keep-deleted', 'k', InputOption::VALUE_NONE, 'If set, deleted cron jobs will not be removed') | 
                                                        |
| 21 | -             ->addOption('default-disabled', 'd', InputOption::VALUE_NONE, 'If set, new jobs will be disabled by default'); | 
                                                        |
| 19 | +                ->setDescription('Scans for any new or deleted cron jobs') | 
                                                        |
| 20 | +                ->addOption('keep-deleted', 'k', InputOption::VALUE_NONE, 'If set, deleted cron jobs will not be removed') | 
                                                        |
| 21 | +                ->addOption('default-disabled', 'd', InputOption::VALUE_NONE, 'If set, new jobs will be disabled by default'); | 
                                                        |
| 22 | 22 | }  | 
                                                        
| 23 | 23 | |
| 24 | 24 | protected function execute(InputInterface $input, OutputInterface $output)  | 
                                                        
@@ -9,7 +9,6 @@  | 
                                                    ||
| 9 | 9 | use Symfony\Component\Console\Input\InputInterface;  | 
                                                        
| 10 | 10 | use Symfony\Component\Console\Input\InputOption;  | 
                                                        
| 11 | 11 | use Symfony\Component\Console\Output\OutputInterface;  | 
                                                        
| 12 | -use Alpixel\Bundle\CronBundle\Entity\CronJobResult;  | 
                                                        |
| 13 | 12 | |
| 14 | 13 | class CronScanCommand extends ContainerAwareCommand  | 
                                                        
| 15 | 14 |  { | 
                                                        
@@ -165,7 +165,7 @@ discard block  | 
                                                    ||
| 165 | 165 | /**  | 
                                                        
| 166 | 166 | * Add results.  | 
                                                        
| 167 | 167 | *  | 
                                                        
| 168 | - * @param Alpixel\Bundle\CronBundle\Entity\CronJobResult $results  | 
                                                        |
| 168 | + * @param CronJobResult $results  | 
                                                        |
| 169 | 169 | */  | 
                                                        
| 170 | 170 | public function addCronJobResult(\Alpixel\Bundle\CronBundle\Entity\CronJobResult $results)  | 
                                                        
| 171 | 171 |      { | 
                                                        
@@ -185,7 +185,7 @@ discard block  | 
                                                    ||
| 185 | 185 | /**  | 
                                                        
| 186 | 186 | * Set mostRecentRun.  | 
                                                        
| 187 | 187 | *  | 
                                                        
| 188 | - * @param Alpixel\Bundle\CronBundle\Entity\CronJobResult $mostRecentRun  | 
                                                        |
| 188 | + * @param CronJobResult $mostRecentRun  | 
                                                        |
| 189 | 189 | */  | 
                                                        
| 190 | 190 | public function setMostRecentRun(\Alpixel\Bundle\CronBundle\Entity\CronJobResult $mostRecentRun)  | 
                                                        
| 191 | 191 |      { | 
                                                        
@@ -195,7 +195,7 @@ discard block  | 
                                                    ||
| 195 | 195 | /**  | 
                                                        
| 196 | 196 | * Get mostRecentRun.  | 
                                                        
| 197 | 197 | *  | 
                                                        
| 198 | - * @return Alpixel\Bundle\CronBundle\Entity\CronJobResult  | 
                                                        |
| 198 | + * @return CronJobResult  | 
                                                        |
| 199 | 199 | */  | 
                                                        
| 200 | 200 | public function getMostRecentRun()  | 
                                                        
| 201 | 201 |      { |