* Adapted the annotated type on the GraphNavigator.php for $exclusion…

…Strategy * Removed unneeded @todo
schmittjoh · May 24, 2022 · 4c230d2 · 4c230d2
1 parent 02d8e87
commit 4c230d2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/GraphNavigator.php b/src/GraphNavigator.php
@@ -4,7 +4,7 @@
 
 namespace JMS\Serializer;
 
-use JMS\Serializer\Exclusion\ExclusionStrategyInterface;
+use JMS\Serializer\Exclusion\DisjunctExclusionStrategy;
 
 /**
  * Handles traversal along the object graph.
@@ -32,7 +32,7 @@ abstract class GraphNavigator implements GraphNavigatorInterface
      */
     protected $format;
     /**
-     * @var ExclusionStrategyInterface
+     * @var DisjunctExclusionStrategy
      */
     protected $exclusionStrategy;
 

diff --git a/src/Metadata/Driver/AnnotationDriver.php b/src/Metadata/Driver/AnnotationDriver.php
@@ -186,7 +186,6 @@ public function loadMetadataForClass(\ReflectionClass $class): ?BaseClassMetadat
                         $propertyMetadata->serializedName = $annot->name;
                     } elseif ($annot instanceof SkipWhenEmpty) {
                         $propertyMetadata->skipWhenEmpty = true;
-                        // @Todo: Check the context about the configuration whether skipWhenEmpty on all properties.
                     } elseif ($annot instanceof Expose) {
                         $isExpose = true;
                         if (null !== $annot->if) {