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

[Doctrine] replace ManagerRegistry in doctrine associations doc#18091

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
javiereguiluz merged 1 commit intosymfony:6.2fromMrYamous:doctrine/update
Mar 21, 2023

Conversation

@MrYamous
Copy link
Contributor

No description provided.

@MrYamousMrYamous changed the titlereplace ManagerRegistry in doctrine associations doc[Doctrine] replace ManagerRegistry in doctrine associations docMar 19, 2023
@OskarStark
Copy link
Contributor

Can you explain why?

@MrYamous
Copy link
ContributorAuthor

I think it can be considered as good practice to inject directlyEntityManagerInterface orFooRepository instead of Doctrine and then fetching the service we need through the container, this is also the way shown in the book & SymfonyCasts

@javiereguiluz
Copy link
Member

I agree. Injecting the entireManagerRegistry feels like an anti-pattern. Thanks Matthieu!

@javiereguiluzjaviereguiluz merged commitf35a726 intosymfony:6.2Mar 21, 2023
weaverryan pushed a commit that referenced this pull requestMar 28, 2023
…ons doc (MrYamous)This PR was merged into the 6.2 branch.Discussion----------[Doctrine] replace ManagerRegistry in doctrine associations docCommits-------e8313e6 replace ManagerRegistry with EntityManagerInterface in doctrine associations doc
weaverryan added a commit that referenced this pull requestMar 28, 2023
* 6.2:  feature#17513 [Security] Use expression for `#[IsGranted()]` subject (HypeMC)  minor#18114 update default value for framework.ide (GromNaN)  minor#18018 [Messenger] add `WorkerRateLimitedEvent` (MrYamous)  minor#18081 Update events.rst (hbgamra)  minor#18091 [Doctrine] replace ManagerRegistry in doctrine associations doc (MrYamous)  minor#18029 [HtmlSanitizer] Fix API usage on PHP config files (alexandre-daubois)  Move Notifier bridges doc to the code repo README files  Add note about custom port configuration being ignored by specific mailer transport DSN  [Notifier] Add link to readme bridges
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

5.4

Development

Successfully merging this pull request may close these issues.

4 participants

@MrYamous@OskarStark@javiereguiluz@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp