Martin Beták has posted comments on this change.
Change subject: webadmin: Fix NPE in VmModelBehaviorBase
......................................................................
Patch Set 2: I would prefer that you didn't submit this
(1 inline comment)
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
Line 179: }
Line 180:
Line 181: public TimeZoneType getTimeZoneType() {
Line 182: VmOsType vmOsType = (VmOsType)
getModel().getOSType().getSelectedItem();
Line 183: return vmOsType == null ? null :
TimeZoneType.getTimeZoneByOs(vmOsType);
This is not correct fix, the return value of getTimeZoneType() is used to
lookup the appropriate values in TimeZoneModel and it cannot be null, you will
just trade one NPE for another. Please alter TimeZoneType.getTimeZoneByOs to
return GENERIC_TIMEZONE if the ostype is null.
Line 184: }
Line 185:
Line 186: protected void updateDomain()
Line 187: {
--
To view, visit http://gerrit.ovirt.org/15282
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iba1b40aeb51613b7283202c33b737b04f1c5a599
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: ofri masad <oma...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Martin Beták <mbe...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-Reviewer: ofri masad <oma...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
