Alona Kaplan has uploaded a new change for review.

Change subject: webadmin: Dragging a nic on itself tries to create a bond 
(#855770)
......................................................................

webadmin: Dragging a nic on itself tries to create a bond (#855770)

https://bugzilla.redhat.com/855770

[SetupNetworks] Dragging a nic on itself tries to create a bond,
should block the action and display an error message.

Change-Id: I9815bac6d17938d144f187d9b04898c908e607cc
Signed-off-by: Alona Kaplan <alkap...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/NetworkOperationFactory.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/99/7899/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/NetworkOperationFactory.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/NetworkOperationFactory.java
index cc43034..b7d69db 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/NetworkOperationFactory.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/NetworkOperationFactory.java
@@ -82,7 +82,7 @@
             // op2: nic
             else if (op2 instanceof NetworkInterfaceModel) {
                 NetworkInterfaceModel nic2 = (NetworkInterfaceModel) op2;
-                if (!nic1.isBonded()) {
+                if (!nic1.isBonded() && !nic1.equals(nic2)) {
 
                     boolean containsUnmanaged = containsUnmanaged(nic1) || 
containsUnmanaged(nic2);
                     if (containsUnmanaged){


--
To view, visit http://gerrit.ovirt.org/7899
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9815bac6d17938d144f187d9b04898c908e607cc
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <alkap...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to