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

feat: replace TYPO3 inject annotations with setter methods #2760

Merged
merged 5 commits into from
Dec 19, 2021

Conversation

helsner
Copy link
Collaborator

@helsner helsner commented Dec 14, 2021

Resolves: #1780

@helsner
Copy link
Collaborator Author

helsner commented Dec 14, 2021

I'm currently having issues with my local debugger so i cannot look further into why the replacement does not work

@sabbelasichon
Copy link
Owner

@helsner will have a look now

@helsner
Copy link
Collaborator Author

helsner commented Dec 19, 2021

Don't worry
We can do that after Christmas as well 😅
At least I will get back on rector then

@sabbelasichon sabbelasichon changed the title [WIP] feat: replace TYPO3 inject annotations with setter methods feat: replace TYPO3 inject annotations with setter methods Dec 19, 2021
@sabbelasichon
Copy link
Owner

@helsner What do you think, is it ready to ship?

@sabbelasichon sabbelasichon merged commit 97315fa into main Dec 19, 2021
@sabbelasichon sabbelasichon deleted the issue-1780-fix branch December 19, 2021 22:21
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.

Breaking: #90799 - Dependency injection with non-public properties has been removed
3 participants