Completed
Branch master (288a42)
by Benjamin
04:20
created
src/Alpixel/Bundle/CronBundle/Command/CronStatusCommand.php 1 patch
Braces   +5 added lines, -10 removed lines patch added patch discarded remove patch
@@ -32,14 +32,12 @@  discard block
 block discarded – undo
32 32
             try
33 33
             {
34 34
                 $cronJobs = array($jobRepo->findOneByCommand($jobName));
35
-            }
36
-            catch(\Exception $e)
35
+            } catch(\Exception $e)
37 36
             {
38 37
                 $output->writeln("Couldn't find a job by the name of $jobName");
39 38
                 return CronJobResult::FAILED;
40 39
             }
41
-        }
42
-        else
40
+        } else
43 41
         {
44 42
             $cronJobs = $em->getRepository('CronBundle:CronJob')->findAll();
45 43
         }
@@ -56,16 +54,14 @@  discard block
 block discarded – undo
56 54
             if(!$cronJob->getEnabled())
57 55
             {
58 56
                 $output->writeln("   Not scheduled");
59
-            }
60
-            else
57
+            } else
61 58
             {
62 59
                 $output->write("   Scheduled for: ");
63 60
                 $now = new \DateTime();
64 61
                 if($cronJob->getNextRun() <= $now)
65 62
                 {
66 63
                     $output->writeln("Next run");
67
-                }
68
-                else
64
+                } else
69 65
                 {
70 66
                     $output->writeln(strftime("%c", $cronJob->getNextRun()->getTimestamp()));
71 67
                 }
@@ -86,8 +82,7 @@  discard block
 block discarded – undo
86 82
                         break;
87 83
                 }
88 84
                 $output->writeln("   Last run was: $status");
89
-            }
90
-            else
85
+            } else
91 86
             {
92 87
                 $output->writeln("   This job has not yet been run");
93 88
             }
Please login to merge, or discard this patch.
src/Alpixel/Bundle/CronBundle/Command/CronScanCommand.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -68,8 +68,7 @@
 block discarded – undo
68 68
                             $currentJob->setNextRun($newTime);
69 69
                             $output->writeln("Updated interval for $job to {$anno->value}");
70 70
                         }
71
-                    }
72
-                    else
71
+                    } else
73 72
                     {
74 73
                         $this->newJobFound($em, $output, $command, $anno, $defaultDisabled);
75 74
                     }
Please login to merge, or discard this patch.
src/Alpixel/Bundle/CronBundle/Command/CronPruneLogsCommand.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -28,8 +28,7 @@  discard block
 block discarded – undo
28 28
         if($job)
29 29
         {
30 30
             $output->writeln("Pruning logs for cron job $job");
31
-        }
32
-        else
31
+        } else
33 32
         {
34 33
             $output->writeln("Pruning logs for all cron jobs");
35 34
         }
@@ -44,8 +43,7 @@  discard block
 block discarded – undo
44 43
             }
45 44
 
46 45
             $em->getRepository('CronBundle:CronJobResult')->deleteOldLogs($jobObj);
47
-        }
48
-        else
46
+        } else
49 47
         {
50 48
             $em->getRepository('CronBundle:CronJobResult')->deleteOldLogs();
51 49
         }
Please login to merge, or discard this patch.
src/Alpixel/Bundle/CronBundle/Command/CronRunCommand.php 1 patch
Braces   +6 added lines, -12 removed lines patch added patch discarded remove patch
@@ -41,14 +41,12 @@  discard block
 block discarded – undo
41 41
                 {
42 42
                     $jobsToRun = array($jobObj);
43 43
                 }
44
-            }
45
-            catch(\Exception $e)
44
+            } catch(\Exception $e)
46 45
             {
47 46
                 $output->writeln("Couldn't find a job by the name of $jobName");
48 47
                 return CronJobResult::FAILED;
49 48
             }
50
-        }
51
-        else
49
+        } else
52 50
         {
53 51
             $jobsToRun = $jobRepo->findDueTasks();
54 52
         }
@@ -76,8 +74,7 @@  discard block
 block discarded – undo
76 74
         try
77 75
         {
78 76
             $commandToRun = $this->getApplication()->get($job->getCommand());
79
-        }
80
-        catch(InvalidArgumentException $ex)
77
+        } catch(InvalidArgumentException $ex)
81 78
         {
82 79
             $output->writeln(" skipped (command no longer exists)");
83 80
             $this->recordJobResult($em, $job, 0, "Command no longer exists", CronJobResult::SKIPPED);
@@ -93,8 +90,7 @@  discard block
 block discarded – undo
93 90
         try
94 91
         {
95 92
             $returnCode = $commandToRun->execute($emptyInput, $jobOutput);
96
-        }
97
-        catch(\Exception $ex)
93
+        } catch(\Exception $ex)
98 94
         {
99 95
             $returnCode = CronJobResult::FAILED;
100 96
             $jobOutput->writeln("");
@@ -114,12 +110,10 @@  discard block
 block discarded – undo
114 110
         if($returnCode == CronJobResult::SKIPPED)
115 111
         {
116 112
             $statusStr = "skipped";
117
-        }
118
-        elseif($returnCode == CronJobResult::SUCCEEDED)
113
+        } elseif($returnCode == CronJobResult::SUCCEEDED)
119 114
         {
120 115
             $statusStr = "succeeded";
121
-        }
122
-        elseif($returnCode == CronJobResult::FAILED)
116
+        } elseif($returnCode == CronJobResult::FAILED)
123 117
         {
124 118
             $statusStr = "failed";
125 119
         }
Please login to merge, or discard this patch.