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

[VarExporter] Fix exporting classes with __serialize() but not __unserialize() #51017

Merged

Conversation

fancyweb
Copy link
Contributor

Q A
Branch? 5.4
Bug fix? yes
New feature? no
Deprecations? -
Tickets #51010
License MIT
Doc PR -

@fancyweb fancyweb force-pushed the var-export/fix-__seri-without-__unseri branch from 5547e8e to fc51414 Compare July 18, 2023 17:24
@nicolas-grekas nicolas-grekas force-pushed the var-export/fix-__seri-without-__unseri branch from fc51414 to 6d8a363 Compare July 20, 2023 07:21
@nicolas-grekas
Copy link
Member

Thank you @fancyweb.

@nicolas-grekas nicolas-grekas merged commit 89ab7f3 into symfony:5.4 Jul 20, 2023
9 of 11 checks passed
@fancyweb fancyweb deleted the var-export/fix-__seri-without-__unseri branch July 20, 2023 07:24
This was referenced Jul 29, 2023
derrabus added a commit that referenced this pull request Jul 31, 2023
This PR was merged into the 6.4 branch.

Discussion
----------

[VarExporter] Remove obsolete PHP version check

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | Follows #51017
| License       | MIT
| Doc PR        | not needed

Commits
-------

f2e0d3e [VarExporter] Remove obsolete PHP version check
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