Tomas Jelinek has posted comments on this change.
Change subject: webadmin: fix NPE on add NFS
......................................................................
Patch Set 1: (1 inline comment)
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
Line 1469: storageDomain.setStorageName((String)
model.getName().getEntity());
Line 1470: storageDomain.setDescription((String)
model.getDescription().getEntity());
Line 1471: storageDomain.setStorageFormat((StorageFormatType)
model.getFormat().getSelectedItem());
Line 1472:
Line 1473: connection = new StorageServerConnections();
Done
Line 1474: connection.setid(storageDomain.getStorage());
Line 1475:
Line 1476: if (isNew)
Line 1477: {
--
To view, visit http://gerrit.ovirt.org/13573
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibf0f41ceb4d58997a918674bd1919337911d09ef
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjeli...@redhat.com>
Gerrit-Reviewer: Alissa Bonas <abo...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhorn...@redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
