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