Red Hat Bugzilla – Bug 697850
Child Resources subtab refreshes twice
Last modified: 2013-09-03 12:57:00 EDT
Go to any resource that has children. Specifically navigate to its Inventory > ChildResources subtab and notice the table refreshes twice in quick succession.
We don't want to refresh or load the table twice everytime we go to the child resources page.
I'm not sure there are really two loads here.
When switching subtabs I only see one call to fetch the children (findResourceCompositesByCriteria()). When navigating directly (via, say, the back button or a browser url when not currently in the resource detail view) there are in fact two calls to findResourceCompositesByCriteria(). But, this is expected. We always call findResourceCompositesByCriteria() when loading a resource detail view, that call loads the target resource. The second call then loads the children.
paraphrased from mazz: "the issue is a visible redraw of the list grid contents. Whether or not this is tied to an unnecessary db fetch is not known. The spinner spins, though. But, at last try it did not reproduce."
I suggest we drop prio on this and move to jon3 from rhq4. Stopping work for now.
Unassigning as work is stopped
Note, this issue actually affects all subtabs (or all that implement RefreshableView, which may be all of them). There is a separate BZ for the same issue on the Configuration>Current subtab (https://bugzilla.redhat.com/show_bug.cgi?id=704742), which is a bit different than most subtabs, because it is not a Table.
Author: Jay Shaughnessy <email@example.com>
Date: Wed May 18 15:22:25 2011 -0400
For the resource and group detail views (The two level tab views) ensure
that for RefreshableViews the subtab canvas is not rendered until after
refresh() is called on the canvas. This allows the refresh method to
clear stale data from an existing canvas before initiating the async pull
of new data, preventing the user from briefly seeing the stale data and
then having the refresh kick in (which can look a lot like an unnecessary
For Tables this works well without any changes to the refresh() method.
That's because the ds invalidateCache method clears the grid. Other
views may have to ensure the canvas is presentable in the refresh impl.
This commit includes a change like this for the current resource config view.
Note that the other 3 config views (resPlugin, groupPlugin, groupConf)
already handled this as needed.
Bulk closing of old issues that are in VERIFIED state.