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

Appearance settings

[Form] Fix Guess phpdoc return type #37725

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

Merged
merged 1 commit into from
Aug 13, 2020
Merged

[Form] Fix Guess phpdoc return type #37725

merged 1 commit into from
Aug 13, 2020

Conversation

franmomu
Copy link
Contributor

@franmomu franmomu commented Aug 2, 2020

Q A
Branch? 3.4
Bug fix? no
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

Since Guess is abstract, static represents better the return type, so calling TypeGuess::getBestGuess() would return TypeGuess instead of Guess.

@franmomu franmomu requested a review from xabbuh as a code owner August 2, 2020 18:58
@yceruto yceruto added this to the 3.4 milestone Aug 3, 2020
src/Symfony/Component/Form/Guess/Guess.php Show resolved Hide resolved
@fabpot
Copy link
Member

fabpot commented Aug 13, 2020

Thank you @franmomu.

@fabpot fabpot merged commit 7241f65 into symfony:3.4 Aug 13, 2020
@franmomu franmomu deleted the patch-1 branch August 13, 2020 07:28
This was referenced Aug 31, 2020
franmomu added a commit to franmomu/SonataAdminBundle that referenced this pull request Aug 31, 2020
franmomu added a commit to franmomu/SonataAdminBundle that referenced this pull request Aug 31, 2020
OskarStark pushed a commit to sonata-project/SonataAdminBundle that referenced this pull request Sep 1, 2020
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.

6 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.