alpixel / AlpixelCronBundle

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

Unused Use Statements -7 removed lines

@@ -1,19 +1,12 @@
 <?php
 namespace Alpixel\Bundle\CronBundle\Command;
 use Doctrine\ORM\EntityManager;
-
 use Symfony\Component\Console\Input\ArgvInput;
-
 use Alpixel\Bundle\CronBundle\Entity\CronJobResult;
-
 use Alpixel\Bundle\CronBundle\Entity\CronJob;
-
 use Symfony\Component\Console\Output\OutputInterface;
-
 use Symfony\Component\Console\Input\InputInterface;
-
 use Symfony\Component\Console\Input\InputArgument;
-
 use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
 
 class CronRunCommand extends ContainerAwareCommand

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/Command/CronScanCommand.php

Unused Use Statements -9 removed lines

@@ -1,21 +1,12 @@
 <?php
 namespace Alpixel\Bundle\CronBundle\Command;
 use Alpixel\Bundle\CronBundle\Entity\CronJob;
-
 use Doctrine\ORM\EntityManager;
-
 use Symfony\Component\Console\Command\Command;
-
 use Alpixel\Bundle\CronBundle\Annotation\CronJob as CronJobAnno;
-
-use Symfony\Bundle\DoctrineBundle\Registry;
-
 use Symfony\Component\Console\Output\OutputInterface;
-
 use Symfony\Component\Console\Input\InputInterface;
-
 use Symfony\Component\Console\Input\InputOption;
-
 use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
 
 class CronScanCommand extends ContainerAwareCommand

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/CronStatusCommand.php

Unused Use Statements -5 removed lines

@@ -1,15 +1,10 @@
 <?php
 namespace Alpixel\Bundle\CronBundle\Command;
 use Alpixel\Bundle\CronBundle\Entity\CronJobResult;
-
 use Alpixel\Bundle\CronBundle\Entity\CronJob;
-
 use Symfony\Component\Console\Output\OutputInterface;
-
 use Symfony\Component\Console\Input\InputInterface;
-
 use Symfony\Component\Console\Input\InputArgument;
-
 use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
 
 class CronStatusCommand extends ContainerAwareCommand

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/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();
         }