Merge pull request #397 from Strate/issue_367

#367 Exclude annotation not preventing attempt to find public methods when using AccessType

Author: goetas

src/JMS/Serializer/Metadata/Driver/AnnotationDriver.php

@@ -214,10 +214,10 @@ public function loadMetadataForClass(\ReflectionClass $class)
                     }
                 }
 
-                $propertyMetadata->setAccessor($accessType, $accessor[0], $accessor[1]);
 
                 if ((ExclusionPolicy::NONE === $exclusionPolicy && ! $isExclude)
                     || (ExclusionPolicy::ALL === $exclusionPolicy && $isExpose)) {
+                    $propertyMetadata->setAccessor($accessType, $accessor[0], $accessor[1]);
                     $classMetadata->addPropertyMetadata($propertyMetadata);
                 }
             }

tests/JMS/Serializer/Tests/Fixtures/GetSetObject.php

@@ -19,6 +19,7 @@
 namespace JMS\Serializer\Tests\Fixtures;
 
 use JMS\Serializer\Annotation\AccessType;
+use JMS\Serializer\Annotation\Exclude;
 use JMS\Serializer\Annotation\Type;
 use JMS\Serializer\Annotation\ReadOnly;
 
@@ -36,6 +37,12 @@ class GetSetObject
      */
     private $readOnlyProperty = 42;
 
+    /**
+     * This property should be exlcluded
+     * @Exclude()
+     */
+    private $excludedProperty;
+
     public function getId()
     {
         throw new \RuntimeException('This should not be called.');