joindin / callingallpapers-api

src/PersistenceLayer/CfpPersistenceLayer.php

Unused Use Statements -1 removed lines

@@ -28,7 +28,6 @@
 namespace Callingallpapers\Api\PersistenceLayer;
 
 use Callingallpapers\Api\Entity\Cfp;
-use Monolog\Logger;
 
 class CfpPersistenceLayer
 {

Indentation +6 added lines, -6 removed lines

@@ -67,7 +67,7 @@ 
             $cfp->setDateCfpStart(new \DateTimeImmutable());
         }
         $statement = 'INSERT into `cfp`(`dateCfpStart`, `dateCfpEnd`, `dateEventStart`, `dateEventEnd`, `name`, `uri`, `hash`, `timezone`, `description`, `eventUri`, `iconUri`, `latitude`, `longitude`, `location`, `tags`, `lastUpdate`) ' .
-                     'VALUES (:dateCfpStart, :dateCfpEnd, :dateEventStart, :dateEventEnd, :name, :uri, :hash, :timezone, :description, :eventUri, :iconUri, :latitude, :longitude, :location, :tags, :lastUpdate);';
+                        'VALUES (:dateCfpStart, :dateCfpEnd, :dateEventStart, :dateEventEnd, :name, :uri, :hash, :timezone, :description, :eventUri, :iconUri, :latitude, :longitude, :location, :tags, :lastUpdate);';
         $statement = $this->pdo->prepare($statement);
 
         $values = [
@@ -129,7 +129,7 @@ 
 
         foreach ($options as $option) {
             $method = 'get' . $option;
-           // Merge values from tags and source before comparing!
+            // Merge values from tags and source before comparing!
             if (in_array($option, ['tags', 'source'])) {
                 $setter = 'set' . $option;
                 $cfp->$setter(array_merge($oldValues->$method(), $cfp->$method()));
@@ -158,9 +158,9 @@ 
         }
 
         $statement = 'UPDATE `cfp` SET '
-                   . implode(',', $statementElements) . ','
-                   . '`lastUpdate` = :lastUpdate '
-                   . 'WHERE `hash` = :fetchHash';
+                    . implode(',', $statementElements) . ','
+                    . '`lastUpdate` = :lastUpdate '
+                    . 'WHERE `hash` = :fetchHash';
         $statement = $this->pdo->prepare($statement);
         $values['fetchHash']  = $fetchHash;
         $values['lastUpdate'] = (new \DateTime('now', new \DateTimezone('UTC')))->format('c');
@@ -353,7 +353,7 @@ 
             $cfp->setLocation($item['location']);
             $cfp->setTags(explode(',', $item['tags']));
             $cfp->setLastUpdated(new \DateTimeImmutable($item['lastUpdate']));
-         //   $cfp->setSource(explode(',', $item['source']));
+            //   $cfp->setSource(explode(',', $item['source']));
 
             $list->add($cfp);
         }

Spacing +15 added lines, -15 removed lines

@@ -66,7 +66,7 @@ 
         if ($cfp->getDateCfpStart()->getTimestamp() < 10000) {
             $cfp->setDateCfpStart(new \DateTimeImmutable());
         }
-        $statement = 'INSERT into `cfp`(`dateCfpStart`, `dateCfpEnd`, `dateEventStart`, `dateEventEnd`, `name`, `uri`, `hash`, `timezone`, `description`, `eventUri`, `iconUri`, `latitude`, `longitude`, `location`, `tags`, `lastUpdate`) ' .
+        $statement = 'INSERT into `cfp`(`dateCfpStart`, `dateCfpEnd`, `dateEventStart`, `dateEventEnd`, `name`, `uri`, `hash`, `timezone`, `description`, `eventUri`, `iconUri`, `latitude`, `longitude`, `location`, `tags`, `lastUpdate`) '.
                      'VALUES (:dateCfpStart, :dateCfpEnd, :dateEventStart, :dateEventEnd, :name, :uri, :hash, :timezone, :description, :eventUri, :iconUri, :latitude, :longitude, :location, :tags, :lastUpdate);';
         $statement = $this->pdo->prepare($statement);
 
@@ -98,7 +98,7 @@ 
 
     public function update(Cfp $cfp, $fetchHash)
     {
-        if (! $fetchHash) {
+        if (!$fetchHash) {
             throw new \UnexpectedValueException('No Hash given', 400);
         }
         $oldValues = $this->select($fetchHash);
@@ -128,17 +128,17 @@ 
         ];
 
         foreach ($options as $option) {
-            $method = 'get' . $option;
+            $method = 'get'.$option;
            // Merge values from tags and source before comparing!
             if (in_array($option, ['tags', 'source'])) {
-                $setter = 'set' . $option;
+                $setter = 'set'.$option;
                 $cfp->$setter(array_merge($oldValues->$method(), $cfp->$method()));
             }
             if ($cfp->$method() !== $oldValues->$method()) {
                 if ($option == 'dateCfpStart' && $cfp->$method()->getTimestamp() < 100000) {
                     continue;
                 }
-                $statementElements[] = '`' . $option . '` = :' . $option;
+                $statementElements[] = '`'.$option.'` = :'.$option;
                 $values[$option]     = $cfp->$method();
                 if ($values[$option] instanceof \DateTimeInterface) {
                     $values[$option] = $values[$option]->format('c');
@@ -158,7 +158,7 @@ 
         }
 
         $statement = 'UPDATE `cfp` SET '
-                   . implode(',', $statementElements) . ','
+                   . implode(',', $statementElements).','
                    . '`lastUpdate` = :lastUpdate '
                    . 'WHERE `hash` = :fetchHash';
         $statement = $this->pdo->prepare($statement);
@@ -294,13 +294,13 @@ 
         $values    = [];
         $where = [];
         foreach ($parameters as $key => $value) {
-            if (! in_array($key, $fields)) {
+            if (!in_array($key, $fields)) {
                 continue;
             }
-            if (! is_array($value)){
+            if (!is_array($value)) {
                 $value = [$value];
             }
-            foreach($value as $itemkey => $item) {
+            foreach ($value as $itemkey => $item) {
                 $compare = '=';
                 if (in_array($key, [
                     'date_cfp_end',
@@ -308,19 +308,19 @@ 
                     'date_event_end',
                     'date_event_start'
                 ])) {
-                    if (array_key_exists($key . '_compare', $parameters) && isset($parameters[$key . '_compare'][$itemkey]) && in_array($parameters[$key . '_compare'][$itemkey], ['=', '<', '>', '<>'])) {
-                        $compare = $parameters[$key . '_compare'][$itemkey];
+                    if (array_key_exists($key.'_compare', $parameters) && isset($parameters[$key.'_compare'][$itemkey]) && in_array($parameters[$key.'_compare'][$itemkey], ['=', '<', '>', '<>'])) {
+                        $compare = $parameters[$key.'_compare'][$itemkey];
                     }
-                    $where[] = 'datetime(`' . $transformer->transform($key) . '`) ' . $compare . ' datetime(:' . $key . '_' . $itemkey . ')';
+                    $where[] = 'datetime(`'.$transformer->transform($key).'`) '.$compare.' datetime(:'.$key.'_'.$itemkey.')';
 //                    $where[] = '`' . $transformer->transform($key) . '` ' . $compare . ' CONVERT_TZ(:' . $key . ', timezone, \'UTC\')';
                     $value = (new DateTime($item))->setTimezone(new \DateTimeZone('UTC'))->format('c');
                 } else {
-                    $where[] = '`' . $transformer->transform($key) . '` ' . $compare . ' :' . $key;
+                    $where[] = '`'.$transformer->transform($key).'` '.$compare.' :'.$key;
                 }
-                $values[$key . '_' . $itemkey] = $value;
+                $values[$key.'_'.$itemkey] = $value;
             }
         }
-        if (! $where) {
+        if (!$where) {
             throw new \UnexpectedValueException('No CFPs found', 404);
         }
 

config/routes.php

Spacing +8 added lines, -8 removed lines

@@ -41,7 +41,7 @@ 
     \Psr\Http\Message\ResponseInterface $response
 ) use ($app){
     $params = $request->getParsedBody();
-    if (! is_array($params)) {
+    if (!is_array($params)) {
         throw new UnexpectedValueException('Expected array');
     }
 
@@ -54,18 +54,18 @@ 
     $cpl->insert($cfp);
 
     $uri = $request->getUri();
-    $uri = $uri->withPath('v1/cfp/' . $cfp->getId());
+    $uri = $uri->withPath('v1/cfp/'.$cfp->getId());
     return $response->withRedirect((string)$uri, 201);
 });
 
 
-$app->put('/v1/cfp/{hash}', function (
+$app->put('/v1/cfp/{hash}', function(
     \Psr\Http\Message\ServerRequestInterface $request,
     \Psr\Http\Message\ResponseInterface $response,
     array $args
 ) use ($app){
     $params = $request->getParsedBody();
-    if (! is_array($params)) {
+    if (!is_array($params)) {
         throw new UnexpectedValueException('Expected array');
     }
 
@@ -78,12 +78,12 @@ 
     $cpl->update($cfp, $args['hash']);
 
     $uri = $request->getUri();
-    $uri = $uri->withPath('v1/cfp/' . $cfp->getId());
+    $uri = $uri->withPath('v1/cfp/'.$cfp->getId());
     return $response->withHeader('Location', (string)$uri)->withStatus(204);
 });
 
 
-$app->delete('/v1/cfp/{id}', function (
+$app->delete('/v1/cfp/{id}', function(
     \Psr\Http\Message\ServerRequestInterface $request,
     \Psr\Http\Message\ResponseInterface $response
 ) use ($app){
@@ -96,7 +96,7 @@ 
     return $response->withHeader('Content-Length', '0')->withStatus(204);
 });
 
-$app->get('/v1/search', function (
+$app->get('/v1/search', function(
     \Psr\Http\Message\ServerRequestInterface $request,
     \Psr\Http\Message\ResponseInterface $response,
     array $args
@@ -110,5 +110,5 @@ 
 
     $cfpMapper = new \Callingallpapers\Api\Entity\CfpListMapper();
 
-    return $this->view->render($response, $cfpMapper->map($list), 200, 'cfp/list.twig');;
+    return $this->view->render($response, $cfpMapper->map($list), 200, 'cfp/list.twig'); ;
 });