Change in ovirt-engine[master]: webadmin: edit vfs: dynamically maintain suggested labels
Code Review
gerrit at ovirt.org
Tue Feb 14 10:50:26 UTC 2017
>From Alona Kaplan <alkaplan at redhat.com>:
Alona Kaplan has submitted this change and it was merged.
Change subject: webadmin: edit vfs: dynamically maintain suggested labels
......................................................................
webadmin: edit vfs: dynamically maintain suggested labels
Fixing a bug where labels are being generated once during
VfsNicLabelModel's instantiation and aren't maintained throughout
the configuration of the physical function.
suggestedLabels is now maintained (updateSuggestedLabels is
triggered on onValueChange and modifies it) and the underlying
ListModels setItems are invoked with the now up-to-date
variation of suggestedLabels.
Bug-Id: https://bugzilla.redhat.com/1410322
Signed-off-by: Leon Goldberg <lgoldber at redhat.com>
Change-Id: I0f17638dd2e55ebde99bdd00a044b691e32da29b
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/AddRemoveRowWidget.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/VfsConfigModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/VfsNicLabelModel.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/VfsNicLabelWidget.java
4 files changed, 25 insertions(+), 15 deletions(-)
Approvals:
Alona Kaplan: Looks good to me, approved
Jenkins CI: Passed CI tests
Leon Goldberg: Verified
--
To view, visit https://gerrit.ovirt.org/71804
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0f17638dd2e55ebde99bdd00a044b691e32da29b
Gerrit-PatchSet: 16
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Leon Goldberg <lgoldber at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Leon Goldberg <lgoldber at redhat.com>
Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
More information about the Engine-commits
mailing list