fathomminds / php-rest-models

src/Database/DynamoDb/Resource.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use Aws\DynamoDb\Marshaler;
 use Fathomminds\Rest\Exceptions\RestException;
 use Fathomminds\Rest\Contracts\IResource;
-use Fathomminds\Rest\Helpers\Uuid;
 use Aws\DynamoDb\Exception\DynamoDbException;
 
 class Resource implements IResource

Spacing +2 added lines, -2 removed lines

@@ -142,7 +142,7 @@ 
         } catch (DynamoDbException $ex) {
             $this->throwAwsPutError($ex);
         } catch (\Exception $ex) {
-            throw new RestException($ex->getMessage(), ['result'=>empty($res)?null:$res]);
+            throw new RestException($ex->getMessage(), ['result'=>empty($res) ? null : $res]);
         }
         return $newResource;
     }
@@ -165,7 +165,7 @@ 
             $res = $this->client->deleteItem($query);
             return $resourceId;
         } catch (\Exception $ex) {
-            throw new RestException($ex->getMessage(), ['result'=>empty($res)?null:$res]);
+            throw new RestException($ex->getMessage(), ['result'=>empty($res) ? null : $res]);
         }
     }
 

Doc Comments +6 added lines

@@ -85,6 +85,9 @@ 
         return $query;
     }
 
