Merge pull request #26 from GoIntegro/bug/serializationException

Error in the SerializationException and LinkedResourcesSerializer
GoIntegro · Jun 19, 2015 · fdba4b8 · fdba4b8
2 parents 4ecb046 + c2680e9
commit fdba4b8
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -9,6 +9,9 @@ php:
   # aliased to a recent 5.6.x version
   - 5.6
 
+env:
+  - SYMFONY_VERSION=2.3.*
+
 addons:
   code_climate:
     repo_token: 292a00e2f5cb75b3c35254f42c549690071404e7ffa415746b00beda15b1ddab

diff --git a/JsonApi/Serializer/LinkedResourcesSerializer.php b/JsonApi/Serializer/LinkedResourcesSerializer.php
@@ -235,10 +235,8 @@ private function processToManyRelationship(
                 if (!empty($linkedResource)) {
                     $linkedResources[] = $linkedResource;
                 } else {
-                    // @todo This should never happen.
                     throw new SerializationException(sprintf(
-                        self::ERROR_INHERITANCE_MAPPING,
-                        implode('.', $relationships)
+                        self::ERROR_INHERITANCE_MAPPING, $relationship->type
                     ));
                 }
             }

diff --git a/JsonApi/Serializer/SerializationException.php b/JsonApi/Serializer/SerializationException.php
@@ -5,7 +5,7 @@
  * @author Javier Lorenzana <javier.lorenzana@gointegro.com>
  */
 
-namespace GoIntegro\Hateoas\JsonApi\Request;
+namespace GoIntegro\Hateoas\JsonApi\Serializer;
 
 // JSON-API.
 use GoIntegro\Hateoas\JsonApi\Exception\ErrorException;