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: Remove rule for rewriting Node::getNodeType to Node::nodeType #92

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

dlubitz
Copy link
Contributor

@dlubitz dlubitz commented Nov 7, 2024

… as this is not available anymore

@bwaidelich
Copy link
Member

I guess this is good to go, but I don't understand what

this is not available anymore

means?
Wouldn't this fix calls to Node::getNodeType() if it existed in some legacy code?

@dlubitz
Copy link
Contributor Author

dlubitz commented Nov 8, 2024

Yes, but it was rewritten to $node->nodeType and we don't have this property anymore on the node

@kitsunet
Copy link
Member

kitsunet commented Nov 8, 2024

And there isn't even a direct way from Node to NodeType anymore (without additional dependencies that is)

@dlubitz
Copy link
Contributor Author

dlubitz commented Nov 8, 2024

Yeah, there might be a way to rector this, but I just wanted to remove this wrong migration as it would lead to even more confusion.

Copy link
Member

@bwaidelich bwaidelich left a comment

Choose a reason for hiding this comment

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

RIght, of course. Thanks for the explanation, makes sense

@bwaidelich bwaidelich merged commit 071852a into main Nov 8, 2024
2 checks passed
@bwaidelich bwaidelich deleted the bugfix/remove-get-nodetype branch November 8, 2024 09:24
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.

3 participants