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

Don`t mark property as static #1134

Open
Xsaven opened this issue Jul 9, 2024 · 4 comments
Open

Don`t mark property as static #1134

Xsaven opened this issue Jul 9, 2024 · 4 comments

Comments

@Xsaven
Copy link

Xsaven commented Jul 9, 2024

This property is static in my code, but in the node there is not a single identifying sign that it is static, so I cannot assemble the code correctly.

image
@czosel
Copy link
Collaborator

czosel commented Jul 10, 2024

Hi @Xsaven, can you share some example source code?

@Xsaven
Copy link
Author

Xsaven commented Jul 10, 2024

This is the class I'm trying to parse
image
Here is the code of the parser itself
image

@mreiden
Copy link
Contributor

mreiden commented Aug 15, 2024

I believe you are wanting the isStatic and visibility properties on the PropertyStatement object (the parent of the properties array containing the Property you highlighted in your class).

I found this counterintuitive, but the properties array is there to handle declaring multiple properties in one statement (though I never see anyone use this format): protected static ?string $test = null, $test2= 'str';

I do not think there is a bug here though.

@Xsaven
Copy link
Author

Xsaven commented Oct 8, 2024

@mreiden What do you mean you don't think there is a mistake there? The fact remains that there is no isStatic property!

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

No branches or pull requests

3 participants