WeareJH / Flexitime

src/JhFlexiTime/Options/NotificationOptions.php

Doc Comments +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
     }
 
     /**
-     * @return DateTime
+     * @return string
      */
     public function getRemindDays()
     {

src/JhFlexiTime/Repository/BookedCreditRepository.php

Unused Use Statements -2 removed lines

@@ -3,10 +3,8 @@
 namespace JhFlexiTime\Repository;
 
 use Doctrine\ORM\QueryBuilder;
-use Doctrine\Common\Persistence\ObjectRepository;
 use Doctrine\ORM\EntityRepository;
 use ZfcUser\Entity\UserInterface;
-use Doctrine\Common\Persistence\ObjectManager;
 use DoctrineORMModule\Paginator\Adapter\DoctrinePaginator;
 use Doctrine\ORM\Tools\Pagination\Paginator as ORMPaginator;
 use Zend\Paginator\Paginator;

src/JhFlexiTime/Repository/Factory/BookedCreditRepositoryFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 {
     /**
      * @param \Zend\ServiceManager\ServiceLocatorInterface $serviceLocator
-     * @return \JhFlexiTime\Repository\BalanceRepository
+     * @return BookedCreditRepository
      */
     public function createService(ServiceLocatorInterface $serviceLocator)
     {

src/JhFlexiTime/Service/BookingService.php

Doc Comments +2 added lines, -2 removed lines

@@ -169,9 +169,9 @@
 
     /**
      * @param $userId
-     * @param DateTime|string $date
+     * @param DateTime $date
      * @throws \Exception
-     * @return object
+     * @return Booking
      */
     public function getBookingByUserAndDate($userId, DateTime $date)
     {

src/JhFlexiTime/Service/Factory/CappedCreditServiceFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 {
     /**
      * @param ServiceLocatorInterface $serviceLocator
-     * @return BookingService
+     * @return CappedCreditService
      */
     public function createService(ServiceLocatorInterface $serviceLocator)
     {

src/JhFlexiTime/Service/Factory/MissingBookingReminderServiceFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 {
     /**
      * @param ServiceLocatorInterface $serviceLocator
-     * @return RunningBalanceService
+     * @return MissingBookingReminderService
      */
     public function createService(ServiceLocatorInterface $serviceLocator)
     {

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use JhFlexiTime\Service\RunningBalanceService;
 use Zend\ServiceManager\FactoryInterface;
 use Zend\ServiceManager\ServiceLocatorInterface;
-use JhFlexiTime\DateTime\DateTime;
 
 /**
  * Class MissingBookingReminderServiceFactory

src/JhFlexiTime/Validator/UniqueObject.php

Doc Comments +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     /**
      * Returns false if there is another object with the same field values but other identifiers.
      *
-     * @param  mixed $value
+     * @param  string $value
      * @param  array $context
      * @return boolean
      */

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Doctrine\Common\Persistence\Mapping\MappingException;
 use DoctrineModule\Validator\UniqueObject as DoctrineUniqueObject;
-use ZfcUser\Entity\UserInterface;
 
 /**
  * Class UniqueObject

test/JhFlexiTimeTest/Controller/BookingControllerTest.php

Doc Comments +15 added lines

@@ -89,6 +89,9 @@ 
         $this->user = $ZfcUserMock;
     }
 
+    /**
+     * @param string $method
+     */
     public function configureMockBookingService($method, array $params, $return)
     {
         $expects = $this->bookingService->expects($this->once())
@@ -98,6 +101,9 @@ 
         call_user_func_array([$expects, "with"], $params);
     }
 
+    /**
+     * @param string $method
+     */
     public function configureMockTimeCalculatorService($method, array $params, $return)
     {
         $expects = $this->timeCalculatorService->expects($this->once())
@@ -107,6 +113,9 @@ 
         call_user_func_array([$expects, "with"], $params);
     }
 
+    /**
+     * @return \JhFlexiTime\Service\BookingService
+     */
     public function getBookingService()
     {
         $mock = $this->getMockBuilder('JhFlexiTime\Service\BookingService')
@@ -118,6 +127,9 @@ 
         return $mock;
     }
 
+    /**
+     * @return \JhFlexiTime\Service\TimeCalculatorService
+     */
     public function getTimeCalculatorService()
     {
         $mock = $this->getMockBuilder('JhFlexiTime\Service\TimeCalculatorService')
@@ -129,6 +141,9 @@ 
         return $mock;
     }
 
+    /**
+     * @return \Zend\Form\FormInterface
+     */
     public function getForm()
     {
         return $this->getMock('Zend\Form\FormInterface');

Unused Use Statements -2 removed lines

@@ -3,13 +3,11 @@
 namespace JhFlexiTimeTest\Controller;
 
 use JhFlexiTime\Controller\BookingRestController;
-
 use JhFlexiTime\DateTime\DateTime;
 use JhFlexiTime\Entity\UserSettings;
 use JhFlexiTime\Repository\UserSettingsRepositoryInterface;
 use JhUser\Entity\User;
 use Zend\Http\Request;
-use Zend\Http\Response;
 use Zend\Mvc\MvcEvent;
 use Zend\Mvc\Router\RouteMatch;
 use Zend\Mvc\Router\Http\TreeRouteStack as HttpRouter;

test/JhFlexiTimeTest/Controller/BookingRestControllerTest.php

Doc Comments +12 added lines

@@ -99,6 +99,9 @@ 
         $this->user = $ZfcUserMock;
     }
 
+    /**
+     * @param string $method
+     */
     public function configureMockBookingService($method, array $params, $return)
     {
         $expects = $this->bookingService->expects($this->once())
@@ -108,6 +111,9 @@ 
         call_user_func_array([$expects, "with"], $params);
     }
 
+    /**
+     * @param string $method
+     */
     public function configureMockTimeCalculatorService($method, array $params, $return)
     {
         $expects = $this->timeCalculatorService->expects($this->once())
@@ -117,6 +123,9 @@ 
         call_user_func_array([$expects, "with"], $params);
     }
 
+    /**
+     * @return \JhFlexiTime\Service\BookingService
+     */
     public function getBookingService()
     {
         $mock = $this->getMockBuilder('JhFlexiTime\Service\BookingService')
@@ -128,6 +137,9 @@ 
         return $mock;
     }
 
+    /**
+     * @return \JhFlexiTime\Service\TimeCalculatorService
+     */
     public function getTimeCalculatorService()
     {
         $mock = $this->getMockBuilder('JhFlexiTime\Service\TimeCalculatorService')

Unused Use Statements -2 removed lines

@@ -3,13 +3,11 @@
 namespace JhFlexiTimeTest\Controller;
 
 use JhFlexiTime\Controller\BookingRestController;
-
 use JhFlexiTime\DateTime\DateTime;
 use JhFlexiTime\Entity\UserSettings;
 use JhFlexiTime\Repository\UserSettingsRepositoryInterface;
 use JhUser\Entity\User;
 use Zend\Http\Request;
-use Zend\Http\Response;
 use Zend\Mvc\MvcEvent;
 use Zend\Mvc\Router\RouteMatch;
 use Zend\Mvc\Router\Http\TreeRouteStack as HttpRouter;