Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Typo fix in the Serializer deserialization example for existing object #6552

Merged

Conversation

fre5h
Copy link
Contributor

@fre5h fre5h commented May 9, 2016

Q A
Doc fix? yes
New docs? no
Applies to 2.7+
Fixed tickets N/A

Just found a small typo in the comment with result values of deserialization. As the init value of the age of the person is 99, then after deserializing the existing person object it is 69. But in comment it is still 99 // $jsonContent contains {"name":"foo","age":99,"sportsman":false}

@xabbuh
Copy link
Member

xabbuh commented May 9, 2016

👍

@wouterj
Copy link
Member

wouterj commented May 13, 2016

Whoa, very nice catch Artem! Thanks for submitting this fix.

@wouterj wouterj merged commit 44ecd19 into symfony:2.7 May 13, 2016
wouterj added a commit that referenced this pull request May 13, 2016
…ting object (fre5h)

This PR was merged into the 2.7 branch.

Discussion
----------

Typo fix in the Serializer deserialization example for existing object

| Q              | A
| ------------- | ---
| Doc fix?          | yes
| New docs?     | no
| Applies to       | 2.7+
| Fixed tickets   | N/A

Just found a small typo in the comment with result values of deserialization. As the init value of the `age` of the `person` is 99, then after deserializing the existing `person` object it is 69. But in comment it is still 99 `// $jsonContent contains {"name":"foo","age":99,"sportsman":false}`

Commits
-------

44ecd19 Typo fix in the Serializer deserialization example
@fre5h fre5h deleted the typo-fix-in-serializer-deserialization-example branch June 2, 2021 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants