Liron Ar has posted comments on this change.

Change subject: core: add ability edit NFS path in webadmin
......................................................................


Patch Set 11: (1 inline comment)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/UpdateStorageServerConnectionCommand.java
Line 112:         return super.canDoAction();
Line 113:     }
Line 114: 
Line 115: 
Line 116:     protected String createDomainNamesList(List<StorageDomain> 
domains) {
bad wording, i meant to the logic here.
Line 117:              //Build domain names list to display in the error
Line 118:             StringBuilder domainNames = new StringBuilder();
Line 119:             for(StorageDomain domain : domains) {
Line 120:                 domainNames.append(domain.getStorageName());


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifaff5344ff191d6bdf53cc706c7bb796167a56b3
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alissa Bonas <[email protected]>
Gerrit-Reviewer: Alissa Bonas <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Liron Ar <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to