mapado / rest-client-sdk

src/Mapping/Driver/AnnotationDriver.php

Doc Comments +3 added lines, -3 removed lines

@@ -191,9 +191,9 @@
     /**
      * getPropertyAnnotation
      *
-     * @param mixed $reader
-     * @param mixed $property
-     * @param mixed $classname
+     * @param FileCacheReader $reader
+     * @param \ReflectionProperty $property
+     * @param string $classname
      * @access private
      * @return void
      */

Spacing +2 added lines, -2 removed lines

@@ -76,7 +76,7 @@ 
 
         foreach ($iterator as $file) {
             $sourceFile = $file[0];
-            if (! preg_match('(^phar:)i', $sourceFile)) {
+            if (!preg_match('(^phar:)i', $sourceFile)) {
                 $sourceFile = realpath($sourceFile);
             }
 
@@ -114,7 +114,7 @@ 
     {
         $metadata = $this->getClassMetadataForClassname($classname);
 
-        return $metadata ? [$metadata,] : [];
+        return $metadata ? [$metadata, ] : [];
     }
 
     /**

src/Model/ModelHydrator.php

Doc Comments +4 added lines, -1 removed lines

@@ -79,7 +79,7 @@ 
      * @param array $data
      * @param string $modelName
      * @access public
-     * @return array
+     * @return HydraCollection
      */
     public function hydrateList($data, $modelName)
     {
@@ -92,6 +92,9 @@ 
         return new HydraCollection();
     }
 
+    /**
+     * @param string $modelName
+     */
     public function deserializeAll($data, $modelName)
     {
         $data['hydra:member'] = array_map(

Indentation +7 added lines, -7 removed lines

@@ -33,13 +33,13 @@
     }
 
     /**
-      * convertId
-      *
-      * @param string $id
-      * @param string $modelName
-      * @access public
-      * @return string
-      */
+     * convertId
+     *
+     * @param string $id
+     * @param string $modelName
+     * @access public
+     * @return string
+     */
     public function convertId($id, $modelName)
     {
         // add slash if needed to have a valid hydra id

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
     public function deserializeAll($data, $modelName)
     {
         $data['hydra:member'] = array_map(
-            function ($member) use ($modelName) {
+            function($member) use ($modelName) {
                 return $this->deserialize($member, $modelName);
             },
             $data['hydra:member']

Tests/Units/Mapping.php

Doc Comments +1 added lines, -1 removed lines

@@ -202,7 +202,7 @@
      * getMappingArray
      *
      * @access private
-     * @return void
+     * @return ClassMetadata[]
      */
     private function getMappingArray()
     {

Spacing +7 added lines, -7 removed lines

@@ -24,7 +24,7 @@ 
             ->given($this->newTestedInstance)
                 ->and($this->testedInstance->setMapping([new ClassMetadata('foo', null, null)]))
             ->then($testedInstance = $this->testedInstance)
-            ->exception(function () use ($testedInstance) {
+            ->exception(function() use ($testedInstance) {
                 @$testedInstance->getModelName();
             })
                 ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')
@@ -34,7 +34,7 @@ 
             ->given($this->newTestedInstance)
                 ->and($this->testedInstance->setMapping([new ClassMetadata('foo', null, null)]))
             ->then($testedInstance = $this->testedInstance)
-            ->exception(function () use ($testedInstance) {
+            ->exception(function() use ($testedInstance) {
                 $testedInstance->getModelName('orders');
             })
                 ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')
@@ -44,7 +44,7 @@ 
             ->given($this->newTestedInstance)
             ->and($this->testedInstance->setMapping([new ClassMetadata('orders', null, null)]))
             ->then($testedInstance = $this->testedInstance)
-            ->exception(function () use ($testedInstance) {
+            ->exception(function() use ($testedInstance) {
                 $testedInstance->getModelName('orders');
             })
                 ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')
@@ -92,7 +92,7 @@ 
             // no mappings
             ->given($this->newTestedInstance)
             ->then($testedInstance = $this->testedInstance)
-                ->exception(function () use ($testedInstance) {
+                ->exception(function() use ($testedInstance) {
                     $testedInstance->getKeyFromId('/orders/8');
                 })
                 ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')
@@ -141,7 +141,7 @@ 
                     ->isEqualTo('order_items')
 
             ->then($testedInstance = $this->testedInstance)
-            ->exception(function () use ($testedInstance) {
+            ->exception(function() use ($testedInstance) {
                 $testedInstance->getKeyFromModel('\Not\Viable\Classname');
             })
                 ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')
@@ -167,7 +167,7 @@ 
                     ->isInstanceOf('Mapado\RestClientSdk\Mapping\ClassMetadata')
 
             ->then
-            ->exception(function () use ($testedInstance) {
+            ->exception(function() use ($testedInstance) {
                 $testedInstance->getClassMetadata('Foo\Bar');
             })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')
@@ -224,6 +224,6 @@ 
             'Foo\Bar\Client\ClientClient'
         );
 
-        return [ $order, $orderItem, $client, ];
+        return [$order, $orderItem, $client, ];
     }
 }

Tests/Model/Cart.php

Spacing +1 added lines, -1 removed lines

@@ -154,7 +154,7 @@
 
     public function addCartItemList($cartItem)
     {
-        $this->cartItemList[] =  $cartItem;
+        $this->cartItemList[] = $cartItem;
     }
 
     /**

Tests/Units/RestClient.php

Indentation +2 added lines, -2 removed lines

@@ -71,7 +71,7 @@ 
             ->then
                 ->object($this->testedInstance->get('/not-json'))
                     ->isInstanceOf('\GuzzleHttp\Psr7\Response')
-           ;
+            ;
     }
 
     /**
@@ -110,7 +110,7 @@ 
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestException')
                     ->hasMessage('Error while deleting resource')
                     ->hasCode(2)
-           ;
+            ;
     }
 
     /**

Spacing +9 added lines, -9 removed lines

@@ -60,7 +60,7 @@ 
                     ->isNull()
 
             ->then($testedInstance = $this->testedInstance)
-                ->exception(function () use ($testedInstance) {
+                ->exception(function() use ($testedInstance) {
                     $testedInstance->get('/error');
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestException')
@@ -104,7 +104,7 @@ 
                     ->isNull()
 
             ->then($testedInstance = $this->testedInstance)
-                ->exception(function () use ($testedInstance) {
+                ->exception(function() use ($testedInstance) {
                     $testedInstance->delete('/error');
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestException')
@@ -149,7 +149,7 @@ 
                     ->string($data['activityUuid'])
                         ->isEqualTo('63d108be-629c-11e5-b5ce-f153631dac50')
             ->then($testedInstance = $this->testedInstance)
-                ->exception(function () use ($testedInstance, $params) {
+                ->exception(function() use ($testedInstance, $params) {
                     $testedInstance->post('/not-found', $params);
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestClientException')
@@ -157,7 +157,7 @@ 
                     ->hasCode(3)
 
             ->then
-                ->exception(function () use ($testedInstance, $params) {
+                ->exception(function() use ($testedInstance, $params) {
                     $testedInstance->post('/error', $params);
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestException')
@@ -202,7 +202,7 @@ 
                     ->string($data['activityUuid'])
                         ->isEqualTo('a9e82f60-629e-11e5-8903-0d978bd11e5d')
             ->then($testedInstance = $this->testedInstance)
-                ->exception(function () use ($testedInstance, $params) {
+                ->exception(function() use ($testedInstance, $params) {
                     $testedInstance->put('/not-found', $params);
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestClientException')
@@ -210,7 +210,7 @@ 
                     ->hasCode(5)
 
             ->then
-                ->exception(function () use ($testedInstance, $params) {
+                ->exception(function() use ($testedInstance, $params) {
                     $testedInstance->put('/error', $params);
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestException')
@@ -275,7 +275,7 @@ 
 
             ->if($testedInstance = $this->testedInstance)
             ->and
-                ->exception(function () use ($testedInstance) {
+                ->exception(function() use ($testedInstance) {
                     $testedInstance->get('/');
                 })
             ->then
@@ -290,13 +290,13 @@ 
             [
                 new Response(
                     200,
-                    [ 'Content-Type' => 'application/ld+json' ],
+                    ['Content-Type' => 'application/ld+json'],
                     file_get_contents(__DIR__ . '/../data/ticketing.list.no_result.json')
                 ),
 
                 new Response(
                     200,
-                    [ 'Content-Type' => 'application/ld+json' ],
+                    ['Content-Type' => 'application/ld+json'],
                     file_get_contents(__DIR__ . '/../data/ticketing.list.no_result.json')
                 ),
             ]

Tests/Units/SdkClient.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
                 ->object($testedInstance->getRepository('orders'))
                     ->isInstanceOf('Mapado\RestClientSdk\Tests\Model\ModelRepository')
 
-                ->exception(function () use ($testedInstance) {
+                ->exception(function() use ($testedInstance) {
                     $testedInstance->getRepository('foo');
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')

src/RestClient.php

Spacing +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
     public function __construct(ClientInterface $httpClient, $baseUrl = null)
     {
         $this->httpClient = $httpClient;
-        $this->baseUrl      = substr($baseUrl, -1) === '/' ? substr($baseUrl, 0, -1) : $baseUrl;
+        $this->baseUrl = substr($baseUrl, -1) === '/' ? substr($baseUrl, 0, -1) : $baseUrl;
 
         $this->logHistory = false;
         $this->requestHistory = [];
@@ -188,7 +188,7 @@ 
         $headers = $response->getHeaders();
         $jsonContentTypeList = ['application/ld+json', 'application/json'];
 
-        $requestIsJson  = false;
+        $requestIsJson = false;
 
         if (isset($headers['Content-Type'])) {
             foreach ($jsonContentTypeList as $contentType) {

Doc Comments +3 added lines, -3 removed lines

@@ -208,9 +208,9 @@
     /**
      * logRequest
      *
-     * @param mixed $startTime
-     * @param mixed $method
-     * @param mixed $url
+     * @param double|null $startTime
+     * @param string $method
+     * @param string $url
      * @param mixed $parameters
      * @param mixed $response
      * @access private

src/Mapping.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
     public function getMappingKeys()
     {
         return array_map(
-            function ($mapping) {
+            function($mapping) {
                 return $mapping->getKey();
             },
             $this->mapping

src/Collection/HydraPaginatedCollection.php

Indentation +2 added lines, -2 removed lines

@@ -15,12 +15,12 @@
     private $firstPage = null;
 
     /**
-    * @var string URI of the last page
+     * @var string URI of the last page
      */
     private $lastPage = null;
 
     /**
-    * @var string URI of the next page
+     * @var string URI of the next page
      */
     private $nextPage = null;