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 <derez@redhat.com> --- 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 <derez@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek@redhat.com>