Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[TypeInfo] Fix create union with nullable type#59818

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

Merged

Conversation

mtarld
Copy link
Contributor

QA
Branch?7.2
Bug fix?yes
New feature?no
Deprecations?no
Issues
LicenseMIT

Fix the creation of a union type with a nullable type, such as

Type::union(Type::nullable(Type::int()), Type::string());// it will be handled the same as:Type::nullable(Type::int(), Type::string())Type::union(Type::int(), Type::string(), Type::null())

@nicolas-grekas
Copy link
Member

Thank you@mtarld.

@nicolas-grekasnicolas-grekas merged commit33f70b7 intosymfony:7.2Feb 20, 2025
9 of 11 checks passed
@mtarldmtarld deleted the fix/union-factory-nullable-type branchFebruary 20, 2025 12:03
mtarld pushed a commit to mtarld/symfony that referenced this pull requestFeb 21, 2025
…rld)This PR was merged into the 7.2 branch.Discussion----------[TypeInfo] Fix create union with nullable type| Q             | A| ------------- | ---| Branch?       | 7.2| Bug fix?      | yes| New feature?  | no| Deprecations? | no| Issues        || License       | MITFix the creation of a union type with a nullable type, such as```phpType::union(Type::nullable(Type::int()), Type::string());// it will be handled the same as:Type::nullable(Type::int(), Type::string())Type::union(Type::int(), Type::string(), Type::null())```Commits-------a5f779c [TypeInfo] Fix create union with nullable type
@fabpotfabpot mentioned this pull requestFeb 26, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@chalasrchalasrchalasr approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
7.2
Development

Successfully merging this pull request may close these issues.

4 participants
@mtarld@nicolas-grekas@chalasr@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp