Allon Mureinik has posted comments on this change.

Change subject: webadmin: Add dialog for attaching an attached Storage Domain 
[2]
......................................................................


Patch Set 11:

(1 comment)

http://gerrit.ovirt.org/#/c/34733/11/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDataCenterListModel.java
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDataCenterListModel.java:

Line 392:             if (a.getIsSelected()) {
Line 393:                 items.add((StoragePool) a.getEntity());
Line 394:             }
Line 395:         }
Line 396: 
Please remove all the formatting noise from this patch
Line 397:         if (items.size() == 0) {
Line 398:             cancel();
Line 399:             return;
Line 400:         }


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9e9b29e24d7c13b16ef80a84eabfe109413d46e9
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to