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

[BUGFIX] Fix fatal error in QueryInterfaceDynamicReturnTypeExtension #119

Merged
merged 1 commit into from
Apr 13, 2023

Conversation

sascha-egerer
Copy link
Owner

In some rare cases (only in combination with rector?) the QueryInterfaceDynamicReturnTypeExtension failed with an exception as it has been tried to convert a Repository name to a model name even if the QueryInterface was not used in a Repository. It is checked if we're in the scope of a repository and if not we use a mixed type as we could not find out which type exactly it is.

In some rare cases (only in combination with rector?) the
QueryInterfaceDynamicReturnTypeExtension failed with an exception
as it has been tried to convert a Repository name to a model name
even if the QueryInterface was not used in a Repository.
It is checked if we're in the scope of a repository and if not
we use a mixed type as we could not find out which type exactly
it is.
Copy link
Contributor

@Kanti Kanti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This also seems to solve the problem.

@sascha-egerer sascha-egerer merged commit 52bdb11 into master Apr 13, 2023
@sascha-egerer sascha-egerer deleted the bugfix/fix-fatal-in-queryinterface-d-r-t branch February 6, 2025 07:42
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.

2 participants