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

\Doctrine\Common\Collections\Collection mistakes? #6947

Closed
PhilETaylor opened this issue Jan 18, 2022 · 6 comments
Closed

\Doctrine\Common\Collections\Collection mistakes? #6947

PhilETaylor opened this issue Jan 18, 2022 · 6 comments
Labels

Comments

@PhilETaylor
Copy link

PhilETaylor commented Jan 18, 2022

Between rector/rector-src@bf2f6dd0e9a4b0abccdc345e588f1f713452848b and rector/rector-src@654b5861ca943231ffd6430062cc06d2a6f956e5 (The last day or two of changes)

something changed that now is changing all public properties to \Doctrine\Common\Collections\Collection type hinted?
Screenshot 2022-01-18 at 04 05 58

@PhilETaylor
Copy link
Author

Here is an example, a Symfony Command file, that gets changed with TypedPropertyFromToManyRelationTypeRector even though this is nothing to do with a doctrine entity or relationship
Screenshot 2022-01-18 at 04 29 06

@samsonasik
Copy link
Member

@PhilETaylor could you create failing test case PR for it? Thank you.

@PhilETaylor
Copy link
Author

Yeah sorry, was on my todo list, but it was 3am and I needed sleep.

The PR is here in rector-doctrine as that's where the source is rectorphp/rector-doctrine#76

Hope that helps.

@TomasVotruba
Copy link
Member

Thanks for the test case 👍 I was working on it yestereday, so it seems I broke it. I'll check the test and fix it

@TomasVotruba
Copy link
Member

Fixed in rectorphp/rector-doctrine#77 👍

@PhilETaylor
Copy link
Author

Awesome - thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants