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/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/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;
         }
     }

src/Middleware/OAuth.php

Spacing +2 added lines, -2 removed lines

@@ -68,7 +68,7 @@ 
         }
 
         $bearer = explode(' ', $auth[0]);
-        if (! isset($bearer[1])) {
+        if (!isset($bearer[1])) {
             return $this->getNoAuthResponse($response);
         }
         $bearer = $bearer[1];
@@ -87,7 +87,7 @@ 
 
     protected function getNoAuthResponse($response, $error = 'no token', $description = 'No Access-Token provided')
     {
-        $response = $response->withHeader('WWW-Authenticate', 'Bearer realm="callingallpapers", error="' . $error . '", error_desciption="' . $description . '"');
+        $response = $response->withHeader('WWW-Authenticate', 'Bearer realm="callingallpapers", error="'.$error.'", error_desciption="'.$description.'"');
         $response = $response->withStatus(401);
 
         return $response;

src/Middleware/Renderer.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 
         if (strpos($accept[0], 'text/html') !== false) {
             $container         = $this->app->getContainer();
-            $container['view'] = function ($container) {
+            $container['view'] = function($container) {
                 $config = $container['settings'];
                 $tView  = new \Slim\Views\Twig($config['renderer']['template_path'],
                     [

src/Renderer/RssRenderer.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
             'uri'   => 'http://andreas.heigl.org',
         ]);
         $feed->setDateModified(time());
-        if (! isset($data['cfps'])) {
+        if (!isset($data['cfps'])) {
             $data['cfps'] = [];
         }
         foreach ($data['cfps'] as $cfp) {

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 {
     /**
      *
-     * @param Response $response
+     * @param ResponseInterface $response
      * @param array $data
      * @param int $status
      *

src/Renderer/IcalendarRenderer.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
     public function render(ResponseInterface $response, array $data = [], $status = 200)
     {
         $icalendar = new VCalendar();
-        if (! isset($data['cfps'])) {
+        if (!isset($data['cfps'])) {
             $data['cfps'] = [];
         }
         foreach ($data['cfps'] as $cfp) {
@@ -66,7 +66,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'],
 
             ]);

public/index.php

Spacing +5 added lines, -5 removed lines

@@ -27,22 +27,22 @@
  * @since     15.01.2016
  * @link      http://github.com/heiglandreas/callingallpapers
  */
-require_once realpath(__DIR__ . '/../vendor/autoload.php');
+require_once realpath(__DIR__.'/../vendor/autoload.php');
 
 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();