neomerx / json-api

src/Encoder/Encoder.php

Doc Comments +1 added lines, -1 removed lines

@@ -819,7 +819,7 @@
      * @param iterable $iterable1
      * @param iterable $iterable2
      *
-     * @return iterable
+     * @return \Generator
      */
     private function mergeIterables(iterable $iterable1, iterable $iterable2): iterable
     {

Spacing +1 added lines, -1 removed lines

@@ -184,7 +184,7 @@
     {
         assert(
             call_user_func(
-                function (array $paths): bool {
+                function(array $paths): bool {
                     $pathsOk = true;
                     foreach ($paths as $path) {
                         $pathsOk = $pathsOk === true && is_string($path) === true && empty($path) === false;

src/Representation/FieldSetFilter.php

Doc Comments +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
      * @param string   $type
      * @param iterable $fields
      *
-     * @return iterable
+     * @return \Generator
      */
     protected function filterFields(string $type, iterable $fields): iterable
     {

src/Schema/BaseSchema.php

Spacing +4 added lines, -4 removed lines

@@ -73,7 +73,7 @@ 
     {
         // Feel free to override this method to change default URL or add meta
 
-        $url = $this->getSelfSubUrl($resource) . '/' . DocumentInterface::KEYWORD_RELATIONSHIPS . '/' . $name;
+        $url = $this->getSelfSubUrl($resource).'/'.DocumentInterface::KEYWORD_RELATIONSHIPS.'/'.$name;
 
         return $this->getFactory()->createLink(true, $url, false);
     }
@@ -85,7 +85,7 @@ 
     {
         // Feel free to override this method to change default URL or add meta
 
-        $url = $this->getSelfSubUrl($resource) . '/' . $name;
+        $url = $this->getSelfSubUrl($resource).'/'.$name;
 
         return $this->getFactory()->createLink(true, $url, false);
     }
@@ -152,7 +152,7 @@ 
     protected function getResourcesSubUrl(): string
     {
         if ($this->subUrl === null) {
-            $this->subUrl = '/' . $this->getType();
+            $this->subUrl = '/'.$this->getType();
         }
 
         return $this->subUrl;
@@ -165,6 +165,6 @@ 
      */
     protected function getSelfSubUrl($resource): string
     {
-        return $this->getResourcesSubUrl() . '/' . $this->getId($resource);
+        return $this->getResourcesSubUrl().'/'.$this->getId($resource);
     }
 }

src/Schema/ErrorCollection.php

Spacing +11 added lines, -11 removed lines

@@ -566,7 +566,7 @@ 
      */
     protected function getPathToData(): string
     {
-        return '/' . DocumentInterface::KEYWORD_DATA;
+        return '/'.DocumentInterface::KEYWORD_DATA;
     }
 
     /**
@@ -574,7 +574,7 @@ 
      */
     protected function getPathToType(): string
     {
-        return $this->getPathToData() . '/' . DocumentInterface::KEYWORD_TYPE;
+        return $this->getPathToData().'/'.DocumentInterface::KEYWORD_TYPE;
     }
 
     /**
@@ -582,7 +582,7 @@ 
      */
     protected function getPathToId(): string
     {
-        return $this->getPathToData() . '/' . DocumentInterface::KEYWORD_ID;
+        return $this->getPathToData().'/'.DocumentInterface::KEYWORD_ID;
     }
 
     /**
@@ -590,7 +590,7 @@ 
      */
     protected function getPathToAttributes(): string
     {
-        return $this->getPathToData() . '/' . DocumentInterface::KEYWORD_ATTRIBUTES;
+        return $this->getPathToData().'/'.DocumentInterface::KEYWORD_ATTRIBUTES;
     }
 
     /**
@@ -600,7 +600,7 @@ 
      */
     protected function getPathToAttribute(string $name): string
     {
-        return $this->getPathToData() . '/' . DocumentInterface::KEYWORD_ATTRIBUTES . '/' . $name;
+        return $this->getPathToData().'/'.DocumentInterface::KEYWORD_ATTRIBUTES.'/'.$name;
     }
 
     /**
@@ -608,7 +608,7 @@ 
      */
     protected function getPathToRelationships(): string
     {
-        return $this->getPathToData() . '/' . DocumentInterface::KEYWORD_RELATIONSHIPS;
+        return $this->getPathToData().'/'.DocumentInterface::KEYWORD_RELATIONSHIPS;
     }
 
     /**
@@ -618,7 +618,7 @@ 
      */
     protected function getPathToRelationship(string $name): string
     {
-        return $this->getPathToRelationships() . '/' . $name;
+        return $this->getPathToRelationships().'/'.$name;
     }
 
     /**
@@ -628,8 +628,8 @@ 
      */
     protected function getPathToRelationshipType(string $name): string
     {
-        return $this->getPathToRelationship($name) . '/' .
-            DocumentInterface::KEYWORD_DATA . '/' . DocumentInterface::KEYWORD_TYPE;
+        return $this->getPathToRelationship($name).'/'.
+            DocumentInterface::KEYWORD_DATA.'/'.DocumentInterface::KEYWORD_TYPE;
     }
 
     /**
@@ -639,7 +639,7 @@ 
      */
     protected function getPathToRelationshipId(string $name): string
     {
-        return $this->getPathToRelationship($name) . '/' .
-            DocumentInterface::KEYWORD_DATA . '/' . DocumentInterface::KEYWORD_ID;
+        return $this->getPathToRelationship($name).'/'.
+            DocumentInterface::KEYWORD_DATA.'/'.DocumentInterface::KEYWORD_ID;
     }
 }

src/Parser/IdentifierAndResource.php

Spacing +5 added lines, -5 removed lines

@@ -175,15 +175,15 @@ 
         $currentType    = $this->getType();
         $currentPath    = $this->getPosition()->getPath();
         $nextLevel      = $this->getPosition()->getLevel() + 1;
-        $nextPathPrefix = empty($currentPath) === true ? '' : $currentPath . PositionInterface::PATH_SEPARATOR;
+        $nextPathPrefix = empty($currentPath) === true ? '' : $currentPath.PositionInterface::PATH_SEPARATOR;
         foreach ($this->getSchema()->getRelationships($this->getData()) as $name => $description) {
             assert(
                 is_string($name) === true && empty($name) === false,
-                "Relationship names for type `" . $currentType . '` should be non-empty strings.'
+                "Relationship names for type `".$currentType.'` should be non-empty strings.'
             );
             assert(
                 is_array($description) === true && empty($description) === false,
-                "Relationship `$name` for type `" . $currentType . '` should be a non-empty array.'
+                "Relationship `$name` for type `".$currentType.'` should be a non-empty array.'
             );
 
             $hasData = array_key_exists(SchemaInterface::RELATIONSHIP_DATA, $description);
@@ -230,13 +230,13 @@ 
             $hasLinks = $addSelfLink === true || $addRelatedLink === true || empty($schemaLinks) === false;
             assert(
                 $hasData || $hasMeta || $hasLinks,
-                "Relationship `$name` for type `" . $currentType .
+                "Relationship `$name` for type `".$currentType.
                 '` MUST contain at least one of the following: links, data or meta.'
             );
 
             $nextPosition = $this->getFactory()->createPosition(
                 $nextLevel,
-                $nextPathPrefix . $name,
+                $nextPathPrefix.$name,
                 $currentType,
                 $name
             );

src/Parser/Parser.php

Spacing +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
 
     /** @var string */
     public const MSG_CAN_NOT_PARSE_RELATIONSHIP =
-        'For resource of type `%s` with ID `%s` relationship `%s` cannot be parsed because it either ' .
+        'For resource of type `%s` with ID `%s` relationship `%s` cannot be parsed because it either '.
         'has `null` or identifier as data. Skipping.';
 
     /**
@@ -533,7 +533,7 @@ 
         foreach ($paths as $path) {
             $curPath = '';
             foreach (explode($separator, $path) as $pathPart) {
-                $curPath                   = empty($curPath) === true ? $pathPart : $curPath . $separator . $pathPart;
+                $curPath                   = empty($curPath) === true ? $pathPart : $curPath.$separator.$pathPart;
                 $normalizedPaths[$curPath] = true;
             }
         }