Ticket #18 (closed: fixed)
Basic input screen does not show newly added group
Reported by: | shermanl | Owned by: | administrator |
---|---|---|---|
Priority: | normal | Milestone: | Ecopath 6: build 6.0.7 |
Component: | Ecopath | Version: | |
Severity: | minor | Keywords: | |
Cc: |
Description
When a new group has been added to the group list, the basic input screen does not show this new group while other screens do.
Edit Groups
+ to add groups and modify name
Basic input screen...
will not show newly added groups
Actual Bug
Changes to the group list are stored in the ecopath model in the database. The core reloads the ecopath model and the Core Interface layer is fully updated; the new group information is available to the GUI.
The GUI screens however do not refresh their content. Only when newly opened a GUI screen will display the new group, which is why DietComp? (which would have not been open when the group was added, otherwise it would have shown the same behaviour) shows the group when opened.
Solution
It has once been decided to close all forms whenever the core lists such as groups and fleets have been manipulated. This must be implemented/fixed.
Change History
comment:2 Changed 16 years ago by joeh
JoeH: we should refresh the Basic Input screen to display the newly added groups once the user click OK in the Edit Group UI. Then the new behavior will be consistent with the behavior of Edit Multi Stanza UI, i.e., after clicking the OK button in the Edit Multi Stanza UI, the basic input screen get updated.
Jeroen: For simplicity reasons it was decided to bluntly close all input screens rather than refreshing them many months ago. This may be a good moment to open the discussion again in tomorrow's meeting before doing any work on it.
comment:3 Changed 16 years ago by jeroens
I've built a solution that makes grid reload their content whenever core data has been added and/or removed. At the time of writing, this fix is limited to refreshing grids only, and not all EwE grids are yet happy with this treatment. This will require testing on a per-grid basis.
has duplicate #19