minor #30934 [Serializer] add return type hints for ObjectPropertyListExtractorInterface (dmaicher)

fabpot · fabpot · commit 9afcc7b8337c · 2019-04-06T21:25:56.000+02:00
This PR was merged into the 4.3-dev branch. Discussion ---------- [Serializer] add return type hints for ObjectPropertyListExtractorInterface | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Fixed tickets | - | License | MIT | Doc PR | - @nicolas-grekas as just discussed here the type-hint + phpdoc cleanup as follow-up for #30904 😉 cc @joelwurtz Commits ------- 7628972 [Serializer] add return type hints for ObjectPropertyListExtractorInterface
diff --git a/src/Symfony/Component/Serializer/Extractor/ObjectPropertyListExtractor.php b/src/Symfony/Component/Serializer/Extractor/ObjectPropertyListExtractor.php
@@ -30,7 +30,7 @@ public function __construct(PropertyListExtractorInterface $propertyListExtracto
     /**
      * {@inheritdoc}
      */
-    public function getProperties($object, array $context = [])
+    public function getProperties($object, array $context = []): ?array
     {
         $class = $this->objectClassResolver ? ($this->objectClassResolver)($object) : \get_class($object);
 
diff --git a/src/Symfony/Component/Serializer/Extractor/ObjectPropertyListExtractorInterface.php b/src/Symfony/Component/Serializer/Extractor/ObjectPropertyListExtractorInterface.php
@@ -20,9 +20,8 @@ interface ObjectPropertyListExtractorInterface
      * Gets the list of properties available for the given object.
      *
      * @param object $object
-     * @param array  $context
      *
      * @return string[]|null
      */
-    public function getProperties($object, array $context = []);
+    public function getProperties($object, array $context = []): ?array;
 }

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ public function __construct(PropertyListExtractorInterface $propertyListExtracto`
`30`	`30`	`/**`
`31`	`31`	`* {@inheritdoc}`
`32`	`32`	`*/`
`33`		`- public function getProperties($object, array $context = [])`
	`33`	`+ public function getProperties($object, array $context = []): ?array`
`34`	`34`	`{`
`35`	`35`	`$class = $this->objectClassResolver ? ($this->objectClassResolver)($object) : \get_class($object);`
`36`	`36`
Original file line number	Diff line number	Diff line change
`@@ -20,9 +20,8 @@ interface ObjectPropertyListExtractorInterface`
`20`	`20`	`* Gets the list of properties available for the given object.`
`21`	`21`	`*`
`22`	`22`	`* @param object $object`
`23`		`- * @param array $context`
`24`	`23`	`*`
`25`	`24`	`* @return string[]\|null`
`26`	`25`	`*/`
`27`		`- public function getProperties($object, array $context = []);`
	`26`	`+ public function getProperties($object, array $context = []): ?array;`
`28`	`27`	`}`