neomerx / json-api

src/Encoder/Parameters/ParametersAnalyzer.php

Spacing +3 added lines, -4 removed lines

@@ -110,7 +110,7 @@ 
         foreach ($includePaths as $curPath) {
             if ($pathLength === 0) {
                 $relationshipName = $this->getRelationshipNameForTopResource($curPath);
-            } elseif (strpos($curPath, $pathBeginning . DocumentInterface::PATH_SEPARATOR) === 0) {
+            } elseif (strpos($curPath, $pathBeginning.DocumentInterface::PATH_SEPARATOR) === 0) {
                 $relationshipName = $this->getRelationshipNameForResource($curPath, $pathLength);
             } else {
                 $relationshipName = null;
@@ -166,7 +166,7 @@ 
 
         if ($path !== null) {
             foreach ($paths as $targetPath) {
-                if (strpos($targetPath, $path . DocumentInterface::PATH_SEPARATOR) === 0) {
+                if (strpos($targetPath, $path.DocumentInterface::PATH_SEPARATOR) === 0) {
                     $hasMatch = true;
                     break;
                 }
@@ -221,8 +221,7 @@ 
     {
         $nextSeparatorPos = strpos($curPath, DocumentInterface::PATH_SEPARATOR, $pathLength + 1);
         $relationshipName = $nextSeparatorPos === false ?
-            substr($curPath, $pathLength + 1) :
-            substr($curPath, $pathLength + 1, $nextSeparatorPos - $pathLength - 1);
+            substr($curPath, $pathLength + 1) : substr($curPath, $pathLength + 1, $nextSeparatorPos - $pathLength - 1);
 
         return $relationshipName;
     }

src/Encoder/Parser/Parser.php

Doc Comments +1 added lines, -1 removed lines

@@ -160,7 +160,7 @@
     }
 
     /**
-     * @return iterable
+     * @return \Generator
      *
      * @SuppressWarnings(PHPMD.ElseExpression)
      * @SuppressWarnings(PHPMD.CyclomaticComplexity)

Spacing +3 added lines, -4 removed lines

@@ -90,11 +90,11 @@ 
      */
     const MESSAGES = [
         self::MSG_GET_SCHEMA_FAILED_FOR_RESOURCE_AT_ROOT =>
-            'Getting Schema for a top-level resource of type `%s` failed. ' .
+            'Getting Schema for a top-level resource of type `%s` failed. '.
             'Please check you have added a Schema for this type.',
 
         self::MSG_GET_SCHEMA_FAILED_FOR_RESOURCE_AT_PATH =>
-            'Getting Schema for a resource of type `%s` at path `%s` failed. ' .
+            'Getting Schema for a resource of type `%s` at path `%s` failed. '.
             'Please check you have added a Schema for this type.',
     ];
 
@@ -339,8 +339,7 @@ 
     {
         ($data === null || (is_array($data) === true && empty($data) === true)) ?: Exceptions::throwLogicException();
 
-        $replyType = ($data === null ? ParserReplyInterface::REPLY_TYPE_NULL_RESOURCE_STARTED :
-            ParserReplyInterface::REPLY_TYPE_EMPTY_RESOURCE_STARTED);
+        $replyType = ($data === null ? ParserReplyInterface::REPLY_TYPE_NULL_RESOURCE_STARTED : ParserReplyInterface::REPLY_TYPE_EMPTY_RESOURCE_STARTED);
 
         return $this->parserFactory->createEmptyReply($replyType, $this->stack);
     }

src/Http/Query/BaseQueryParserTrait.php

Doc Comments +5 added lines, -5 removed lines

@@ -30,7 +30,7 @@ 
      * @param array  $parameters
      * @param string $errorTitle
      *
-     * @return iterable
+     * @return \Generator
      */
     protected function getIncludes(array $parameters, string $errorTitle): iterable
     {
@@ -53,7 +53,7 @@ 
      * @param array  $parameters
      * @param string $errorTitle
      *
-     * @return iterable
+     * @return \Generator
      */
     protected function getFields(array $parameters, string $errorTitle): iterable
     {
@@ -73,7 +73,7 @@ 
      * @param array  $parameters
      * @param string $errorTitle
      *
-     * @return iterable
+     * @return \Generator
      */
     protected function getSorts(array $parameters, string $errorTitle): iterable
     {
@@ -106,7 +106,7 @@ 
      * @param string|mixed $shouldBeString
      * @param string       $errorTitle
      *
-     * @return iterable
+     * @return \Generator
      */
     private function splitCommaSeparatedStringAndCheckNoEmpties(
         string $paramName,
@@ -122,7 +122,7 @@ 
      * @param string       $separator
      * @param string       $errorTitle
      *
-     * @return iterable
+     * @return \Generator
      */
     private function splitStringAndCheckNoEmpties(
         string $paramName,

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     protected function getIncludes(array $parameters, string $errorTitle): iterable
     {
         if (array_key_exists(P::PARAM_INCLUDE, $parameters) === true) {
-            $splitByDot = function (string $path) use ($errorTitle): iterable {
+            $splitByDot = function(string $path) use ($errorTitle): iterable {
                 foreach ($this->splitStringAndCheckNoEmpties(P::PARAM_INCLUDE, $path, '.', $errorTitle) as $link) {
                     yield $link;
                 }

src/Contracts/Encoder/EncoderInterface.php

Doc Comments +1 added lines, -1 removed lines

@@ -132,7 +132,7 @@
     /**
      * Encode errors as JSON API string.
      *
-     * @param ErrorInterface[]|ErrorCollection|iterable $errors
+     * @param ErrorInterface[] $errors
      *
      * @return string
      */

src/Schema/BaseSchema.php

Spacing +7 added lines, -7 removed lines

@@ -78,16 +78,16 @@ 
     {
         assert(
             is_string($this->getResourceType()) === true && empty($this->getResourceType()) === false,
-            'Resource type is not set for Schema \'' . static::class . '\'.'
+            'Resource type is not set for Schema \''.static::class.'\'.'
         );
 
         if ($this->selfSubUrl === null) {
-            $this->selfSubUrl = '/' . $this->getResourceType();
+            $this->selfSubUrl = '/'.$this->getResourceType();
         } else {
             assert(
                 is_string($this->selfSubUrl) === true && empty($this->selfSubUrl) === false &&
                 $this->selfSubUrl[0] === '/' && $this->selfSubUrl[strlen($this->selfSubUrl) - 1] != '/',
-                '\'Self\' sub-url set incorrectly for Schema \'' . static::class . '\'.'
+                '\'Self\' sub-url set incorrectly for Schema \''.static::class.'\'.'
             );
         }
 
@@ -107,7 +107,7 @@ 
      */
     public function getSelfSubUrl($resource = null): string
     {
-        return $resource === null ? $this->selfSubUrl : $this->selfSubUrl . '/' . $this->getId($resource);
+        return $resource === null ? $this->selfSubUrl : $this->selfSubUrl.'/'.$this->getId($resource);
     }
 
     /**
@@ -272,7 +272,7 @@ 
      */
     protected function getRelationshipSelfUrl($resource, $name)
     {
-        $url = $this->getSelfSubUrl($resource) . '/' . DocumentInterface::KEYWORD_RELATIONSHIPS . '/' . $name;
+        $url = $this->getSelfSubUrl($resource).'/'.DocumentInterface::KEYWORD_RELATIONSHIPS.'/'.$name;
 
         return $url;
     }
@@ -285,7 +285,7 @@ 
      */
     protected function getRelationshipRelatedUrl($resource, $name)
     {
-        $url = $this->getSelfSubUrl($resource) . '/' . $name;
+        $url = $this->getSelfSubUrl($resource).'/'.$name;
 
         return $url;
     }
@@ -339,7 +339,7 @@ 
             array_key_exists(self::DATA, $desc) === true ||
             array_key_exists(self::META, $desc) === true ||
             array_key_exists(self::LINKS, $desc) === true,
-            'A `' . $this->getResourceType() . ".$name` relationship must contain at least data, links or meta."
+            'A `'.$this->getResourceType().".$name` relationship must contain at least data, links or meta."
         );
 
         $data          = $desc[self::DATA] ?? null;