Oved Ourfali has submitted this change and it was merged. Change subject: userportal: SearchResultsLimit race condition (#853904) ......................................................................
userportal: SearchResultsLimit race condition (#853904) https://bugzilla.redhat.com/853904 SearchableListModel - a race condition in SearchResultsLimit configuration value fetching prevented auto-refresh mechanism. * SearchableListModel -> Search() If 'SearchResultsLimit' isn't fetched yet, 'searchRequested' flag is marked on RequestSearch(). * RequestSearch() Clear model's items and raise changed events (affects only for first search). * 'GetSearchResultsLimit' -> OnSuccess() Search command should be invoked iff 'searchRequested' and model's timer isn't running (since Search starts the timer). Checking 'getIsTimerDisabled' is not sufficient since the timer is being started only *after* SearchResultsLimit value is fetched. * MainTabBasicDetailsView Add clear() method for cleaning details pane. Change-Id: Ibac4f4126feadbaec90ecc4981a9a23079ee715a Signed-off-by: Daniel Erez <[email protected]> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SearchableListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/tags/TagListModel.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/main/presenter/tab/basic/MainTabBasicDetailsPresenterWidget.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/main/view/tab/basic/MainTabBasicDetailsView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/footer/AlertsEventsFooterView.java 5 files changed, 41 insertions(+), 5 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8181 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibac4f4126feadbaec90ecc4981a9a23079ee715a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Tomas Jelinek <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
