Daniel Erez has posted comments on this change.

Change subject: ui: fixing display of host fqdn in addHost form
......................................................................


Patch Set 7: Code-Review+1

(1 comment)

http://gerrit.ovirt.org/#/c/35214/7/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.java
File 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.java:

Line 774:                                 String base = 
nameEditor.asEditor().getSubEditor().getValue();
Line 775:                                 if (base == null) {
Line 776:                                     base = constants.empty();
Line 777:                                 }
Line 778:                                 String generatedHostName = base +
please format
Line 779:                                         "." + //$NON-NLS-1$
Line 780:                                         dhg.getDomainName();
Line 781:                                 
object.getHost().setEntity(generatedHostName);
Line 782:                             }


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I74e17d69423e1c02028f596ecafb13a21ab3e3ca
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to