Looking at the code, it seems that a normalizer might be called with anull value for$data, and thus it's only sensible that it be allowed to returnnull too:

symfony/src/Symfony/Component/Serializer/Serializer.php

Lines 141 to 148 in7064ff3

	// If a normalizer supports the given data, use it
	if ($normalizer =$this->getNormalizer($data,$format,$context)) {
	return$normalizer->normalize($data,$format,$context);
	}

	if (null ===$data \|\|is_scalar($data)) {
	return$data;
	}

Updating the phpdoc to match.

Allow returning null from NormalizerInterface::normalize

1c8edc5

teohhanhui requested a review fromdunglas as acode owner

November 12, 2019 17:56

carsonbot added Status: Needs Review Bug labels

Nov 12, 2019

nicolas-grekas added this to the3.4 milestone

Nov 13, 2019

nicolas-grekas approved these changes

Nov 13, 2019

View reviewed changes

carsonbot added Status: Reviewed and removed Status: Needs Review labels

Nov 13, 2019

fabpot pushed a commit that referenced this pull request

Nov 13, 2019

minor#34340Allow returning null from NormalizerInterface::normalize…

53dc781

… (teohhanhui)This PR was merged into the 3.4 branch.Discussion----------Allow returning null from NormalizerInterface::normalize| Q             | A| ------------- | ---| Branch?       | 3.4| Bug fix?      | yes?| New feature?  | no| Deprecations? | no| Tickets       | N/A| License       | MIT| Doc PR        | N/ALooking at the code, it seems that a normalizer might be called with a `null` value for `$data`, and thus it's only sensible that it be allowed to return `null` too:https://github.com/symfony/symfony/blob/7064ff35f2539e2c915257a50eb37839b485dbeb/src/Symfony/Component/Serializer/Serializer.php#L141-L148Updating the phpdoc to match.Commits-------1c8edc5 Allow returning null from NormalizerInterface::normalize