Skip to content

[PropertyInfo] Mentioned support for PHP typed properties #13719

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 29, 2020
Merged

[PropertyInfo] Mentioned support for PHP typed properties #13719

merged 1 commit into from
May 29, 2020

Conversation

javiereguiluz
Copy link
Member

Fixes #12689.

@javiereguiluz javiereguiluz added this to the 5.1 milestone May 28, 2020
@javiereguiluz javiereguiluz merged commit e4d77fb into symfony:5.1 May 29, 2020
@ogizanagi ogizanagi mentioned this pull request Aug 28, 2020
Merged
fabpot added a commit to symfony/symfony that referenced this pull request Aug 28, 2020
@fabpot
bug #37971 [PropertyInfo] Backport support for typed properties (PHP …
03d494d 
…7.4) (dunglas)

This PR was merged into the 4.4 branch.

Discussion
----------

[PropertyInfo] Backport support for typed properties (PHP 7.4)

| Q             | A
| ------------- | ---
| Branch?       | 4.4 <!-- see below -->
| Bug fix?      | yes
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | N/A. <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License       | MIT
| Doc PR        | symfony/symfony-docs#13719 to be fixed

Backports #34557 as suggested.

One of my workmate hit this issue with the Serializer component and it was not obvious for him why it was not working, without hint.

Commits
-------

1b19f25 [PropertyInfo] Backport support for typed properties (PHP 7.4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
PropertyInfo Status: Needs Review
Projects
None yet
Milestone
5.1
Development

Successfully merging this pull request may close these issues.
2 participants
@javiereguiluz @carsonbot