Merge pull request #3 from sroze/non-writable-attributes-ignored

Don't use non-setters methods as setters

Author: dunglas

Mapping/ClassMetadata.php

@@ -169,15 +169,13 @@ public function getAttributes(
 
                 if (!isset($attributes[$attributeName])) {
                     $attributes[$attributeName] = [
-                        'readable' => false,
-                        'writable' => false,
                         'description' => (new DocBlock($reflMethod))->getShortDescription(),
                     ];
                 }
 
                 if (0 === $reflMethod->getNumberOfRequiredParameters()) {
                     $attributes[$attributeName]['readable'] = true;
-                } else {
+                } elseif (0 === strpos($methodName, 'set')) {
                     $attributes[$attributeName]['writable'] = true;
                 }
             }
@@ -196,6 +194,11 @@ public function getAttributes(
 
             // populate attributes
             foreach ($attributes as $attributeName => $attribute) {
+                if (!isset($attribute['readable']) && !isset($attribute['writable'])) {
+                    unset($attributes[$attributeName]);
+                    continue;
+                }
+
                 $this->populateAttribute($attributeName, $attributes[$attributeName], $validationGroups);
             }
         }
@@ -288,8 +291,8 @@ private function populateAttribute($name, array &$attribute, array $validationGr
             $attribute['readable'] = false;
         }
 
-        if (!isset($attribute['writeable'])) {
-            $attribute['writeable'] = false;
+        if (!isset($attribute['writable'])) {
+            $attribute['writable'] = false;
         }
 
         foreach ($this->validatorClassMetadata->getPropertyMetadata($name) as $propertyMetadata) {

Tests/Behat/TestBundle/Entity/Dummy.php

@@ -55,4 +55,9 @@ public function getName()
     {
         return $this->name;
     }
+
+    public function hasRole($role)
+    {
+
+    }
 }