Skip to content

[DependencyInjection] Revert "Use weak references in the container" #50440

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 26, 2023
Merged

[DependencyInjection] Revert "Use weak references in the container" #50440

merged 1 commit into from
May 26, 2023

Conversation

nicolas-grekas
Copy link
Member

@nicolas-grekas nicolas-grekas commented May 26, 2023 •
edited
Loading

Q A
Branch? 6.3
Bug fix? yes
New feature? no
Deprecations? no
Tickets Fix #50439
License MIT
Doc PR -

This is the second time this issue is reported, let's revert PR #48469 and #49684 before it does more harm :)

@carsonbot carsonbot added this to the 6.3 milestone May 26, 2023
@carsonbot carsonbot changed the title Revert "[DependencyInjection] Use weak references in ContainerBuilder" [DependencyInjection] Revert " Use weak references in ContainerBuilder" May 26, 2023
@nicolas-grekas nicolas-grekas changed the title [DependencyInjection] Revert " Use weak references in ContainerBuilder" [DependencyInjection] Revert " Use weak references in the container" May 26, 2023
@nicolas-grekas nicolas-grekas changed the title [DependencyInjection] Revert " Use weak references in the container" [DependencyInjection] Revert "Use weak references in the container" May 26, 2023
@nicolas-grekas nicolas-grekas merged commit 542f039 into symfony:6.3 May 26, 2023
@nicolas-grekas nicolas-grekas deleted the di-no-ref branch May 26, 2023 08:19
@fabpot fabpot mentioned this pull request May 27, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Bug DependencyInjection Status: Needs Review
Projects
None yet
Milestone
6.3
Development

Successfully merging this pull request may close these issues.
2 participants
@nicolas-grekas @carsonbot