alpixel / AlpixelCronBundle

src/Alpixel/Bundle/CronBundle/Command/CronRunCommand.php

Doc Comments +3 added lines

@@ -136,6 +136,9 @@
         $job->setNextRun($newTime);
     }
 
+    /**
+     * @param string $output
+     */
     protected function recordJobResult(EntityManager $em, CronJob $job, $timeTaken, $output, $resultCode)
     {
         // Create a new CronJobResult

Braces +6 added lines, -12 removed lines

@@ -41,14 +41,12 @@ 
                 {
                     $jobsToRun = array($jobObj);
                 }
-            }
-            catch(\Exception $e)
+            } catch(\Exception $e)
             {
                 $output->writeln("Couldn't find a job by the name of $jobName");
                 return CronJobResult::FAILED;
             }
-        }
-        else
+        } else
         {
             $jobsToRun = $jobRepo->findDueTasks();
         }
@@ -76,8 +74,7 @@ 
         try
         {
             $commandToRun = $this->getApplication()->get($job->getCommand());
-        }
-        catch(InvalidArgumentException $ex)
+        } catch(InvalidArgumentException $ex)
         {
             $output->writeln(" skipped (command no longer exists)");
             $this->recordJobResult($em, $job, 0, "Command no longer exists", CronJobResult::SKIPPED);
@@ -93,8 +90,7 @@ 
         try
         {
             $returnCode = $commandToRun->execute($emptyInput, $jobOutput);
-        }
-        catch(\Exception $ex)
+        } catch(\Exception $ex)
         {
             $returnCode = CronJobResult::FAILED;
             $jobOutput->writeln("");
@@ -114,12 +110,10 @@ 
         if($returnCode == CronJobResult::SKIPPED)
         {
             $statusStr = "skipped";
-        }
-        elseif($returnCode == CronJobResult::SUCCEEDED)
+        } elseif($returnCode == CronJobResult::SUCCEEDED)
         {
             $statusStr = "succeeded";
-        }
-        elseif($returnCode == CronJobResult::FAILED)
+        } elseif($returnCode == CronJobResult::FAILED)
         {
             $statusStr = "failed";
         }

src/Alpixel/Bundle/CronBundle/Entity/CronJob.php

Doc Comments +4 added lines, -4 removed lines

@@ -136,7 +136,7 @@ 
     /**
      * Set nextRun
      *
-     * @param datetime $nextRun
+     * @param \DateTime $nextRun
      */
     public function setNextRun($nextRun)
     {
@@ -156,7 +156,7 @@ 
     /**
      * Add results
      *
-     * @param Alpixel\Bundle\CronBundle\Entity\CronJobResult $results
+     * @param CronJobResult $results
      */
     public function addCronJobResult(\Alpixel\Bundle\CronBundle\Entity\CronJobResult $results)
     {
@@ -176,7 +176,7 @@ 
     /**
      * Set mostRecentRun
      *
-     * @param Alpixel\Bundle\CronBundle\Entity\CronJobResult $mostRecentRun
+     * @param CronJobResult $mostRecentRun
      */
     public function setMostRecentRun(\Alpixel\Bundle\CronBundle\Entity\CronJobResult $mostRecentRun)
     {
@@ -186,7 +186,7 @@ 
     /**
      * Get mostRecentRun
      *
-     * @return Alpixel\Bundle\CronBundle\Entity\CronJobResult
+     * @return CronJobResult
      */
     public function getMostRecentRun()
     {

src/Alpixel/Bundle/CronBundle/Entity/CronJobResult.php

Doc Comments +2 added lines, -2 removed lines

@@ -150,7 +150,7 @@ 
     /**
      * Set job
      *
-     * @param Alpixel\Bundle\CronBundle\Entity\CronJob $job
+     * @param CronJob $job
      */
     public function setJob(\Alpixel\Bundle\CronBundle\Entity\CronJob $job)
     {
@@ -160,7 +160,7 @@ 
     /**
      * Get job
      *
-     * @return Alpixel\Bundle\CronBundle\Entity\CronJob
+     * @return CronJob
      */
     public function getJob()
     {

src/Alpixel/Bundle/CronBundle/Command/CronStatusCommand.php

Braces +5 added lines, -10 removed lines

@@ -32,14 +32,12 @@ 
             try
             {
                 $cronJobs = array($jobRepo->findOneByCommand($jobName));
-            }
-            catch(\Exception $e)
+            } catch(\Exception $e)
             {
                 $output->writeln("Couldn't find a job by the name of $jobName");
                 return CronJobResult::FAILED;
             }
-        }
-        else
+        } else
         {
             $cronJobs = $em->getRepository('CronBundle:CronJob')->findAll();
         }
@@ -56,16 +54,14 @@ 
             if(!$cronJob->getEnabled())
             {
                 $output->writeln("   Not scheduled");
-            }
-            else
+            } else
             {
                 $output->write("   Scheduled for: ");
                 $now = new \DateTime();
                 if($cronJob->getNextRun() <= $now)
                 {
                     $output->writeln("Next run");
-                }
-                else
+                } else
                 {
                     $output->writeln(strftime("%c", $cronJob->getNextRun()->getTimestamp()));
                 }
@@ -86,8 +82,7 @@ 
                         break;
                 }
                 $output->writeln("   Last run was: $status");
-            }
-            else
+            } else
             {
                 $output->writeln("   This job has not yet been run");
             }

src/Alpixel/Bundle/CronBundle/Command/CronScanCommand.php

Braces +1 added lines, -2 removed lines

@@ -68,8 +68,7 @@
                             $currentJob->setNextRun($newTime);
                             $output->writeln("Updated interval for $job to {$anno->value}");
                         }
-                    }
-                    else
+                    } else
                     {
                         $this->newJobFound($em, $output, $command, $anno, $defaultDisabled);
                     }

src/Alpixel/Bundle/CronBundle/Command/CronPruneLogsCommand.php

Braces +2 added lines, -4 removed lines

@@ -28,8 +28,7 @@ 
         if($job)
         {
             $output->writeln("Pruning logs for cron job $job");
-        }
-        else
+        } else
         {
             $output->writeln("Pruning logs for all cron jobs");
         }
@@ -44,8 +43,7 @@ 
             }
 
             $em->getRepository('CronBundle:CronJobResult')->deleteOldLogs($jobObj);
-        }
-        else
+        } else
         {
             $em->getRepository('CronBundle:CronJobResult')->deleteOldLogs();
         }