Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[Serializer] ObjectNormalizer: don't serialize static methods and props#16546
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
dunglas commentedNov 14, 2015
| Q | A |
|---|---|
| Bug fix? | yes |
| New feature? | no |
| BC breaks? | no |
| Deprecations? | no |
| Tests pass? | yes |
| Fixed tickets | #16485 |
| License | MIT |
| Doc PR | n/a |
dunglas commentedNov 14, 2015
Maybe should I open another PR for the PropertyAccess Component to prevent it from trying to access static methods/properties? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
wrong cs change
dunglas commentedNov 14, 2015
CS fixed. Travis failure is not related. |
dunglas commentedNov 17, 2015
ping @symfony/deciders |
etiennerunge commentedNov 17, 2015
yeah thanks@dunglas to take seriously my problem! |
stof commentedNov 17, 2015
makes sense to me |
Tobion commentedNov 17, 2015
👍 Status: Reviewed |
xabbuh commentedNov 17, 2015
👍 |
fabpot commentedNov 17, 2015
Thank you@dunglas. |
…ods and props (dunglas)This PR was squashed before being merged into the 2.7 branch (closes#16546).Discussion----------[Serializer] ObjectNormalizer: don't serialize static methods and props| Q | A| ------------- | ---| Bug fix? | yes| New feature? |no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |#16485| License | MIT| Doc PR | n/aCommits-------1fab27b [Serializer] ObjectNormalizer: don't serialize static methods and props