Skip to content

[MysqlToMysqli] Handle infinite loop on MysqlQueryMysqlErrorWithLinkRector when no connection instance on first dept stmt#4375

Merged
samsonasik merged 3 commits intomainfrom
handle-infinite-
Jun 29, 2023
Merged

[MysqlToMysqli] Handle infinite loop on MysqlQueryMysqlErrorWithLinkRector when no connection instance on first dept stmt#4375
samsonasik merged 3 commits intomainfrom
handle-infinite-

Conversation

@samsonasik
Copy link
Copy Markdown
Member

@samsonasik samsonasik commented Jun 29, 2023

@7thstorm this should fix the infinite loop when no connectino defined.

Ref rectorphp/rector#8025 (comment)

…ector when no connection instance on first dept stmt
@samsonasik samsonasik requested a review from TomasVotruba as a code owner June 29, 2023 08:04
}

if ($parentNode instanceof Node) {
// no higer parent than FileWithoutNamespace or Namespace_
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

useless comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ok, removed 👍

@samsonasik
Copy link
Copy Markdown
Member Author

All checks have passed 🎉 @TomasVotruba I am merging it ;)

@samsonasik samsonasik merged commit a808deb into main Jun 29, 2023
@samsonasik samsonasik deleted the handle-infinite- branch June 29, 2023 08:12
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