Skip to content

#1802 GUI State management implementation#1807

Draft
laim2003 wants to merge 14 commits intodevonfw:mainfrom
laim2003:#1802-state-management-implementation
Draft

#1802 GUI State management implementation#1807
laim2003 wants to merge 14 commits intodevonfw:mainfrom
laim2003:#1802-state-management-implementation

Conversation

@laim2003
Copy link
Copy Markdown
Contributor

@laim2003 laim2003 commented Apr 9, 2026

This PR fixes #1802

Implemented changes:

  • Implemented basic State Management for the IdeContext in the GUI
  • State management should be thread safe: Thread safe singleton implementation
  • Added custom adaption of JavaFXs alert class for IDEasy

Checklist for this PR

Make sure everything is checked before merging this PR. For further info please also see
our DoD.

  • When running mvn clean test locally all tests pass and build is successful
  • PR title is of the form #«issue-id»: «brief summary» (e.g. #921: fixed setup.bat). If no issue ID exists, title only.
  • PR top-level comment summarizes what has been done and contains link to addressed issue(s)
  • PR and issue(s) have suitable labels
  • Issue is set to In Progress and assigned to you or there is no issue (might happen for very small PRs)
  • You followed all coding conventions
  • You have added the issue implemented by your PR in CHANGELOG.adoc unless issue is labeled
    with internal

laim2003 added 13 commits March 27, 2026 17:52
- Added logging to IdeGuiStateManager.
- Added functionality, that selecting a different project now switches the IdeContext to the new project.
- Added logging to IdeGuiStateManager.
- Added functionality, that selecting a different project now switches the IdeContext to the new project.
- Added functionality, that selecting a different project now switches the IdeContext to the new project.
@github-project-automation github-project-automation bot moved this to 🆕 New in IDEasy board Apr 9, 2026
@laim2003 laim2003 self-assigned this Apr 9, 2026
@laim2003 laim2003 added GUI Graphical User Interface of IDEasy (aka dashboard) build with JavaFx bugfix PR that fixes a bug issue core FileAccess, ProcessUtil, IdeContext, etc. labels Apr 9, 2026
@laim2003 laim2003 moved this from 🆕 New to 🏗 In progress in IDEasy board Apr 9, 2026
- added DI for IdeGuiStateManager.switchContext
@coveralls
Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 24198413112

Coverage decreased (-0.05%) to 70.425%

Details

  • Coverage decreased (-0.05%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 26 coverage regressions across 5 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

26 previously-covered lines in 5 files lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/ide/gui/App.java 13 0.0%
com/devonfw/ide/gui/MainController.java 7 66.25%
com/devonfw/tools/ide/git/repository/RepositoryCommandlet.java 3 70.98%
com/devonfw/ide/gui/IdeGuiContext.java 2 50.0%
com/devonfw/ide/gui/AppLauncher.java 1 0.0%

Coverage Stats

Coverage Status
Relevant Lines: 15109
Covered Lines: 11089
Line Coverage: 73.39%
Relevant Branches: 6690
Covered Branches: 4263
Branch Coverage: 63.72%
Branches in Coverage %: Yes
Coverage Strength: 3.09 hits per line

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix PR that fixes a bug issue core FileAccess, ProcessUtil, IdeContext, etc. GUI Graphical User Interface of IDEasy (aka dashboard) build with JavaFx

Projects

Status: 🏗 In progress

Development

Successfully merging this pull request may close these issues.

Enable state management for the IdeContext in the GUI.

2 participants