Change in ovirt-engine[master]: frontend: Fix "Port Mirroring' property is editable although...

Code Review gerrit at ovirt.org
Tue Jul 25 12:55:49 UTC 2017


>From Alexander Wels <awels at redhat.com>:

Alexander Wels has submitted this change and it was merged. ( https://gerrit.ovirt.org/79780 )

Change subject: frontend: Fix "Port Mirroring' property is editable although the 'passthrough' property is marked on edit vNIC profile dialog
......................................................................


frontend: Fix "Port Mirroring' property is editable although the
'passthrough' property is marked on edit vNIC profile dialog

Added check for previous state of portMirroring entity so it is not
overwritten by the new state.

Change-Id: I4919ffa66176e5521bed13bfc84979eb3ae90062
Signed-off-by: Ales Musil <amusil at redhat.com>
Bug-Url: https://bugzilla.redhat.com/1390930
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/VnicProfileModel.java
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Ales Musil: Verified
  Alexander Wels: Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/79780
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4919ffa66176e5521bed13bfc84979eb3ae90062
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ales Musil <amusil at redhat.com>
Gerrit-Reviewer: Ales Musil <amusil at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Dominik Holler <dholler at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list