Moved readOnly check to GraphNavigator

Serializer/GenericDeserializationVisitor.php

@@ -173,10 +173,6 @@ abstract class GenericDeserializationVisitor extends AbstractDeserializationVisi
             return;
         }
 
-        if ($metadata->readOnly) {
-	        return;
-	    }
-
         if (!$metadata->type) {
             throw new RuntimeException(sprintf('You must define a type for %s::$%s.', $metadata->reflection->getDeclaringClass()->getName(), $metadata->name));
         }

Serializer/GraphNavigator.php

@@ -114,6 +114,10 @@ final class GraphNavigator
                     continue;
                 }
 
+		        if (self::DIRECTION_DESERIALIZATION === $this->direction && $propertyMetadata->readOnly) {
+			        continue;
+			    }
+
                 // try custom handler
                 if (!$visitor->visitPropertyUsingCustomHandler($propertyMetadata, $data)) {
                     $visitor->visitProperty($propertyMetadata, $data);

Serializer/XmlDeserializationVisitor.php

@@ -196,11 +196,7 @@ class XmlDeserializationVisitor extends AbstractDeserializationVisitor
     public function visitProperty(PropertyMetadata $metadata, $data)
     {
         $name = $this->namingStrategy->translateName($metadata);
-        
-        if ($metadata->readOnly) {
-	        return;
-	    }
-	
+
         if (!$metadata->type) {
             throw new RuntimeException(sprintf('You must define a type for %s::$%s.', $metadata->reflection->getDeclaringClass()->getName(), $metadata->name));
         }