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

Fix return type deprecation warnings #178

Merged
merged 1 commit into from
Jul 12, 2022

Conversation

jfcherng
Copy link
Contributor

@jfcherng jfcherng commented Feb 19, 2022

Similar to #173
Resolves #171

#173 only fixes #171 (comment) but it doesn't fix OP's issue, which is my issue as well.

  • Method Doctrine\DBAL\Types\Type::convertToPHPValue() might add mixed as a native return type declaration in the future. Do the same in child class Ramsey\Uuid\Doctrine\UuidBinaryOrderedTimeType now to avoid errors or add an explicit @return annotation to suppress this message.
  • Method Doctrine\DBAL\Types\Type::convertToDatabaseValue() might add mixed as a native return type declaration in the future. Do the same in child class Ramsey\Uuid\Doctrine\UuidBinaryOrderedTimeType now to avoid errors or add an explicit @return annotation to suppress this message.

Signed-off-by: Jack Cherng <jfcherng@gmail.com>
@jfcherng jfcherng requested a review from ramsey as a code owner February 19, 2022 04:12
@tarlepp
Copy link

tarlepp commented Jul 12, 2022

@ramsey is there something that should be fixed/changed within this PR?

@ramsey
Copy link
Owner

ramsey commented Jul 12, 2022

@tarlepp Thanks for the reminder. I'll merge it in and tag a release soon.

@ramsey
Copy link
Owner

ramsey commented Jul 12, 2022

Thank you for contributing! 🎉

@ramsey ramsey merged commit af426c2 into ramsey:main Jul 12, 2022
@tarlepp
Copy link

tarlepp commented Jul 12, 2022

awesome! thank you!

@jfcherng jfcherng deleted the fix/deprecation-warning branch July 12, 2022 17:40
@ToshY
Copy link

ToshY commented Oct 30, 2022

@tarlepp Thanks for the reminder. I'll merge it in and tag a release soon.

@ramsey Please tag a release for this fix.

@ToshY ToshY mentioned this pull request Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecation notice after updating from Symfony 5.3 to 5.4
4 participants