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
 {

Spacing +6 added lines, -6 removed lines

@@ -55,7 +55,7 @@ 
                 $cfp->getEventUri()
             ), 400);
         }
-        $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);
 
@@ -88,7 +88,7 @@ 
 
     public function update(Cfp $cfp, $fetchHash)
     {
-        if (! $fetchHash) {
+        if (!$fetchHash) {
             throw new \UnexpectedValueException('No Hash given', 400);
         }
         $oldValues = $this->select($fetchHash);
@@ -117,9 +117,9 @@ 
         ];
 
         foreach ($options as $option) {
-            $method = 'get' . $option;
+            $method = 'get'.$option;
             if ($cfp->$method() != $oldValues->$method()) {
-                $statementElements[] = '`' . $option . '` = :' . $option;
+                $statementElements[] = '`'.$option.'` = :'.$option;
                 $values[$option]     = $cfp->$method();
                 if ($values[$option] instanceof \DateTimeInterface) {
                     $values[$option] = $values[$option]->format('c');
@@ -131,12 +131,12 @@ 
         }
 
         // No values to update, just, return
-        if (! $values) {
+        if (!$values) {
             return $cfp->getHash();
         }
 
         $statement = 'UPDATE `cfp` SET '
-                   . implode(',', $statementElements) . ','
+                   . implode(',', $statementElements).','
                    . '`lastUpdate` = :lastUpdate '
                    . 'WHERE `hash` = :fetchHash';
         $statement = $this->pdo->prepare($statement);

public/index.php

Spacing +6 added lines, -6 removed lines

@@ -27,10 +27,10 @@ 
  * @since     15.01.2016
  * @link      http://github.com/heiglandreas/callingallpapers
  */
-require_once realpath(__DIR__ . '/../vendor/autoload.php');
+require_once realpath(__DIR__.'/../vendor/autoload.php');
 
 $app = new \Slim\App;
-$app->get('/foo', function ($req, $res, $args) {
+$app->get('/foo', function($req, $res, $args) {
     return $res->withHeader(
         'Content-Type',
         'application/json'
@@ -40,17 +40,17 @@ 
 session_start();
 
 // Instantiate the app
-$settings = require __DIR__ . '/../config/settings.php';
+$settings = require __DIR__.'/../config/settings.php';
 $app = new \Slim\App($settings);
 
 // Set up dependencies
-require __DIR__ . '/../config/dependencies.php';
+require __DIR__.'/../config/dependencies.php';
 
 // Register middleware
-require __DIR__ . '/../config/middleware.php';
+require __DIR__.'/../config/middleware.php';
 
 // Register routes
-require __DIR__ . '/../config/routes.php';
+require __DIR__.'/../config/routes.php';
 
 // Run app
 $app->run();
\ No newline at end of file

config/routes.php

Spacing +4 added lines, -4 removed lines

@@ -55,12 +55,12 @@ 
     $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
@@ -76,12 +76,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,
     array $args

config/dependencies.php

Spacing +2 added lines, -2 removed lines

@@ -4,13 +4,13 @@
 $container = $app->getContainer();
 
 // view renderer
-$container['renderer'] = function ($c) {
+$container['renderer'] = function($c) {
     $settings = $c->get('settings')['renderer'];
     return new Slim\Views\PhpRenderer($settings['template_path']);
 };
 
 // monolog
-$container['logger'] = function ($c) {
+$container['logger'] = function($c) {
     $settings = $c->get('settings')['logger'];
     $logger = new Monolog\Logger($settings['name']);
     $logger->pushProcessor(new Monolog\Processor\UidProcessor());

src/Renderer/IcalendarRenderer.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
                 'DTSTAMP' => $lastChange,
                 'UID'     => $cfp['_rel']['cfp_uri'],
                 'DESCRIPTION' => $cfp['description'],
-                'GEO'     => round($cfp['latitude'], 6) . ';' . round($cfp['longitude'], 6),
+                'GEO'     => round($cfp['latitude'], 6).';'.round($cfp['longitude'], 6),
                 'LOCATION' => $cfp['location'],
 
             ]);

src/Service/CfpFactory.php

Spacing +14 added lines, -14 removed lines

@@ -49,7 +49,7 @@ 
         $missingFields = [];
 
         foreach ($requiredFields as $field) {
-            if (! isset($params[$field])) {
+            if (!isset($params[$field])) {
                 $missingFields[] = $field;
             }
         }
@@ -81,7 +81,7 @@ 
 
     public static function setName(Cfp $cfp, array $array)
     {
-        if (! isset($array['name'])) {
+        if (!isset($array['name'])) {
             throw new \InvalidArgumentException('Name has to be specified');
         }
         $cfp->setName(filter_var($array['name'], FILTER_SANITIZE_STRING));
@@ -89,7 +89,7 @@ 
 
     public static function setDateCfpStart(Cfp $cfp, array $array)
     {
-        if (! isset($array['dateCfpStart'])) {
+        if (!isset($array['dateCfpStart'])) {
             throw new \InvalidArgumentException('CFP-StartDate has to be specified');
         }
         $cfp->setDateCfpStart(new DateTimeImmutable($array['dateCfpStart']));
@@ -97,7 +97,7 @@ 
 
     public static function setDateCfpEnd(Cfp $cfp, array $array)
     {
-        if (! isset($array['dateCfpEnd'])) {
+        if (!isset($array['dateCfpEnd'])) {
             throw new \InvalidArgumentException('CFP-EndDate has to be specified');
         }
         $cfp->setDateCfpEnd(new DateTimeImmutable($array['dateCfpEnd']));
@@ -105,7 +105,7 @@ 
 
     public static function setTimezone(Cfp $cfp, array $array)
     {
-        if (! isset($array['timezone'])) {
+        if (!isset($array['timezone'])) {
             throw new \InvalidArgumentException('Timezone has to be specified');
         }
         $cfp->setTimezone(filter_var($array['timezone'], FILTER_SANITIZE_STRING));
@@ -113,7 +113,7 @@ 
 
     public static function setUri(Cfp $cfp, array $array)
     {
-        if (! isset($array['uri'])) {
+        if (!isset($array['uri'])) {
             throw new \InvalidArgumentException('URI has to be specified');
         }
 
@@ -122,7 +122,7 @@ 
 
     public static function setEventUri(Cfp $cfp, array $array)
     {
-        if (! isset($array['eventUri'])) {
+        if (!isset($array['eventUri'])) {
             throw new \InvalidArgumentException('Event-URI has to be specified');
         }
 
@@ -149,7 +149,7 @@ 
 
     public static function setIconUri(Cfp $cfp, array $array)
     {
-        if (! isset($array['iconUri'])) {
+        if (!isset($array['iconUri'])) {
             return;
         }
 
@@ -158,7 +158,7 @@ 
 
     public static function setDescription(Cfp $cfp, array $array)
     {
-        if (! isset($array['description'])) {
+        if (!isset($array['description'])) {
             return;
         }
 
@@ -167,7 +167,7 @@ 
 
     public static function setLocation(Cfp $cfp, array $array)
     {
-        if (! isset($array['location'])) {
+        if (!isset($array['location'])) {
             return;
         }
 
@@ -176,11 +176,11 @@ 
 
     public static function setGeolocation(Cfp $cfp, array $array)
     {
-        if (! isset($array['latitude'])) {
+        if (!isset($array['latitude'])) {
             return;
         }
 
-        if (! isset($array['longitude'])) {
+        if (!isset($array['longitude'])) {
             return;
         }
 
@@ -213,11 +213,11 @@ 
 
     public static function setTags(Cfp $cfp, array $array)
     {
-        if (! isset($array['tags'])) {
+        if (!isset($array['tags'])) {
             return;
         }
 
-        $cfp->setTags(array_map(function ($item) {
+        $cfp->setTags(array_map(function($item) {
             return filter_var($item, FILTER_SANITIZE_STRING);
         }, $array['tags']));
     }

src/Entity/Cfp.php

Spacing +1 added lines, -1 removed lines

@@ -241,7 +241,7 @@
 
     public function setTimezone($timezone)
     {
-        if (! $timezone instanceof \DateTimeZone) {
+        if (!$timezone instanceof \DateTimeZone) {
             $timezone = new \DateTimeZone($timezone);
         }
 

src/Entity/CfpListMapper.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
                 'tags'           => $cfp->getTags(),
                 'lastChange'     => $cfp->getLastUdated()->format('c'),
                 '_rel'            => [
-                    'cfp_uri'    => 'v1/cfp/' . $cfp->getHash(),
+                    'cfp_uri'    => 'v1/cfp/'.$cfp->getHash(),
                 ]
             ];
 

src/Entity/CfpList.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 
     public function add(Cfp $cfp)
     {
-        if (! $this->has($cfp)) {
+        if (!$this->has($cfp)) {
             $this->cfps[] = $cfp;
         }
     }