+    /**
+     * @param DynamoDbException $exception
+     */
     protected function throwAwsPostError($exception)
     {
         switch ($exception->getAwsErrorCode()) {
@@ -121,6 +124,9 @@ 
         return $query;
     }
 
+    /**
+     * @param DynamoDbException $exception
+     */
     protected function throwAwsPutError($exception)
     {
         switch ($exception->getAwsErrorCode()) {

src/Database/Clusterpoint/RestObject.php

Spacing +2 added lines, -3 removed lines

@@ -26,8 +26,7 @@ 
         if ((int)$res->hits() > 0) {
             $results = json_decode($res->rawResponse())->results;
             $message = $results[0]->{$this->primaryKey} === $this->getPrimaryKeyValue() ?
-                'Primary key collision' :
-                'Unique constraint violation';
+                'Primary key collision' : 'Unique constraint violation';
             throw new RestException(
                 $message,
                 ['resourceName' => $this->resourceName, 'confilct' => $results[0]]
@@ -38,7 +37,7 @@ 
     private function getUniqueFieldQuery($query, $uniqueFields)
     {
         // @codeCoverageIgnoreStart
-        $query->where(function ($query) use ($uniqueFields) {
+        $query->where(function($query) use ($uniqueFields) {
             foreach ($uniqueFields as $fieldName) {
                 if (property_exists($this->resource, $fieldName)) {
                     $query->orWhere($fieldName, '==', $this->resource->{$fieldName});

src/Database/Clusterpoint/Resource.php

Spacing +5 added lines, -7 removed lines

@@ -52,11 +52,10 @@ 
             $this->failOnError($res);
             $object = $this->toObject($res);
             $newResource->{$this->primaryKey} = empty($object->{$this->primaryKey}) ?
-                null :
-                $object->{$this->primaryKey};
+                null : $object->{$this->primaryKey};
             return $newResource;
         } catch (\Exception $ex) {
-            throw new RestException($ex->getMessage(), ['result'=>empty($res)?null:$res]);
+            throw new RestException($ex->getMessage(), ['result'=>empty($res) ? null : $res]);
         }
     }
 
@@ -67,7 +66,7 @@ 
             $this->failOnError($res);
             return $this->toObject($res);
         } catch (\Exception $ex) {
-            throw new RestException($ex->getMessage(), ['result'=>empty($res)?null:$res]);
+            throw new RestException($ex->getMessage(), ['result'=>empty($res) ? null : $res]);
         }
     }
 
@@ -78,7 +77,7 @@ 
             $this->failOnError($res);
             return $this->toObject($res);
         } catch (\Exception $ex) {
-            throw new RestException($ex->getMessage(), ['result'=>empty($res)?null:$res]);
+            throw new RestException($ex->getMessage(), ['result'=>empty($res) ? null : $res]);
         }
     }
 
@@ -88,8 +87,7 @@ 
             return;
         }
         $message = $res->error()[0]->message === 'Requested document does not exist' ?
-            'Resource does not exist' :
-            'Database operation failed';
+            'Resource does not exist' : 'Database operation failed';
         throw new RestException(
             $message,
             [

examples/Clusterpoint/Models/Schema/FooSchema.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 
     public function __construct()
     {
-        $this->setDefault('_id', function () {
+        $this->setDefault('_id', function() {
             return (new Uuid)->generate();
         });
     }

bootstrap.php

Spacing +5 added lines, -5 removed lines

@@ -1,12 +1,12 @@
 <?php
 include_once  __DIR__ . '/vendor/autoload.php';
-if (!is_file(__DIR__. '/clusterpoint.php')) {
-    copy(__DIR__. '/clusterpoint-example.php', __DIR__. '/clusterpoint.php');
+if (!is_file(__DIR__ . '/clusterpoint.php')) {
+    copy(__DIR__ . '/clusterpoint-example.php', __DIR__ . '/clusterpoint.php');
 }
-if (!is_file(__DIR__. '/.env')) {
-    copy(__DIR__. '/.env-example', __DIR__. '/.env');
+if (!is_file(__DIR__ . '/.env')) {
+    copy(__DIR__ . '/.env-example', __DIR__ . '/.env');
 }
-if (is_file(__DIR__. '/.env')) {
+if (is_file(__DIR__ . '/.env')) {
     $dotenv = new Dotenv\Dotenv(__DIR__);
     $dotenv->load();
 }

clusterpoint-example.php

Indentation +5 added lines, -5 removed lines

@@ -20,11 +20,11 @@
 
 return [
     'default' => [
-      'host' => getenv('CLUSTERPOINT_HOST'),
-      'account_id' => getenv('CLUSTERPOINT_ACCOUNT_ID'),
-      'username' => getenv('CLUSTERPOINT_USERNAME'),
-      'password' => getenv('CLUSTERPOINT_PASSWORD'),
-      'debug' => getenv('CLUSTERPOINT_DEBUG_MODE'),
+        'host' => getenv('CLUSTERPOINT_HOST'),
+        'account_id' => getenv('CLUSTERPOINT_ACCOUNT_ID'),
+        'username' => getenv('CLUSTERPOINT_USERNAME'),
+        'password' => getenv('CLUSTERPOINT_PASSWORD'),
+        'debug' => getenv('CLUSTERPOINT_DEBUG_MODE'),
     ],
     'development' => [
         'host' => 'https://api-eu.clusterpoint.com/v4/',

src/Database/DynamoDb/RestObject.php

Spacing +9 added lines, -9 removed lines

@@ -34,7 +34,7 @@ 
         $indexedFields = array_intersect($uniqueFields, $indexes); //Is unique and value is set and is indexed
         sort($uniqueFields);
         sort($indexedFields);
-        return $uniqueFields===$indexedFields;
+        return $uniqueFields === $indexedFields;
     }
 
     protected function scanUniqueFields($fields)
@@ -60,7 +60,7 @@ 
         $filter = $this->generateScanFilter($fields);
         if (property_exists($this->resource, $this->primaryKey)) {
             $marshaler = new Marshaler;
-            $filter['FilterExpression'] = '('.$filter['FilterExpression'].') AND #pk<>:pk';
+            $filter['FilterExpression'] = '(' . $filter['FilterExpression'] . ') AND #pk<>:pk';
             $filter['ExpressionAttributeNames']['#pk'] = $this->primaryKey;
             $filter['ExpressionAttributeValues'][':pk'] = $marshaler->marshalValue(
                 $this->resource->{$this->primaryKey}
@@ -84,9 +84,9 @@ 
         ];
         foreach ($fields as $field) {
             if (property_exists($this->resource, $field)) {
-                $ret['FilterExpression'] .= '#'.$field.'=:'.$field.' OR ';
-                $ret['ExpressionAttributeNames']['#'.$field] = $field;
-                $ret['ExpressionAttributeValues'][':'.$field] = $marshaler->marshalValue(
+                $ret['FilterExpression'] .= '#' . $field . '=:' . $field . ' OR ';
+                $ret['ExpressionAttributeNames']['#' . $field] = $field;
+                $ret['ExpressionAttributeValues'][':' . $field] = $marshaler->marshalValue(
                     $this->resource->{$field}
                 );
             }
@@ -130,10 +130,10 @@ 
         $marshaler = new Marshaler;
         $query = [
             'TableName' => $this->database->getDatabaseName() . '-' . $this->resourceName,
-            'KeyConditionExpression' => '#'.$field.'=:'.$field,
+            'KeyConditionExpression' => '#' . $field . '=:' . $field,
             'IndexName' => $this->indexNames[$field],
-            'ExpressionAttributeNames' => ['#'.$field => $field],
-            'ExpressionAttributeValues' => [':'.$field =>  $marshaler->marshalValue(
+            'ExpressionAttributeNames' => ['#' . $field => $field],
+            'ExpressionAttributeValues' => [':' . $field =>  $marshaler->marshalValue(
                 $this->resource->{$field}
             )],
         ];
@@ -160,7 +160,7 @@ 
     protected function generatePromise($client, $request)
     {
         $promise = new Promise(
-            function () use (&$promise, $client, $request) {
+            function() use (&$promise, $client, $request) {
                 $query = new Query($client, $request);
                 while ($res = $query->next()) {
                     if ($res['Count'] !== 0) {