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

Fixed missing return types for DebugTemplate #406

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

alongosz
Copy link
Member

@alongosz alongosz commented Apr 17, 2024

🎫 Issue N/A

This is a backport of ibexa/core#355 as the issue exists on 3.3.

Twig 3.9.x introduced return type to an internal class, which we extend. As a current issue resolution I have added return types where applicable.

As said above - the base Twig Template is internal - so we shouldn't really extend it. It's debatable if that legacy code is still needed or if even if it's working (TBD), however for now we provided the quick fix.

@Steveb-p Steveb-p added Bug Something isn't working Ready for review labels Apr 17, 2024
Copy link

sonarcloud bot commented Apr 17, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@alongosz alongosz requested a review from a team April 17, 2024 14:06
@alongosz alongosz merged commit 81464c6 into 1.3 Apr 17, 2024
24 checks passed
@alongosz alongosz deleted the fix-internal-template-strict-return-type branch April 17, 2024 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Ready for review
Development

Successfully merging this pull request may close these issues.

5 participants