-
Notifications
You must be signed in to change notification settings - Fork 105
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
How to fix "Thread context was already set" #2080
Comments
Hi @piotrkwiecinski. Thank you for your report. Join Magento Community Engineering Slack and ask your questions in #github channel. |
@magento I am working on this |
@VitaliyBoyko could we leave this one open? |
@magento I am working on this |
@VitaliyBoyko I attempted to reproduce this issue in my environment but was unable to do so. First, I noticed that the recommended workaround had already been applied to the dialog window for creating a GraphQL resolver: To investigate further, I removed this fix and tried multiple approaches to reproduce the bug: Despite these efforts, I couldn’t reproduce the issue, and the reported error never appeared.
|
To properly fix this issue JDialog extend in https://github.com/magento/magento2-phpstorm-plugin/blob/5.3.0-develop/src/com/magento/idea/magento2plugin/actions/generation/dialog/AbstractDialog.java#L38 has to be replaced by DialogWrapper:
https://plugins.jetbrains.com/docs/intellij/dialog-wrapper.html?from=jetbrains.org#examples
Based on the answer from the official support: https://intellij-support.jetbrains.com/hc/en-us/community/posts/14397678486418--Thread-context-was-already-set-CoroutineName-commit-workflow-when-showing-window-during-commit-check?page=1#community_comment_15082225353874
Here is an implementation from other plugin: jshiell/checkstyle-idea@8102396
This is going to close ~60 bug reports.
The text was updated successfully, but these errors were encountered: