Merge pull request #1762 from dunglas/bc-1547

Fix a BC break introduced by #1547

Author: dunglas

features/main/relation.feature

@@ -513,7 +513,7 @@ Feature: Relations support
     """
 
   @dropSchema
-  Scenario: Issue #1547 Passing wrong argument to a relation
+  Scenario: Passing an invalid IRI to a relation
     When I add "Content-Type" header equal to "application/ld+json"
     And I send a "POST" request to "/relation_embedders" with body:
     """
@@ -524,8 +524,11 @@ Feature: Relations support
     Then the response status code should be 400
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
-    And the JSON node "hydra:description" should contain "Expected IRI or nested document"
+    And the JSON node "hydra:description" should contain "Invalid value provided (invalid IRI?)."
 
+  @wip
+  @dropSchema
+  Scenario: Passing an invalid type to a relation
     When I add "Content-Type" header equal to "application/ld+json"
     And I send a "POST" request to "/relation_embedders" with body:
     """
@@ -536,4 +539,4 @@ Feature: Relations support
     Then the response status code should be 400
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/ld+json; charset=utf-8"
-    And the JSON node "hydra:description" should contain "Expected IRI or nested document"
+    And the JSON node "hydra:description" should contain "Invalid value provided (invalid IRI?)."

src/Serializer/AbstractItemNormalizer.php

@@ -160,6 +160,10 @@ protected function getAllowedAttributes($classOrObject, array $context, $attribu
      */
     protected function setAttributeValue($object, $attribute, $value, $format = null, array $context = [])
     {
+        if (!\is_string($attribute)) {
+            throw new InvalidArgumentException('Invalid value provided (invalid IRI?).');
+        }
+
         $propertyMetadata = $this->propertyMetadataFactory->create($context['resource_class'], $attribute, $this->getFactoryOptions($context));
         $type = $propertyMetadata->getType();
 
@@ -299,7 +303,7 @@ protected function denormalizeRelation(string $attributeName, PropertyMetadata $
 
         if (
             !$this->resourceClassResolver->isResourceClass($className) ||
-            ($propertyMetadata->isWritableLink() && \is_array($value))
+            $propertyMetadata->isWritableLink()
         ) {
             $context['resource_class'] = $className;
             $context['api_allow_update'] = true;