You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@struts.apache.org by "lukaszlenart (via GitHub)" <gi...@apache.org> on 2026/04/05 08:16:18 UTC
[PR] feat: disable deprecated Graph editor tab by default [struts-intellij-plugin]
lukaszlenart opened a new pull request, #64:
URL: https://github.com/apache/struts-intellij-plugin/pull/64
## Summary
- Disable the deprecated Graph editor tab by default to prevent IDE freezes caused by the legacy `GraphBuilder`/`GraphBuilderFactory` APIs on newer IntelliJ Platform versions.
- Gate the feature behind a JVM system property (`-Dcom.intellij.struts2.enableGraphEditor=true`) so users who need the graph view can still opt in.
- Document the opt-in flag in README and CHANGELOG.
## Changes
- **`Struts2GraphFileEditorProvider.java`**: Add early `isGraphEditorEnabled()` check in `accept()` that returns `false` when the JVM property is absent or `false`.
- **`README.md`**: Add "Optional features" section explaining how to re-enable the graph tab.
- **`CHANGELOG.md`**: Add entry under `[Unreleased]`.
## Test plan
- [ ] Build the plugin (`./gradlew compileJava`) -- verified, passes with no new warnings
- [ ] Open a `struts.xml` file without the JVM flag -- `Graph` tab should not appear
- [ ] Start IDE with `-Dcom.intellij.struts2.enableGraphEditor=true` -- `Graph` tab should appear
- [ ] With the flag enabled, verify the graph tab behaves as before
Made with [Cursor](https://cursor.com)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@struts.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat: disable deprecated Graph editor tab by default [struts-intellij-plugin]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #64:
URL: https://github.com/apache/struts-intellij-plugin/pull/64#issuecomment-4188516651
<!-- plugin-artifact-comment -->
🔌 **Plugin artifact ready for testing!**
Download from [Actions artifacts](https://github.com/apache/struts-intellij-plugin/actions/runs/23997577875#artifacts)
Artifact: `struts2-261.18970.1`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@struts.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] feat: disable deprecated Graph editor tab by default [struts-intellij-plugin]
Posted by "lukaszlenart (via GitHub)" <gi...@apache.org>.
lukaszlenart merged PR #64:
URL: https://github.com/apache/struts-intellij-plugin/pull/64
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@struts.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org