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

IBX-942: Add missing inner service argument #217

Merged
merged 1 commit into from
Sep 1, 2021
Merged

IBX-942: Add missing inner service argument #217

merged 1 commit into from
Sep 1, 2021

Conversation

wizhippo
Copy link
Contributor

This explicitly set the inner service so autowiring does not fail on symfony 5.2.7

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.5% 0.5% Duplication

@Steveb-p Steveb-p requested review from alongosz, adamwojs and a team August 30, 2021 19:52
@wizhippo
Copy link
Contributor Author

Related issues ibexa/oss#16, symfony/symfony#42791

This may not be needed when fixed

@SylvainGuittard SylvainGuittard changed the title COM-20124: Add missing inner service argument IBX-942: Add missing inner service argument Aug 31, 2021
@adamwojs adamwojs merged commit 114720c into ezsystems:master Sep 1, 2021
@maximstrukov
Copy link

Hello! When is it gonna be released? I can't install Ibexa now because of this bug.

@wizhippo wizhippo deleted the patch-2 branch September 12, 2021 17:16
@maximstrukov
Copy link

The latest 3.3.2 release doesn't have this patch.
Improvements:
IBX-105: Added automatic release creation on tag push (#212)
IBX-261: Updated changelog generator (#213)
IBX-704: Added support for LocationChildrenAggregation (#215)

alongosz pushed a commit that referenced this pull request Sep 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants