Add phpcs to travis build process, fix cs

.travis.yml

@@ -14,7 +14,8 @@ before_script:
   - mkdir -p build/logs
 
 script:
-  -  ./vendor/bin/phpunit --coverage-clover ./build/logs/clover.xml
+  - ./vendor/bin/phpunit --coverage-clover ./build/logs/clover.xml
+  - ./vendor/bin/phpcs --standard=PSR2 ./src/
 
 after_script:
   - php vendor/bin/coveralls -v

composer.json

@@ -18,6 +18,7 @@
     },
     "require-dev": {
         "phpunit/phpunit": "4.0.*",
+        "squizlabs/php_codesniffer": "1.4.*",
         "satooshi/php-coveralls": "dev-master"
     },
     "repositories": [

src/JhFlexiTime/Controller/BookingAdminController.php

@@ -121,6 +121,4 @@ public function viewAction()
 
         return $viewModel;
     }
-
-
-}
+}

src/JhFlexiTime/Controller/BookingRestController.php

@@ -146,6 +146,4 @@ public function delete($id)
             'weekTotals'    => $this->timeCalculatorService->getWeekTotals($user, $booking->getDate())
         ));
     }
-
-
 }

src/JhFlexiTime/Controller/GetSetDateTrait.php

@@ -4,7 +4,6 @@
 
 use Zend\Validator\Date as DateValidator;
 
-
 /**
  * Class GetSetDateTrait
  * @author Aydin Hassan <aydin@hotmail.co.uk>
@@ -21,9 +20,10 @@ trait GetSetDateTrait
      * @param string $year
      * @return \DateTime
      */
-    public function getDate($month = null, $year = null) {
+    public function getDate($month = null, $year = null)
+    {
 
-        if(!$this->date) {
+        if (!$this->date) {
             $validator  = new DateValidator(array('format' => 'M Y'));
             if ($validator->isValid(sprintf("%s %s", $month, $year))) {
                 $period = new \DateTime(sprintf('last day of %s %s 23:59:59', $month, $year));
@@ -43,4 +43,4 @@ public function setDate(\DateTime $date)
     {
         $this->date = $date;
     }
-}
+}

src/JhFlexiTime/Controller/RunningBalanceCliController.php

@@ -33,8 +33,11 @@ class RunningBalanceCliController extends AbstractActionController
      * @param RunningBalanceService $runningBalanceService
      * @param AdapterInterface $console
      */
-    public function __construct(UserRepositoryInterface $userRepository, RunningBalanceService $runningBalanceService, AdapterInterface $console)
-    {
+    public function __construct(
+        UserRepositoryInterface $userRepository,
+        RunningBalanceService $runningBalanceService,
+        AdapterInterface $console
+    ) {
         $this->userRepository           = $userRepository;
         $this->runningBalanceService    = $runningBalanceService;
         $this->console                  = $console;
@@ -51,7 +54,7 @@ public function reCalcRunningBalanceAction()
         $request    = $this->getRequest();
         $email      = $request->getParam('userEmail');
 
-        if($email) {
+        if ($email) {
             $user = $this->userRepository->findOneByEmail($email);
 
             if (!$user) {
@@ -75,7 +78,10 @@ public function reCalcRunningBalanceAction()
      */
     public function calcPrevMonthBalanceAction()
     {
-        $this->console->writeLine("Calculating Running Balance for all Users for previous month", ColorInterface::GREEN);
+        $this->console->writeLine(
+            "Calculating Running Balance for all Users for previous month",
+            ColorInterface::GREEN
+        );
         $this->runningBalanceService->calculatePreviousMonthBalance();
         $this->console->writeLine("Finished! ", ColorInterface::GREEN);
     }

src/JhFlexiTime/Controller/SettingsController.php

@@ -40,4 +40,4 @@ public function getAction()
             'settings'  => $this->bookingOptions,
         ));
     }
-} 
+}