rpalladino / scheduler-api

src/Infrastructure/DBAL/DbalMapper.php

Doc Comments +7 added lines

@@ -16,6 +16,9 @@ 
         $this->loadedMap = [];
     }
 
+    /**
+     * @param integer $id
+     */
     protected function abstractFind($id)
     {
         if (isset($this->loadedMap[$id])) {
@@ -96,6 +99,10 @@ 
         return $result;
     }
 
+    /**
+     * @param string $name
+     * @param integer $value
+     */
     protected function setPropertyValue($subject, $name, $value)
     {
         $reflector = new \ReflectionClass($subject);

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
         $findStatement->execute();
         $resultSet = $findStatement->fetch();
 
-        if (! $resultSet) {
+        if (!$resultSet) {
             return null;
         }
 

src/Infrastructure/DBAL/DbalShiftMapper.php

Unused Use Statements -2 removed lines

@@ -5,13 +5,11 @@
 use DateTimeImmutable;
 use DateTimeInterface;
 use Doctrine\DBAL\Connection;
-use Doctrine\DBAL\Driver\PDOStatement;
 use Doctrine\DBAL\Statement;
 use Doctrine\DBAL\Types\Type;
 use Scheduler\Domain\Model\Shift\Shift;
 use Scheduler\Domain\Model\Shift\ShiftMapper;
 use Scheduler\Domain\Model\User\NullUser;
-use Scheduler\Domain\Model\User\User;
 use Scheduler\Domain\Model\User\UserMapper;
 
 class DbalShiftMapper extends DbalMapper implements ShiftMapper

Indentation +1 added lines, -1 removed lines

@@ -176,7 +176,7 @@
                 "phone" => $resultSet["e_phone"],
                 "created_at" => $resultSet["e_created_at"],
                 "updated_at" => $resultSet["e_updated_at"]
-              ])
+                ])
             : new NullUser();
 
         return new Shift($id, $manager, $employee, $break, $start, $end, $created, $updated);

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
 
     protected static function findInTimePeriodByEmployeeIdStatement()
     {
-        return self::findInTimePeriodStatement() . " and s.employee_id = :employee_id";
+        return self::findInTimePeriodStatement()." and s.employee_id = :employee_id";
     }
 
     protected static function findOpenShiftsStatement()
@@ -103,7 +103,7 @@ 
 
     public function find($id)
     {
-        if (! is_int($id)) {
+        if (!is_int($id)) {
             throw new \InvalidArgumentException("The id must be an integer");
         }
 

src/Infrastructure/DBAL/DbalUserMapper.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 
     public function find($id)
     {
-        if (! is_int($id)) {
+        if (!is_int($id)) {
             throw new \InvalidArgumentException("The id must be an integer");
         }
 

src/Web/Resource/ShiftResource.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
         ];
 
         if ($shift->hasCoworkers()) {
-            $resource['coworkers'] = array_map(function (User $coworker) {
+            $resource['coworkers'] = array_map(function(User $coworker) {
                 return $this->userResource->transform($coworker);
             }, $shift->getCoworkers());
         }
@@ -42,7 +42,7 @@ 
     public function collection(array $shifts)
     {
         return [
-            "shifts" => array_map(function (Shift $shift) {
+            "shifts" => array_map(function(Shift $shift) {
                 return $this->transform($shift);
             }, $shifts)
         ];

src/Web/Radar/Responder/ShiftResponder.php

Indentation +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
         $url = "/shifts/{$shift->getId()}";
 
         $this->response = $this->response->withStatus(201)
-                                         ->withHeader("Location", $url);
+                                            ->withHeader("Location", $url);
         $this->jsonBody($this->resource->item($shift));
     }
 

src/Web/Radar/Responder/ApiProblemResponder.php

Indentation +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     protected function notAuthenticated()
     {
         $this->response = $this->response->withStatus(401)
-                                         ->withHeader("WWW-Authenticate", "None");
+                                            ->withHeader("WWW-Authenticate", "None");
 
         $problem = $this->createProblem();
         $problem->setDetail("This resource requires a valid access token.");

src/Web/Radar/Config/RoutesConfig.php

Spacing +2 added lines, -2 removed lines

@@ -38,10 +38,10 @@
         $adr = $di->get('radar/adr:adr');
         $adr->input(Input\Input::class);
 
-        $adr->get('entry', "/", function ($user) {
+        $adr->get('entry', "/", function($user) {
             $payload = new \Aura\Payload\Payload();
 
-            if (! $user->isAuthenticated()) {
+            if (!$user->isAuthenticated()) {
                 return $payload->setStatus($payload::NOT_AUTHENTICATED);
             }
 

src/Web/Radar/Config/SeedConfig.php

Spacing +3 added lines, -3 removed lines

@@ -14,17 +14,17 @@
         $adr = $di->get('radar/adr:adr');
 
         // For demonstration purposes, seed database using test fixtures
-        $adr->get('seed', "/seed", function () use ($di) {
+        $adr->get('seed', "/seed", function() use ($di) {
             $payload = new Payload();
 
             $di->get('db.schema')->drop();
             $di->get('db.schema')->create();
 
             $yaml = new Parser();
-            $yamlFile = __DIR__ . "/../../../../tests/Infrastructure/DBAL/fixtures.yml";
+            $yamlFile = __DIR__."/../../../../tests/Infrastructure/DBAL/fixtures.yml";
             $fixtures = $yaml->parse(file_get_contents($yamlFile));
 
-            if (! is_array($fixtures)) {
+            if (!is_array($fixtures)) {
                 throw new RuntimeException("Could not parse fixtures file");
             }
 

src/Domain/Model/User/InMemoryAuthenticator.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 
     public function authenticate($token)
     {
-        if (! array_key_exists($token, $this->tokenMap)) {
+        if (!array_key_exists($token, $this->tokenMap)) {
             return new NullUser();
         }