Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[PropertyInfo] Split PropertyInfo\Type in children classes of \ReflectionNamedType and \ReflectionUnionType #41482

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

Closed
dunglas opened this issue Jun 1, 2021 · 12 comments

Comments

@dunglas
Copy link
Member

dunglas commented Jun 1, 2021

Description

I would like to split PropertyInfo\Type in children classes of \ReflectionUnionType and \ReflectionNamedType. This would improve the interoperability of this component with the rest of the PHP ecosystem and increase its consistency with the language.

Related to #38093 and #40457. cc @theofidry

@carsonbot
Copy link

Thank you for this issue.
There has not been a lot of activity here for a while. Has this been resolved?

@theofidry
Copy link
Contributor

No

@carsonbot carsonbot removed the Stalled label Dec 2, 2021
@carsonbot
Copy link

Thank you for this issue.
There has not been a lot of activity here for a while. Has this been resolved?

@theofidry
Copy link
Contributor

No

@carsonbot carsonbot removed the Stalled label Jun 3, 2022
@carsonbot
Copy link

Thank you for this issue.
There has not been a lot of activity here for a while. Has this been resolved?

@theofidry
Copy link
Contributor

keep it

@carsonbot carsonbot removed the Stalled label Dec 5, 2022
@carsonbot
Copy link

Thank you for this issue.
There has not been a lot of activity here for a while. Has this been resolved?

@carsonbot
Copy link

Could I get an answer? If I do not hear anything I will assume this issue is resolved or abandoned. Please get back to me <3

@dunglas
Copy link
Member Author

dunglas commented Jul 30, 2023

No

@carsonbot carsonbot removed the Stalled label Jul 30, 2023
@carsonbot
Copy link

Thank you for this issue.
There has not been a lot of activity here for a while. Has this been resolved?

@theofidry
Copy link
Contributor

@mtarld how relevant is this MR with the recent introduction of your component?

@carsonbot carsonbot removed the Stalled label Feb 7, 2024
@mtarld
Copy link
Contributor

mtarld commented Feb 7, 2024

Indeed, this is exactly one of the TypeInfo component main purposes, as you can see on the following hierarchy:
image

Therefore, I think that this issue can be closed.

@xabbuh xabbuh closed this as not planned Won't fix, can't repro, duplicate, stale Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants